HazardyKnusperkeks accepted this revision.
HazardyKnusperkeks added a comment.
This revision is now accepted and ready to land.

Looks good to me, I would just change the wording a bit. Could you please also 
add a entry in the ReleaseNotes.rst?



================
Comment at: clang/include/clang/Format/Format.h:2663
+  /// The number of columns to use for indentation of preprocessor statements.
+  /// When set to -1 this defaults to IndentWidth for backwards compatibility.
+  /// \code
----------------
Not only backwards compatibility, but you can always choose to match those two.


Repository:
  rG LLVM Github Monorepo

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D103286/new/

https://reviews.llvm.org/D103286

_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

Reply via email to