This is an archive of the discontinued Mercurial Phabricator instance.

makefile: add target to apply clang-format in-place
ClosedPublic

Authored by durin42 on Oct 17 2017, 5:44 PM.

Details

Summary

This makes it easy to reformat files after you finish editing them.

Diff Detail

Repository
rHG Mercurial
Lint
Automatic diff as part of commit; lint not applicable.
Unit
Automatic diff as part of commit; unit tests not applicable.

Event Timeline

durin42 created this revision.Oct 17 2017, 5:44 PM
indygreg accepted this revision.Oct 18 2017, 11:45 AM
This revision is now accepted and ready to land.Oct 18 2017, 11:45 AM
This revision was automatically updated to reflect the committed changes.
av6 added a subscriber: av6.Oct 25 2017, 11:42 PM

I'm a bit late to the party, but wanted to remind people that targets that are not actual files (in this case, a file named "format-c") should also go into .PHONY list. Nothing bad will happen right now (unless such file appears at some point), but adding it to "phony" targets helps keeping makefile in proper shape. (Looking at the already unwieldy .PHONY list I wish Make had another way to mark targets "phony").

d'oh - could you send a patch to fix that?

av6 added a comment.Oct 26 2017, 12:01 AM

Yep, mailed.