[clang-format] Add InsertNewlineAtEOF to .clang-format files

[owenpan]

Summary

Also, reformat all clang-format related files.

[owenpan]

Took out Language: Cpp.

[HazardyKnusperkeks]

We also could add a clang-format style, the you wouldn't have to to touch all .clang-format files. ;)

[owenpan]

In D153208#4431239, @HazardyKnusperkeks wrote:

We also could add a clang-format style, the you wouldn't have to to touch all .clang-format files. ;)

Something like BasedOnStyle: clang-format? I like it but feel that it should be an undocumented feature. What do you all think?

[HazardyKnusperkeks]

In D153208#4431697, @owenpan wrote:

In D153208#4431239, @HazardyKnusperkeks wrote:

We also could add a clang-format style, the you wouldn't have to to touch all .clang-format files. ;)

Something like BasedOnStyle: clang-format? I like it but feel that it should be an undocumented feature. What do you all think?

I don't know if I want this or if it just a joke. But if we do it, I would document it, something like. "Basically LLVM Style, which uses some of the newer options, to enforce some of the LLVM style guide rules."

[owenpan]

In D153208#4433511, @HazardyKnusperkeks wrote:

In D153208#4431697, @owenpan wrote:

In D153208#4431239, @HazardyKnusperkeks wrote:

We also could add a clang-format style, the you wouldn't have to to touch all .clang-format files. ;)

Something like BasedOnStyle: clang-format? I like it but feel that it should be an undocumented feature. What do you all think?

I don't know if I want this or if it just a joke. But if we do it, I would document it, something like. "Basically LLVM Style, which uses some of the newer options, to enforce some of the LLVM style guide rules."

That would seem odd with LLVM, Google, and other current BasedOnStyle values unless we make it hidden for "internal" use. I'll just land this patch as is and not worry about it now.

clang/include/clang/Format/.clang-format

 BasedOnStyle: LLVM
 InsertBraces: true
+InsertNewlineAtEOF: true
 LineEnding: LF
 RemoveBracesLLVM: true

clang/lib/Format/.clang-format

 BasedOnStyle: LLVM
 InsertBraces: true
+InsertNewlineAtEOF: true
 LineEnding: LF
 RemoveBracesLLVM: true

clang/tools/clang-format/.clang-format

 BasedOnStyle: LLVM
 InsertBraces: true
+InsertNewlineAtEOF: true
 LineEnding: LF
 RemoveBracesLLVM: true

clang/unittests/Format/.clang-format

 BasedOnStyle: LLVM
 InsertBraces: true
+InsertNewlineAtEOF: true
 LineEnding: LF
 RemoveBracesLLVM: true

clang/unittests/Format/FormatTestMacroExpansion.cpp

TEST_F(FormatTestMacroExpansion,
   verifyIncompleteFormat("O(auto x = [](){\n"
                          "    f();}",
                          Style);
 }
 
 } // namespace
 } // namespace test
 } // namespace format
-} // namespace clang
+} // namespace clang