-
Notifications
You must be signed in to change notification settings - Fork 386
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Use of deprecated ConstructorInitializerAllOnOneLineOrOnePerLine .clang-format option #169
Comments
IMHO: we should use this: https://clang.llvm.org/docs/ClangFormatStyleOptions.html#packconstructorinitializers |
Possibly, I'm not well versed in the "most commonly admitted consensual .clang-format practices" so I'll let the decision to others. I was just looking through the definitions and I saw something deprecated which is not the best default i believe for a modern starter. As a digression I went through the options because originally i wanted to have the & "on the left" for function definitions like
I also don't know if this "left alignment" is a standard practice. |
Thanks for the heads up! I wasn't aware that the option was deprecated. Happy to accept a PR that updates the config without impacting the other style settings much. |
ModernCppStarter/.clang-format
Line 9 in f0b5bdb
Documentation says it's deprecated
I think what you want is
The text was updated successfully, but these errors were encountered: