arrow icon indicating copy to clipboard operation
arrow copied to clipboard

[C++, lint] Use clang-format more efficiently in `check-format` target

Open asfimport opened this issue 6 years ago • 1 comments

clang-format supports command line option -output-replacements-xml which (in the case of no required changes) outputs:

<?xml version='1.0'?>
<replacements xml:space='preserve' incomplete_format='false'>
</replacements>

Using this option during check-format instead of using python to compute a diff between formatted and on-disk should speed up that target significantly

Reporter: Ben Kietzman / @bkietz

Note: This issue was originally created as ARROW-4515. Please see the migration documentation for further details.

asfimport avatar Feb 08 '19 16:02 asfimport

This issue has been marked as stale because it has had no activity in the past 365 days. Please remove the stale label or comment below, or this issue will be closed in 14 days. If this improvement is still desired but has no current owner, please add the 'Status: needs champion' label.

github-actions[bot] avatar Dec 13 '25 11:12 github-actions[bot]