fix multi-game disc support for /app_home

[bibarub]

set_bdvd_as_app_home() was removed because it's redundant (check line 1748)

[aldostools]

I think line 929 should be changed back to static u8 gm = 01;

So PS3_GM01 will be always the first folder mounted and PS3_GAME will be the last.

Your current code will mount PS3_GAME only for the first game. After that all games will follow the behavior described above.

[aldostools]

Or maybe this code:

void map_app_home(const char *path)
{
	unmap_app_home();

	// remap app_home for multi-game discs
	char *mpath = (char *)malloc(strlen(path) + sizeof("/PS3_GAME") + 1);
	if(mpath)
	{
		sprintf(mpath, "%s/PS3_GM%02i", path, gm);
		if(not_exists(mpath))
		{
			sys_map_path("/app_home", path);

			sprintf(mpath, "%s/PS3_GM01", path);
			gm = isDir(mpath) ? 01 : 00; // reset gm to 00 if the game is not a multi-game disc
		}
		else
		{
			sys_map_path(APP_HOME_DIR, mpath); gm++;
		}

		free(mpath);
	}
}

[bibarub]

I think line 929 should be changed back to static u8 gm = 01;

So PS3_GM01 will be always the first folder mounted and PS3_GAME will be the last.

Your current code will mount PS3_GAME only for the first game. After that all games will follow the behavior described above.

PS3_GAME -> PS3_GM01 -> PS3_GM.. is the order the XMB uses for multi-game discs, so i don't think it should be changed
my code does exactly that - it mounts PS3_GAME first, then cycles through GM folders, until it runs out of them, and then mounts PS3_GAME again

(correct me if i'm wrong, i'm new to C)

[aldostools]

Yes your code does it right for the first game. After that the variable gm never return to 00.
So when a new game is mounted, the variable gm return to 01 (for non multi-game discs or mounts the next folder in the cycle if the new game is also a multi-game disc).

The code above fixes this behavior, except when the new game is also a multi-game disc.

[bibarub]

why should it return to 0? if GMXX doesn't exist, it will simply follow the "if(not_exists(mpath))" condition and mount /app_home to /dev_bdvd. adding another check just creates redundancy imo

[bibarub]

oh, i think i finally understand what you mean