notaz / pcsx_rearmed

ARM optimized PCSX fork
GNU General Public License v2.0
384 stars 207 forks source link

Numerical values in config files are hexadecimal and without prefix #285

Closed porg closed 7 months ago

porg commented 1 year ago

Problem

Considerations

Path forward (if you are willing to change at all)

Approach: Update dialog each user needs to answer once

Opt-in: Users who want decimal notation run a one-time conversion

$ pcsx --update-script hex2dec

porg commented 1 year ago

This has a low priority. But I wanted to note it down at least. And cross-reference when/where-ever this is (a bit of) an issue.

notaz commented 7 months ago

The config files were not designed with user modification in mind. And like you said very few ever need it (out of very few that are still using pandora). Your ideas might be good but in the current situation it's not worth spending time on IMO.

Also the .pnd is arranged in such a way that you can have multiple versions of the emulator at once so that you can run an older version in case of regressions. Your one-time conversion would cause problems with that.

In the end the only change I've done is to add the 0x prefix which the old versions can also understand.

porg commented 7 months ago

Thanks that as a minimal improvement the 0x prefix is added and reverse compatible. That way those who tinker with it at least know immediately that this is a hexadecimal value.