eden-emu/eden
24
3
Fork 16
Code Issues 139 Pull requests 62 Projects 1 Releases Wiki Activity Actions

[meta] clang-format the entire codebase #2895

Open
crueter wants to merge 2 commits from clang-format into master
pull from: clang-format
merge into: eden-emu:master
Conversation 3 Commits 2 Files changed 821 +19461 -15019
crueter commented 2025-10-30 01:07:04 +01:00
Owner
Copy link

nuclear bomb

Signed-off-by: crueter crueter@eden-emu.dev

nuclear bomb Signed-off-by: crueter <crueter@eden-emu.dev>
[meta] clang-format the entire codebase
Some checks reported errors
eden-license / license-header (pull_request) Failing after 36s
Details
GitHub Actions [CI] Build cancelled
Details
37d3ac92e3
 
nuclear bomb

Signed-off-by: crueter <crueter@eden-emu.dev>
headers
All checks were successful
eden-license / license-header (pull_request) Successful in 34s
Details
GitHub Actions [CI] Build succeeded
Details
GitHub Releases [CD] Build succeeded – Release published
Details
9ad2bd5dce
 
nuclear bomb part 2: nagasaki edition

Signed-off-by: crueter <crueter@eden-emu.dev>
Lizzie commented 2025-10-30 02:06:46 +01:00
Member
Copy link

NO

NO
DraVee commented 2025-10-30 16:52:06 +01:00
Member
Copy link

What about making clang-format mandatory only on files changed (like license-header.sh)?

What about making clang-format mandatory only on files changed (like license-header.sh)?
Lizzie commented 2025-10-30 21:13:03 +01:00
Member
Copy link

@DraVee wrote in #2895 (comment):

What about making clang-format mandatory only on files changed (like license-header.sh)?

N O

@DraVee wrote in https://git.eden-emu.dev/eden-emu/eden/pulls/2895#issuecomment-9235: > What about making clang-format mandatory only on files changed (like license-header.sh)? N O
All checks were successful
eden-license / license-header (pull_request) Successful in 34s
Required
Details
GitHub Actions [CI] Build succeeded
Details
GitHub Releases [CD] Build succeeded – Release published
Details
This pull request has changes conflicting with the target branch.
  • src/core/hle/service/acc/profile_manager.cpp
  • src/dynarmic/src/dynarmic/backend/arm64/a32_interface.cpp
  • src/dynarmic/src/dynarmic/backend/riscv64/a32_interface.cpp
  • src/dynarmic/src/dynarmic/backend/x64/a32_interface.cpp
  • src/dynarmic/src/dynarmic/backend/x64/a64_interface.cpp
  • src/dynarmic/src/dynarmic/common/llvm_disassemble.cpp
  • src/dynarmic/src/dynarmic/common/x64_disassemble.cpp
  • src/dynarmic/src/dynarmic/common/x64_disassemble.h
  • src/dynarmic/tests/A64/a64.cpp
View command line instructions

Checkout

From your project repository, check out a new branch and test the changes.
git fetch -u origin clang-format:clang-format
git switch clang-format
Sign in to join this conversation.
Reviewers
No reviewers
Lizzie
CamilleLaVey
Maufeat
MrPurple666
MaranBr
Labels
Clear labels   
android

Specific to android platform

  
audio

   
bug

Something is not working

  
ci

For changes to the ci/building

  
cmake

   
confirmed

Used when testers confirm a bug is reported and it is reproducable.

  
core

For core changes to the emulator

  
desktop

Specific to desktop PCs, handhelds, etc.

  
duplicate

This issue or pull request already exists

  
enhancement

New feature

  
frontend

Frontend changes

  
fw

Changes related to firmware

  
git

For enhancements or changes regarding git.

  
graphics

   
loader

For changes to any of the loader implementations

  
nv gpu

For use with changes that involve the emulator GPU and any services.

  
ui

   
vulkan

Used on anything Vulkan-related

No labels
android
audio
bug
ci
cmake
confirmed
core
desktop
duplicate
enhancement
frontend
fw
git
graphics
loader
nv gpu
ui
vulkan
Milestone
Clear milestone
No items
No milestone
Projects
Clear projects
No items
No project
Assignees
Clear assignees
No assignees
3 participants
Notifications
Due date
The due date is invalid or out of range. Please use the format "yyyy-mm-dd".

No due date set.

Dependencies

No dependencies set.

Reference
eden-emu/eden!2895
No description provided.