Re: [Libreoffice-commits] core.git: clang-tidy performance-move-const-arg

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 





On Fri, 7 Sep 2018 at 14:25, Stephan Bergmann <sbergman@xxxxxxxxxx> wrote:
For my taste, that approach is too tightly tied to a class's current
implementation details, something that may change over time.  Imagine a


Yeah, I agree, the particular clang-tidy plugin did not generate useful results:

revert scheduled with:


The reformatting here is an automatic clang thing, which it does when I use the -fix and -format-style params. But it seems to extend it's formatting around the nearest decl, so perhaps a little too enthusiastic.

The alternative is that I hand tweak all the changes, which are typically randomly indented after -fix is done with the code.
Guess I'll be doing that from now on.

_______________________________________________
LibreOffice mailing list
LibreOffice@xxxxxxxxxxxxxxxxxxxxx
https://lists.freedesktop.org/mailman/listinfo/libreoffice

[Index of Archives]     [LARTC]     [Bugtraq]     [Yosemite Forum]     [Photo]

  Powered by Linux