<div dir="ltr"><div class="gmail_extra"><div class="gmail_quote"><div>Hi Piotr,</div><div><br></div><div>I cannot give you a full answer, just a small insight into the issue.</div><div><br></div><div>At the first sight, one could thing that the problem is clang-tidy containining some checks that are only aliases (synonyms) to other checks, e.g. BracesAroundStatementsCheck is both in readability and google modules, similarly NamespaceCommentCheck is both in llvm and google.</div><div>But apparently, it's not the case, since, AFAI, modernize/UseOverrideCheck is not aliased anywhere.</div><div><br></div><div>Given that it happens in headers (and only in headers that are included from different translation units I guess (?)), it must be that the header is checked, fixes are saved (but not applied immediately), the file is checked again and the history repeats itself. Finally, the fixes from two (or more) passes are applied.</div><div><br></div><div>IMO, the fault is either in clang-tidy framework or in the check. Personally I vote for the former, as it shouldn't be the check authors' responsibility.</div><div>Unless I am mistaken, there are checks that handle this repetitive fixes manually.</div><div><br></div><div>Correct me if I am wrong.</div><div><br></div><div>Best regards,</div><div>Marek </div><div><br></div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left-width:1px;border-left-color:rgb(204,204,204);border-left-style:solid;padding-left:1ex">---------- Wiadomość przekazana dalej ----------<br>From: Piotr Padlewski via cfe-dev <<a href="mailto:cfe-dev@lists.llvm.org">cfe-dev@lists.llvm.org</a>><br>To: Clang Dev <<a href="mailto:cfe-dev@lists.llvm.org">cfe-dev@lists.llvm.org</a>><br>Cc: <br>Date: Mon, 7 Dec 2015 17:29:00 +0100<br>Subject: Re: [cfe-dev] [Clang-tidy] applying fixes multiple times on same file<br><div dir="ltr">Bump<div>I can also say that this thing takes place in header files.</div><div><br></div><div>Piotr</div></div><div class="gmail_extra"><br><div class="gmail_quote">2015-11-26 21:18 GMT+01:00 Piotr Padlewski <span dir="ltr"><<a href="mailto:piotr.padlewski@gmail.com" target="_blank">piotr.padlewski@gmail.com</a>></span>:<br><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left-width:1px;border-left-color:rgb(204,204,204);border-left-style:solid;padding-left:1ex"><div dir="ltr"><span style="font-size:12.8px">Hi,</span><div style="font-size:12.8px">So After running clang-tidy it did change some warnings very well, but some of them were fixed multiple times like:</div><div style="font-size:12.8px"><br></div><div style="font-size:12.8px"><div>-    Event(const std::shared_ptr<EventRecord> &record);</div><div>+    explicit explicit Event(const std::shared_ptr<EventRecord> &record);</div></div><div style="font-size:12.8px"><br></div><div style="font-size:12.8px"><br></div><div style="font-size:12.8px"><div>-} // ga</div><div>+} // namespace gamespace ga</div></div><div style="font-size:12.8px"><br></div><div style="font-size:12.8px"><div>-}</div><div>+}  // namespace ga // namespace ga  // namespace ga  // namespace ga</div></div><div style="font-size:12.8px"><br></div><div style="font-size:12.8px"><div>-    RecordStream<ga::Event> getRecordStream();</div><div>+    RecordStream<ga::Event> getRecordStream() override override;</div></div><div style="font-size:12.8px"><br></div><div style="font-size:12.8px">Is it known feature? Am I doing something wrong?</div><span><font color="#888888"><div style="font-size:12.8px"><br></div><div style="font-size:12.8px">Piotr</div></font></span></div>
</blockquote></div></div></blockquote><div><br></div><div> </div></div></div></div>