[cfe-dev] [Clang-tidy] applying fixes multiple times on same file

don hinton via cfe-dev cfe-dev at lists.llvm.org
Wed Dec 16 22:31:51 PST 2015


Are you trying to run it on more than one cpp file at a time?

I think it precesses all the files first and comes up with a big list of
changes, then applies them.  Which means you could get duplicate changes
when processing multiple cpp files that include the same headers.

I just do them one at a time.
On Dec 7, 2015 11:29 AM, "Piotr Padlewski via cfe-dev" <
cfe-dev at lists.llvm.org> wrote:

> Bump
> I can also say that this thing takes place in header files.
>
> Piotr
>
> 2015-11-26 21:18 GMT+01:00 Piotr Padlewski <piotr.padlewski at gmail.com>:
>
>> Hi,
>> So After running clang-tidy it did change some warnings very well, but
>> some of them were fixed multiple times like:
>>
>> -    Event(const std::shared_ptr<EventRecord> &record);
>> +    explicit explicit Event(const std::shared_ptr<EventRecord> &record);
>>
>>
>> -} // ga
>> +} // namespace gamespace ga
>>
>> -}
>> +}  // namespace ga // namespace ga  // namespace ga  // namespace ga
>>
>> -    RecordStream<ga::Event> getRecordStream();
>> +    RecordStream<ga::Event> getRecordStream() override override;
>>
>> Is it known feature? Am I doing something wrong?
>>
>> Piotr
>>
>
>
> _______________________________________________
> cfe-dev mailing list
> cfe-dev at lists.llvm.org
> http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-dev
>
>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.llvm.org/pipermail/cfe-dev/attachments/20151217/5cc2d78c/attachment.html>


More information about the cfe-dev mailing list