hokein added inline comments.
================
Comment at: unittests/include-fixer/IncludeFixerTest.cpp:144
@@ -141,1 +143,3 @@
+ runIncludeFixer("a::b::foo bar;\n",
+ /*FixNamespaceQualifiers=*/true, IncludePath));
----------------
djasper wrote:
> I think we should set this to true everywhere or more precisely completely
> remove this parameter. Now that the flag is gone, we are starting to test
> implementation details IMO.
Yeah, this parameter is not needed. Removed it.
http://reviews.llvm.org/D21603
_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
