This is an archive of the discontinued LLVM Phabricator instance.

ClangTidy: Avoid mixing stdout with stderror when dealing with a large number of files.
ClosedPublic

Authored by Abpostelnicu on Apr 9 2019, 3:33 AM.

Details

Summary

At Mozilla we are using this tool in order to perform review-time static-analysis, since some patches contain a large number of files we've discovered this issue, where stderr gets mixed with stdout thus obfuscating our possibility to parse the output.
The patch that we are currently use can be found here.

This is just an upstream of the original patch.

Event Timeline

Abpostelnicu created this revision.Apr 9 2019, 3:33 AM
Herald added a project: Restricted Project. · View Herald TranscriptApr 9 2019, 3:33 AM
Herald added a subscriber: cfe-commits. · View Herald Transcript
Abpostelnicu added a project: Restricted Project.
Abpostelnicu edited the summary of this revision. (Show Details)Apr 9 2019, 3:38 AM
Abpostelnicu edited the summary of this revision. (Show Details)
JonasToth accepted this revision.Apr 9 2019, 4:08 AM

How does that happen to be a problem? It feels that stdout.write and stderr.write should be unconnected? (if you know the reason I would like to know it too :))
Please note, that run-clang-tidy has an export-feature for the diagnostics and fix-its which might make it easier to analyze the output of clang-tidy.

I suppose you verified the fix already, then LGTM

This revision is now accepted and ready to land.Apr 9 2019, 4:08 AM
This revision was automatically updated to reflect the committed changes.
Herald added a project: Restricted Project. · View Herald TranscriptApr 9 2019, 4:16 AM