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

[Piotr Padlewski via cfe-dev]

Ok thanks Don. I think it would be good to provide that information in
documentation.
17.12.2015 7:31 AM "don hinton" <hintonda at gmail.com> napisał(a):

> 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/bca94692/attachment.html>