Skip to content

Refactor: clean up naming to be consistent with snake_case#99

Open
schicho wants to merge 18 commits into
emukidid:masterfrom
schicho:naming-cleanup
Open

Refactor: clean up naming to be consistent with snake_case#99
schicho wants to merge 18 commits into
emukidid:masterfrom
schicho:naming-cleanup

Conversation

@schicho

@schicho schicho commented Apr 9, 2024

Copy link
Copy Markdown
Contributor

Cleaned up the code to be consistent with snake_case everywhere. Before there was a mixture of diverse naming styles.

Due to modifying almost every file, the line count has gotten quite large. I have split up the changes per .c and corresponding header file into individual commits, to maybe make it a bit more easy to view the changes. Every commit is buildable.

Changes:

  • switch to snake_case
  • rename functions and variables where sensible
    • rename gc_dvd to simply dvd
  • move md5 code into its own directory, just like sha1 and crc32 code
  • move some hardcoded values to enums or defines (shutdown or return to HBC, max DAT download size)
  • prefix functions in header files consistently

I have not changed the code in md5, sha1 and crc32 as these are libraries, which have been written like that by others years ago and is not core cleanrip code. Would you also handle it that way?

If there are any objections to these changes or need for discussion, let me know.

@emukidid

Copy link
Copy Markdown
Owner

Yeah, if you feel like doing this to the entire codebase I don't mind.

@schicho

schicho commented Apr 18, 2024

Copy link
Copy Markdown
Contributor Author

Yeah, if you feel like doing this to the entire codebase I don't mind.

That's what I did. You may take a look now. :)

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