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

Make psvDebugScreen usable from prx #44

Open
wants to merge 1 commit into
base: master
Choose a base branch
from
Open

Conversation

yne
Copy link
Contributor

@yne yne commented Jan 20, 2018

Using psvDebugScreen from a prx lead to the following issues:

  • vsnprintf call crash the app and so, shall be use to the sceClib version instead
  • possible multiple call to psvDebugScreenInit (from eboot and prx) that could lead to a FrameBuffer stealing
  • modulo operation are not available when prx compiled with -nostdlib
    This commit aim to resolve those issues, but also give an explanation in comment.

@yne yne force-pushed the patch-1 branch 2 times, most recently from 9aac54d to 60dbada Compare January 20, 2018 13:53
@frangarcj
Copy link
Contributor

Maybe put prx version under a ifdef?

@yne
Copy link
Contributor Author

yne commented Jan 21, 2018

@frangarcj Yeah, the first version used #ifdef but then a realized that this version also solve the "EBOOT loading EBOOT" issue. So I removed the #define because it will simply work for any cases.

@yne yne force-pushed the patch-1 branch 2 times, most recently from bf1d4bf to 90b0a3b Compare January 24, 2018 22:36
Using psvDebugScreen from a prx lead to the following issues:
- vsnprintf call crash the app and so, shall be directed to the sceClib version
- possible multiple call to psvDebugScreenInit (from eboot and prx) that could lead to a FrameBuffer stealing
- modulo operation are not available when prx compiled with -nostdlib
This commit aim to resolve those issues, but also give an explanation using comment.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants