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

Whole-tree reformat #3571

Merged
merged 1 commit into from
Apr 13, 2024
Merged

Whole-tree reformat #3571

merged 1 commit into from
Apr 13, 2024

Conversation

pmatos
Copy link
Collaborator

@pmatos pmatos commented Apr 12, 2024

This follows discussions from #3413.
Followup commits add clang-format file, script and blame ignore lists.

@pmatos
Copy link
Collaborator Author

pmatos commented Apr 12, 2024

How was this change generated?

Grab #3572 and the git repo at sha a9b7ad8.
Run at the root of the repo:

find . -iname '*.h' -o -iname '*.cpp' -exec python3 Scripts/clang-format.py -i \{\} \;

The resulting repo should match the changes in this commit.

@pmatos
Copy link
Collaborator Author

pmatos commented Apr 12, 2024

Pushed a new reformat. This time using reformatting using clang-format-16 through:

for file in `git ls-files '*.cpp' '*.h'`; do python3 Scripts/clang-format.py -i ${file}; done

The diff stat is: 447 files changed, 77599 insertions(+), 72080 deletions(-)

This follows discussions from FEX-Emu#3413.
Followup commits add clang-format file, script and blame ignore lists.
@pmatos pmatos force-pushed the ReformatWholesale branch from b6834ea to 2b4ec88 Compare April 12, 2024 14:26
@pmatos
Copy link
Collaborator Author

pmatos commented Apr 12, 2024

The stat looks correct. The different between the current stat and the one I mentioned earlier in #3571 (comment) is due to the PR #3574.

@Sonicadvance1 Sonicadvance1 merged commit 1d806ac into FEX-Emu:main Apr 13, 2024
10 checks passed
@pmatos pmatos deleted the ReformatWholesale branch April 16, 2024 12:36
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

Successfully merging this pull request may close these issues.

2 participants