<div dir="ltr"><div dir="ltr"><div class="gmail_default" style="font-family:tahoma,sans-serif"><br></div><br><div class="gmail_quote"><div dir="ltr">On Fri, 7 Sep 2018 at 14:25, Stephan Bergmann <<a href="mailto:sbergman@redhat.com">sbergman@redhat.com</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">For my taste, that approach is too tightly tied to a class's current <br>
implementation details, something that may change over time.  Imagine a <br><br></blockquote><div><br></div><div class="gmail_default" style="font-family:tahoma,sans-serif">Yeah, I agree, the particular clang-tidy plugin did not generate useful results:</div><div class="gmail_default" style="font-family:tahoma,sans-serif"><br></div><div class="gmail_default" style="font-family:tahoma,sans-serif">revert scheduled with:</div><div class="gmail_default" style="font-family:tahoma,sans-serif"><br></div><div class="gmail_default" style="font-family:tahoma,sans-serif">    <a href="https://gerrit.libreoffice.org/#/c/60166/">https://gerrit.libreoffice.org/#/c/60166/</a></div><div class="gmail_default" style="font-family:tahoma,sans-serif"><br></div><div class="gmail_default" style="font-family:tahoma,sans-serif">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.</div><div class="gmail_default" style="font-family:tahoma,sans-serif"><br></div><div class="gmail_default" style="font-family:tahoma,sans-serif">The alternative is that I hand tweak all the changes, which are typically randomly indented after -fix is done with the code.</div><div class="gmail_default" style="font-family:tahoma,sans-serif">Guess I'll be doing that from now on.</div><div class="gmail_default" style="font-family:tahoma,sans-serif"><br></div></div></div></div>