Skip to content
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

improve cellHddGameCheck #8893

Merged
merged 3 commits into from Oct 23, 2020
Merged

Conversation

Megamouse
Copy link
Contributor

No description provided.

@elad335
Copy link
Contributor

elad335 commented Sep 11, 2020

Thats a huge copy paste of cellGameDataCheckCreate2, just use a shared function.

@Megamouse
Copy link
Contributor Author

maybe seems like it but not really. it has both different enum values/log messages as well as different datatypes / HG check / sound format and resolution

@elad335
Copy link
Contributor

elad335 commented Sep 11, 2020

image
image

@elad335
Copy link
Contributor

elad335 commented Sep 11, 2020

logging - use ppu.current_function for logging then and not the original name of the function.

@Megamouse
Copy link
Contributor Author

the strings will vary either way.
Idk what it looks like on real PS3, But one message would be:
The game data was not found!
while the other would be:
The hdd boot game was not found!

@Megamouse
Copy link
Contributor Author

and since i have another PR open that adds translations we'll have to use different enums values
.

@jy95
Copy link
Contributor

jy95 commented Sep 11, 2020

@Megamouse little question : does your PR fixes this strange bug on POP 2008 ? https://forums.rpcs3.net/thread-196966.html

@Megamouse
Copy link
Contributor Author

Try installing update 01.01 for that game. It will create the required folder. It shouldn't be related to this PR I think

@Megamouse Megamouse force-pushed the cellHddGameCheck branch 2 times, most recently from 5be998c to 525e7d1 Compare September 17, 2020 11:32
@Megamouse Megamouse changed the title [WIP] improve cellHddGameCheck improve cellHddGameCheck Sep 17, 2020
@Megamouse Megamouse marked this pull request as ready for review September 17, 2020 11:41
return CELL_GAMEDATA_ERROR_PARAM;
}

if (!fs::create_path(vfs::get(usrdir)))
Copy link
Contributor

@elad335 elad335 Sep 23, 2020

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This function is called cellHddGameCheck, not cellHddGameCheckCreate. At best it only updates PARAM.SFO parameters with strict error checking for it.

Copy link
Contributor

@elad335 elad335 Sep 23, 2020

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I do not think games are allowed to create HG game data, only GD.


// TODO: Find error code
verify(HERE), dir.size() == 9;
verify(HERE), game_dir.size() == 9;
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Maybe remove this assert, if non HG content is accessed ERROR_BROKEN should be returned.

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This function can also create new directories, so I don't see the relevance of HG content in this topic.

@Megamouse Megamouse force-pushed the cellHddGameCheck branch 2 times, most recently from 38efe43 to 943c930 Compare September 28, 2020 20:54
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

4 participants