-
Notifications
You must be signed in to change notification settings - Fork 233
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
Cleanup comments in save & load related files #352
Conversation
This definition was used to control the accessibility of some class members, changing protected qualifiers to public. This introduced unnecessary coupling between components and headers. All conditional access specifiers have been set to public, which should not be a problem given the low number of classes that actually used affected members. Another albeit more complex solution could have been to use friend classes.
D3 used to support Aureal 3-Dimensional sound cards, providing 3D spatial audio. The hardware and drivers is outdated, so all support in code has been removed, including UI mixer setting. Read more about A3D here: https://en.wikipedia.org/wiki/Aureal_Semiconductor#A3D
- Remove commented code - Move versioning comments to header - Add comments in gamesave.cpp for readability
Descent3/loadstate.cpp
Outdated
//@@ // movement info. | ||
//@@ switch (op->movement_type) | ||
//@@ { | ||
//@@ case MT_SHOCKWAVE: |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
do you have any idea what this whole code block could have been for?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
If you search @
you'll find a lot of files with commented blocks like this one. In this block, it looks like old loading code that was replaced.
Remove DDAccess.h header, and associated DD_ACCESS_RING definition
Remove Aureal support
Remove unused cf_ReadHogFileEntry() function.
All functions, that uses cf_ReadBytes(), checks return size, but don't handle potential exceptions. That leads to segfaults on reading of damaged files (i.e. broken savegames).
replace md5 with custom implementation, with tests
Converted line endings to Unix format to match .gitattributes file
Deleted left over VS project file.
…into gamesave-cleanup
Pull Request Type
Description