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
Additional Country Filters #1456
Conversation
Typo fliters -> filters in title. |
b12c9bd
to
5b58842
Compare
@dolphin-emu-bot rebuild |
1 similar comment
@dolphin-emu-bot rebuild |
//case DiskIO::IVolume::COUNTRY_INTERNATIONAL: | ||
//case DiscIO::IVolume::COUNTRY_SDK: | ||
//case DiscIO::IVolume::COUNTRY_UNKNOWN: | ||
// TODO: Separate Region Dir for regionless games? |
This comment was marked as off-topic.
This comment was marked as off-topic.
Sorry, something went wrong.
This comment was marked as off-topic.
This comment was marked as off-topic.
Sorry, something went wrong.
This comment was marked as off-topic.
This comment was marked as off-topic.
Sorry, something went wrong.
ba1298c
to
121abca
Compare
Tried to squash everything together in a way that made sense. Let me know if I need to squash anything else together for a merge. |
Any reason this can't be merged? @lioncash |
Hold off on merging. I'm currently working on adding one more country flag, as well as updating a few Qt references. |
@@ -17,22 +17,20 @@ IVolume::ECountry CountrySwitch(u8 CountryCode) | |||
{ | |||
// Region free - fall through to European defaults for now |
This comment was marked as off-topic.
This comment was marked as off-topic.
Sorry, something went wrong.
This comment was marked as off-topic.
This comment was marked as off-topic.
Sorry, something went wrong.
LGTM. @lioncash probably should go over one last time before merging. |
@waddlesplash I don't want to merge this yet. Hence the WIP in the title. |
@Stevoisiak what's left to do? |
@waddlesplash The guy who runs GameTDB told me that the H country code was used by some games from the Netherlands, so I still need to add a flag for that. In addition, I want to do a bit of research on the X, Y, and Z country codes. |
0084d23
to
ff8b3c7
Compare
@Stevoisiak this is done now, right? |
@waddlesplash Unless someone has a flag I can use for region-free/International titles, then yes, I am done. |
That can probably be saved for another PR I guess. LGTM, @lioncash can you merge? |
Oh wait, don't merge this -- @Stevoisiak since you changed the country filters you need to bump the cache revision (in ISOFile.cpp) [also might wanna put a comment on the ECountry definition that reminds people they must bump the cache revision if they change it]. |
ff8b3c7
to
98dd668
Compare
* Added country flags for games from Netherlands and Spain * Added separate category for Region Free games (Uses European flag as placeholder) * Added missing country filter options in "show regions" menu * Rearranged country filters for readability * Incremented CACHE_REVISION Also fixed various country filters not showing up as options in the "Show regions" menu.
SDK is not an official Nintendo country code, and is not actually used by any official software.
98dd668
to
64f6d91
Compare
@@ -37,7 +37,7 @@ | |||
#include "DolphinWX/ISOFile.h" | |||
#include "DolphinWX/WxUtils.h" | |||
|
|||
static const u32 CACHE_REVISION = 0x115; | |||
static const u32 CACHE_REVISION = 0x117; |
This comment was marked as off-topic.
This comment was marked as off-topic.
Sorry, something went wrong.
@Stevoisiak says this is ready to go. @lioncash, will you do the honors? |
Hey @Stevoisiak How would I make or get .xpm flags. I'm going to make a commit of a missing country... |
@ZephyrSurfer What country is missing? |
@Stevoisiak that would be sweden. |
@ZephyrSurfer I'm pretty sure there isn't a region code for games from Sweden, but I could easily be wrong. We identify a game's country with the region code, which is the 4th digit of the GameID. An incomplete list of region codes can be found on WiiBrew.org |
@Stevoisiak Yeah I know. There is some for example. one sec Edit: Here, here: http://www.gametdb.com/Wii/GAZM69 Edit 2: and that list says the X region is wiiware/ virtual console. However there are retail gamecube games with it. So not only incomplete it is also inaccurate. |
Bump Edit: how do import games even have different GameIDs? |
@ZephyrSurfer |
@Stevoisiak Yes, that's what I'm saying. |
@ZephyrSurfer I just looked through GameTDB. Identifying the 'M' country code as Swedish games doesn't work. While the two Swedish Harry Potter games are the only GameCube games using the 'M' country code, unfortunately there are a few PAL virtual console games that use the M country code. |
@Stevoisiak Oh well, I just checked it against the 6 letter GameID. |
No description provided.