[PATCH] D51036: clang-format: Fix formatting C++ namespaces with preceding 'inline' or 'export' specifier
    Sam McCall via Phabricator via cfe-commits 
    cfe-commits at lists.llvm.org
       
    Fri Aug 24 01:30:43 PDT 2018
    
    
  
sammccall added inline comments.
================
Comment at: unittests/Format/FormatTest.cpp:7582
                Style);
+  verifyFormat("export namespace Foo\n"
+               "{};",
----------------
you may want to add tests for other modules TS syntax (e.g. non-namespace export decls).
It seems this works well today, but without tests it could regress.
Repository:
  rC Clang
https://reviews.llvm.org/D51036
    
    
More information about the cfe-commits
mailing list