[cfe-dev] [Clang-tidy] applying fixes multiple times on same file
Kevin Funk via cfe-dev
cfe-dev at lists.llvm.org
Thu Dec 17 04:23:21 PST 2015
On Thursday, December 17, 2015 10:02:03 AM Piotr Padlewski via cfe-dev wrote:
> Ok thanks Don. I think it would be good to provide that information in
that's why there's the 'run-clang-tidy' python script  as part of clang-
- Parallelize the clang-tidy invocations
- Records changes-to-be-made (via `clang-tidy -export-fixes`)
- And only when joining the subprocesses: Applies the changes
Hope that helps,
> 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
Kevin Funk | kfunk at kde.org | http://kfunk.org
-------------- next part --------------
A non-text attachment was scrubbed...
Size: 181 bytes
Desc: This is a digitally signed message part.
More information about the cfe-dev