Just one looks actually broken / incorrect: http://llvm.org/viewvc/llvm-project/clang-tools-extra/trunk/modularize/Modularize.cpp?rev=198703&r1=198702&r2=198703&view=diff
It looks like the API header search in sort_includes.py is broken/incorrect in this case. It assumes the first include in any 'cpp' file is correct even if it does not match name or section at all. The script should probably search for a matching name in the last chunk of the path (Although one header with multiple implementing independently named cpp files would break it then)
_______________________________________________ cfe-commits mailing list [email protected] http://lists.cs.uiuc.edu/mailman/listinfo/cfe-commits
