summaryrefslogtreecommitdiff
path: root/Source/UnitTests/Common/SettingsHandlerTest.cpp
AgeCommit message (Collapse)Author
2024-10-27Merge pull request #12744 from nlebeck/settingshandler-split-2JMC47
Split `SettingsHandler` into separate reader and writer classes
2024-10-22Add missing algorithm header in SettingsHandlerTest.cppAdmiral H. Curtiss
2024-10-21Split `SettingsHandler` into separate reader and writer classesNiel Lebeck
2024-10-10Modernize `std::equal` with rangesmitaclaw
2024-04-23Eliminate SettingsHandler's `SetBytes` and `Reset` methodsNiel Lebeck
Also make the `Decrypt` method private. As far as I can tell, the only motivation for exposing the `SetBytes` and `Reset` methods is to allow `CBoot::SetupWiiMemory` to use the same `SettingsHandler` instance to read settings data and then write it back. It seems cleaner to just use two separate instances, and require a given `SettingsHandler` instance to be used for either writing data to a buffer or reading data from a buffer, but not both. A natural next step is to split the `SettingsHandler` class into two classes, one for writing data and one for reading data. I've deferred that change for a future PR.
2024-04-12Pass SettingsHandler buffers by const ref instead of rvalue refNiel Lebeck
2024-03-06Add SettingsHandler unit tests exercising the edge case in PR #8704Niel Lebeck
2024-03-01Add simple unit tests for SettingsHandlerNiel Lebeck