libusb icon indicating copy to clipboard operation
libusb copied to clipboard

WIP: add a .clang-format config file

Open seanm opened this issue 1 year ago • 3 comments

seanm avatar Jan 23 '24 03:01 seanm

Can you make another draft/PR with formatting already applied? (and include .git-blame-ignore-revs, if feasible)

Youw avatar Jan 23 '24 15:01 Youw

Yes, I can do that... but I'll wait for some more feedback from others about what options the project wants.

In my local copy, I do diffs using Araxis Merge which has some nice options to ignore only leading whitespace, trailing whitespace, inner whitespace, etc. With my current .clang-format file, if I ignore leading and trailing whitespace, there aren't very many changes in fact.

seanm avatar Jan 23 '24 16:01 seanm

more feedback from others about what options the project wants

Because most of us are not clang-format options experts, it is hard to tell w/o applying and checking what's changed. That's exactly why I'm suggesting a PR to look at.

options to ignore only leading whitespace, trailing whitespace, inner whitespace

Github also has an option to "Hide whitespace" changes when viewing a PR. That should help.

Youw avatar Jan 23 '24 17:01 Youw