Various GameINI updates

[Techjar]

I've itemized each change (or set of changes) into separate commits, making it easier to see what all the specific changes are.

Here's the list of issue reports this addresses:
https://bugs.dolphin-emu.org/issues/7909
https://bugs.dolphin-emu.org/issues/11198
https://bugs.dolphin-emu.org/issues/11389
https://bugs.dolphin-emu.org/issues/10556
https://bugs.dolphin-emu.org/issues/11322
https://bugs.dolphin-emu.org/issues/10703
https://bugs.dolphin-emu.org/issues/11918
https://bugs.dolphin-emu.org/issues/11477
https://bugs.dolphin-emu.org/issues/11732
https://bugs.dolphin-emu.org/issues/12155
https://bugs.dolphin-emu.org/issues/6752
https://bugs.dolphin-emu.org/issues/12189
https://bugs.dolphin-emu.org/issues/12251
https://bugs.dolphin-emu.org/issues/12225
https://bugs.dolphin-emu.org/issues/12023

[JMC47]

I confirmed a lot of these earlier this morning for Techjar. As long as the INIs are done right, lgtm.

[JosJuice]

LGTM once those two things are addressed. There are still some unfixed game INI issues on the issue tracker, but I see no reason to block this PR because of that.

[Rumi-Larry]

This changes can be addressed in a future pr, but I'm frankly put off by Github to do it myself. Please feel free to ignore, since the titles are clear enough as they are. The only one that really need to change, is the Pikmin one.

• # HA9xxx - Super Mario Bros. (Brawl VC) should read # HA9xxx - Super Mario Bros. (Brawl Masterpiece) to use the official terminology
• # NAKE01, NAKP01 - Pokemon Snap should have the "(VC)" suffix, since a new game was announced and it doesn't hurt to be explicit.
• # NALE01, NALJ01, NALP01 - Super Smash Bros. (Virtual Console) # NATJ01, NATP01, NATE01 - Mario Tennis (Virtual Console) this is merely for consistency, but one could shorten the instances of "(Virtual Console)" to "(VC)"
• # R9IE01 - PIKMIN1 for Wii should say # R9IE01 - PIKMIN (Wii)
• # XAFJJ8 - Metal Gear (MSX) the title is unclear # XAFJJ8 - Metal Gear (MSX version) (VC) would be clearer as well as # XANJJ8 - Gradius 2 (MSX version) (VC) and
• # XAQJJ8 - Salamander (MSX version) (VC)
• # XAIJJ8 - Quarth does not specify the version, even though there are several eligible ports. # XAIJJ8 - Quarth (MSX version) (VC) is better

I would love to go through each ini file if necessary. I have this weird thing were I find such activities greatly satisfying 😃

[JosJuice]

The titles listed in game INIs are pretty inconsistent in general (which doesn't actually lead to a lot of problems in practice since the title isn't actually used for anything). If we want to start caring about having correct titles in game INIs, we should probably start by doing a mass replace with titles from GameTDB.com or something similar.

[Rumi-Larry]

@JosJuice I agree, I also can look at every natural language sentence in the code (comments and user facing strings) to look for typos and punctuation errors. I'm not kidding when I say that I like doing it, it's just hard to find a place where they would even want me to.