Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Provide .clang-format #14

Closed
oerdnj opened this issue Jul 22, 2022 · 1 comment
Closed

Provide .clang-format #14

oerdnj opened this issue Jul 22, 2022 · 1 comment

Comments

@oerdnj
Copy link
Contributor

oerdnj commented Jul 22, 2022

Hey,

before, I start doing this (it will require some tweaking), we had great success with adding and using .clang-format to reformat files automatically. I can try to model the .clang-format according to the current code (or just use linux format). But before I do that, is this something you would be willing to accept? There are numerous plugins to editors to format on save and it makes the life much easier in the end.

@simo5
Copy link
Member

simo5 commented Jul 22, 2022

Do you have an example?
I definitely do not like the Linux kernel style eats too much screen estate for no reason and they laso have other dubious practices.

simo5 added a commit to oerdnj/pkcs11-provider that referenced this issue Jul 23, 2022
Fixes latchset#14

Signed-off-by: Simo Sorce <simo@redhat.com>
simo5 added a commit to oerdnj/pkcs11-provider that referenced this issue Jul 23, 2022
Fixes latchset#14

Signed-off-by: Simo Sorce <simo@redhat.com>
simo5 added a commit to oerdnj/pkcs11-provider that referenced this issue Jul 23, 2022
Fixes latchset#14

Signed-off-by: Simo Sorce <simo@redhat.com>
simo5 added a commit to oerdnj/pkcs11-provider that referenced this issue Jul 23, 2022
Fixes latchset#14

Signed-off-by: Simo Sorce <simo@redhat.com>
simo5 added a commit to oerdnj/pkcs11-provider that referenced this issue Jul 24, 2022
Fixes latchset#14

Signed-off-by: Simo Sorce <simo@redhat.com>
@simo5 simo5 closed this as completed in e59cb3e Jul 24, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants