[clang] [clang-format] Separate License text and include blocks (PR #77918)

via cfe-commits cfe-commits at lists.llvm.org
Mon Jan 15 06:07:00 PST 2024


https://github.com/mydeveloperday requested changes to this pull request.

Yeah I'm kind of a not aligned on this approach of hijacking what feels like the wrong setting, it feels complicated after the change which probably means we are not thinking about it enough about what people might request in the future

```c++
//
// License
//
                            <<-- added empty line
                            <<-- added empty line  (NewLinesAfterLicense: 2)
#include <memory>
#include <string>
                            <<-- added empty line (NewLinesBetweenIncludeGroups: 2)
                            <<-- added empty line
#include <lib/memory>
#include <lib/string>
                            <<-- added empty line (NewLinesBetweenIncludeGroups: 2)
                            <<-- added empty line
#include <system/memory>
#include <system/string>
                            <<-- added empty line (NewLinesAfterIncludes: 3)
                            <<-- added empty line
                            <<-- added empty line
namespace Bar {

class Foo {

};
```

} // Bar

https://github.com/llvm/llvm-project/pull/77918


More information about the cfe-commits mailing list