Re: [PATCH] D14592: Qt (version 4 or 5) signal/method checker

2015-11-16 Thread Eugene Zelenko via cfe-commits
Eugene.Zelenko added a subscriber: Eugene.Zelenko. Eugene.Zelenko added a comment. There are dedicated project to Qt checks: https://github.com/KDE/clazy. http://reviews.llvm.org/D14592 ___ cfe-commits mailing list cfe-commits@lists.llvm.org http://

Re: [PATCH] D14592: Qt (version 4 or 5) signal/method checker

2015-11-11 Thread Evgeniy Dushistov via cfe-commits
Dushistov added inline comments. Comment at: lib/StaticAnalyzer/Checkers/QtSignalSlotChecker.cpp:114 @@ +113,3 @@ + printMethodNameWithPramaTypes(Out, C, FName, M, false); + const std::string NS = qtNormalizeSignature(Out.str()); +

Re: [PATCH] D14592: Qt (version 4 or 5) signal/method checker

2015-11-11 Thread Evgeniy Dushistov via cfe-commits
On Wed, Nov 11, 2015 at 11:58:34PM +, Jonathan Roelofs wrote: > jroelofs added a subscriber: jroelofs. > > > Comment at: lib/StaticAnalyzer/Checkers/QtSignalSlotChecker.cpp:114 > @@ +113,3 @@ > + printMethodNameWithPramaTypes(Out, C, FName, M, > false)

Re: [PATCH] D14592: Qt (version 4 or 5) signal/method checker

2015-11-11 Thread Jonathan Roelofs via cfe-commits
jroelofs added a subscriber: jroelofs. Comment at: lib/StaticAnalyzer/Checkers/QtSignalSlotChecker.cpp:114 @@ +113,3 @@ + printMethodNameWithPramaTypes(Out, C, FName, M, false); + const std::string NS = qtNormalizeSignature(Out.str

[PATCH] D14592: Qt (version 4 or 5) signal/method checker

2015-11-11 Thread Evgeniy Dushistov via cfe-commits
Dushistov created this revision. Dushistov added reviewers: dcoughlin, Ayal, xazax.hun, zaks.anna. Dushistov added a subscriber: cfe-commits. In Qt 4/5 it is possible connect classes methods in such way: connect(ObjectPointer1, SIGNAL(methodOfObject1()), ObjectPointer2, SLOT(methodOfObject2());