MarcusJohnson91 marked an inline comment as not done.
MarcusJohnson91 added inline comments.
================
Comment at: clang/lib/Format/Format.cpp:586
IO.mapOptional("SortIncludes", Style.SortIncludes);
IO.mapOptional("SortJavaStaticImport", Style.SortJavaStaticImport);
IO.mapOptional("SortUsingDeclarations", Style.SortUsingDeclarations);
----------------
MyDeveloperDay wrote:
> I'm confused by this diff... the Review is closed, if you are proposing
> changes please make a new review
I went ahead and made a new revision, thanks for the tip.
I've added you and Silvestre as reviewers.
AS for your question about the very last diff, I originally wanted to change as
little as possible, but Silvestre getting confused and thinking my patch broke
something motivated me to fix the confusion with the BraceWrapping table once
and for all, so here we are.
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D75791/new/
https://reviews.llvm.org/D75791
_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
