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