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
_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

Reply via email to