NevolutionX
NevolutionX copied to clipboard
settingsMenu: Add system files dumper
https://github.com/dracc/NevolutionX/blob/1015b98ce3aa009f0b0a3862505ec2dc47b6373f/Sources/dumper.cpp#L22
I'd argue that this line is not necessary, as close()
calls flush()
anyway. Is it placed there just for clarity?
https://github.com/dracc/NevolutionX/blob/1015b98ce3aa009f0b0a3862505ec2dc47b6373f/Sources/dumper.cpp#L22
I'd argue that this line is not necessary, as
close()
callsflush()
anyway. Is it placed there just for clarity?
I'd argue .close()
is unnecessary too. I'll make sure to remove both. Thanks!
I'd be interested in a config sector, and eeprom dumper in the future perhaps you could abstract more?
I'd be interested in a config sector, and eeprom dumper in the future perhaps you could abstract more?
EEPROM dumper has been added. Dumping routines are small enough that I don't think abstraction is really necessary.