<div dir="ltr"><div><div><div>Hey all,<br><br></div>would it make sense to add these tools to the <a href="https://clang.llvm.org/docs/ExternalClangExamples.html">External Clang Examples</a> page? Which is a file in the `docs` directory of the Clang repo.<br><br></div>Regards,<br></div>Laszlo<br></div><div class="gmail_extra"><br><div class="gmail_quote">On Thu, Feb 16, 2017 at 4:55 AM, Sérgio Martins via cfe-dev <span dir="ltr"><<a href="mailto:cfe-dev@lists.llvm.org" target="_blank">cfe-dev@lists.llvm.org</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><span class="">On Tue, Feb 14, 2017 at 7:33 PM, Richard Braun via cfe-dev<br>
<<a href="mailto:cfe-dev@lists.llvm.org">cfe-dev@lists.llvm.org</a>> wrote:<br>
> Hello,<br>
><br>
> During work on an internal Qt-based project at Novasys-Ingenierie,<br>
> a Clang plugin was written to convert string-based signals and slots<br>
> to the Qt5 syntax. The plugin has since been pushed to Github [1]<br>
> in the hope it will be useful to others facing similar issues.<br>
<br>
</span>Hi Richard,<br>
<br>
Beware that the new PMF syntax is not a drop in replacement for the old style.<br>
Applying these fixits on a codebase might introduce bugs.<br>
<br>
Here's some things to watch out for:<br>
<br>
- You can't disconnect with new-syntax if the connect was made with<br>
old-syntax (and vice-versa)<br>
- Difference in behaviour when calling slots of partially destroyed<br>
objects (explained in <<a href="https://codereview.qt-project.org/#/c/83800" rel="noreferrer" target="_blank">https://codereview.qt-<wbr>project.org/#/c/83800</a>>,<br>
not sure if it will be merged. <- Thiago ??)<br>
- Old style is capable of making functions "virtual" without using the<br>
virtual keyword.<br>
<br>
<br>
Just for fun, can you run your plugin on clazy's unit-tests<br>
(clazy/tests/old-style-<wbr>connect/*.cpp) and compared them to<br>
*fixed.cpp.expected, to see if it converted the connects correctly ?<br>
<br>
<br>
Regards,<br>
Sérgio Martins<br>
<div class="HOEnZb"><div class="h5">______________________________<wbr>_________________<br>
cfe-dev mailing list<br>
<a href="mailto:cfe-dev@lists.llvm.org">cfe-dev@lists.llvm.org</a><br>
<a href="http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-dev" rel="noreferrer" target="_blank">http://lists.llvm.org/cgi-bin/<wbr>mailman/listinfo/cfe-dev</a><br>
</div></div></blockquote></div><br></div>