[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