.clang-format: Update to match coding conventions

[tbunch1]

Prerequisite checklist

• Read the contribution guidelines regarding submitting new changes to the project;
• Tested your changes against relevant architectures and platforms;
• Ran the checkpatch.uk on your commit series before opening this PR;
• Updated relevant documentation.

Base target

• Architecture(s): [N/A]
• Platform(s): [N/A]
• Application(s): [N/A]

Additional configuration

N/A

Description of changes

Updates .clang-format to better reflect the Unikraft coding conventions.
Co-authored-by: Eddie Cazares ecazares15@utexas.edu
Co-authored-by: Lindsey Bowen lindseyb803@gmail.com

GitHub-Fixes: #975

[craciunoiuc]

I think you should also merge all commits into a single one 🙏

[tbunch1]

Apologies if this is a dumb question, but how should we do that? We're pretty new to contributing to open source and we can't figure out how to merge them together. I've been trying to rebase them but I'm having trouble because the changes are pushed. Again sorry if this is a simple question, but we're stuck

[craciunoiuc]

No worries,

on your local machine run these commands:

1. git rebase --interactive HEAD~3
2. in the prompt change the last two pick to fixup and the first pick to reword
3. change the first commit message to: .clang-format: Update to match coding conventions and continue
4. git push --force

I hope this works/helps 😅

[tbunch1]

Oh I see. I was worried about messing with the commit that merged the recent updates but I guess I didn't understand how it worked.

[StefanJum]

@tbunch1 please remove the changes to pagefault64.c, since they are not part of this pr

[razvand]

@tbunch1 please remove the changes to pagefault64.c, since they are not part of this pr

@tbunch1 , could you please see the comments from @StefanJum and update the PR accordingly?

See also the checkpatch checks.