asmith marked 2 inline comments as done. asmith added inline comments.
================ Comment at: clang/lib/Format/Format.cpp:1143 Style.PenaltyReturnTypeOnItsOwnLine = 1000; + Style.AllowShortEnumsOnASingleLine = (Language != FormatStyle::LK_Cpp); Style.AllowShortFunctionsOnASingleLine = FormatStyle::SFS_None; ---------------- MyDeveloperDay wrote: > asmith wrote: > > MyDeveloperDay wrote: > > > I think its ok to leave it was, just if up the unit tests > > > > > > (the fact you didn't see this suggests you are not running the unit tests > > > please ensure you are) > > You need to use the latest changes. The is only applied when the style is > > Msft and C++. See the GetMicrosoftStyle() method. > I don't think you should change the style based on the language here, either > on or off, but not changing based on language because what about "ObjectiveC"? > > As I added the Microsoft style to allow C# development I think its ok to set > this to what you originally had, but fix up the tests Ive enabled this for C#. I don't see the connection between ObjectiveC and Microsoft Style. CHANGES SINCE LAST ACTION https://reviews.llvm.org/D78982/new/ https://reviews.llvm.org/D78982 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
