[PATCH] D135789: Add .editorconfig file
Michał Górny via Phabricator via llvm-commits
llvm-commits at lists.llvm.org
Wed Oct 12 21:35:46 PDT 2022
mgorny added inline comments.
================
Comment at: .editorconfig:9
+trim_trailing_whitespace = true
+
----------------
MaskRay wrote:
> MaskRay wrote:
> > beanz wrote:
> > > mgorny wrote:
> > > > Does it have some key not to add empty trailing lines? ;-)
> > > The `insert_final_newline` adds that line, which is generally what we want because strict C states that all files should end with a newline. Both GCC and Clang have warnings for that.
> > Remove the trailing blank line.
> Do we need `end_of_line = lf`?
> The `insert_final_newline` adds that line, which is generally what we want because strict C states that all files should end with a newline. Both GCC and Clang have warnings for that.
But there are *two* newlines at the end of the file now.
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D135789/new/
https://reviews.llvm.org/D135789
More information about the llvm-commits
mailing list