Page MenuHomePhabricator

Refactor DependencyScanningTool to its own file

Authored by kousikk on Oct 18 2019, 11:02 AM.



There's no behavior change - just moving DependencyScanningTool to its own file
since this tool can be reused across both clang-scan-deps binary and an interface
exposed as part of libClang APIs.

Diff Detail

Event Timeline

kousikk created this revision.Oct 18 2019, 11:02 AM
Herald added a project: Restricted Project. · View Herald TranscriptOct 18 2019, 11:02 AM
kousikk updated this revision to Diff 225667.Oct 18 2019, 11:05 AM

Lint fixes

Harbormaster completed remote builds in B39798: Diff 225667.
Bigcheese accepted this revision.Oct 21 2019, 5:30 PM

LGTM with style nit. I like that this decouples DependencyScanningTool from printing the results.


This looks like it needs to be clang-formatted.

This revision is now accepted and ready to land.Oct 21 2019, 5:30 PM
kousikk updated this revision to Diff 225994.Oct 21 2019, 9:53 PM

Run clang format

kousikk marked an inline comment as done.Oct 21 2019, 9:53 PM
This revision was automatically updated to reflect the committed changes.