[clang-tools-extra] r273786 - [clang-tidy] Add modernize-use-using

[Miklos Vajna via cfe-commits]

Hi,

On Sat, Jun 25, 2016 at 06:37:54PM -0000, Krystyna Gajczyk via cfe-commits <cfe-commits at lists.llvm.org> wrote:
> Author: krystynka
> Date: Sat Jun 25 13:37:53 2016
> New Revision: 273786
> 
> URL: http://llvm.org/viewvc/llvm-project?rev=273786&view=rev
> Log:
> [clang-tidy] Add modernize-use-using

Thanks, sounds like a useful check. :-) I just run it on a codebase that
uses cppunit for testing, and it turns out cppunit provides a number of
macros, that expand to typedefs. The macros are provided by system
headers, so it's not easy to change them. Is it possible to filter out
warnings that appear in code that's expanded from system headers?
Example:

/git/dotfiles/.vim/bundle/libclang-vim/qa/deduction.cpp:8:5: error: use 'using' instead of 'typedef' [modernize-use-using,-warnings-as-errors]
    CPPUNIT_TEST_SUITE(deduction_test);
    ^
/usr/include/cppunit/extensions/HelperMacros.h:102:5: note: expanded from macro 'CPPUNIT_TEST_SUITE'
    typedef ATestFixtureType TestFixtureType;                               \
    ^

If the problem is not clear enough, I'm happy to put together a
self-contained example, and file a bugreport if it helps.

Thanks,

Miklos
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 181 bytes
Desc: Digital signature
URL: <http://lists.llvm.org/pipermail/cfe-commits/attachments/20160626/2d71bc07/attachment.sig>