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

Stream at 1920x1080 #221

Open
hugomc92 opened this issue Oct 11, 2022 · 3 comments
Open

Stream at 1920x1080 #221

hugomc92 opened this issue Oct 11, 2022 · 3 comments

Comments

@hugomc92
Copy link

Could be possible to add support for 1080p streaming?

There is currently a fork that enable it (this repo) but it's outdated and can't stablish connection (RTSP handshake error) which is fixed in this repo.

Thanks in advance

@d3m3vilurr
Copy link
Collaborator

hello. 1080p & RTSP handshake problem are different parts.

first, current codeset detect correct resolutions of the vita
https://github.com/xyzz/vita-moonlight/blob/vita/src/gui/ui_settings.c#L486-L490
this codeset already contains 1920x1080.

second, afaik RTSP handshake problem is related sscanf of the newlib
https://github.com/xyzz/vita-moonlight/blob/vita/libgamestream/client.c#L563

current vita newlib doesn't support %2hhx that this is not the part of the posix standard.
so i'm considering to add the patch into vita newlib or replace sscanf codeset to another way.

in this reason, I wouldn't merge pr #220 ...

@hugomc92
Copy link
Author

Hello, I know 1080p and RSTP handshake are diferent things.

I wanted to say that in that repo vpk there is selectable option for 1920x1080 in resolution but when it tries to connect it gives RSTP handshake error and in your vpk there isn't selectable option for 1080 (at least for my PS Vita Slim, maybe I am missing something) and it already connect to PC without any problems.

Maybe the thing is not to merge the pull request, maybe there is another thing could be done to get 1920x1080 selectable resolution if you can or maybe just tell me if there's something I can do to get that resolution as an option.

Thank you very much in advance!

@Fewnity
Copy link

Fewnity commented Mar 5, 2023

second, afaik RTSP handshake problem is related sscanf of the newlib https://github.com/xyzz/vita-moonlight/blob/vita/libgamestream/client.c#L563

current vita newlib doesn't support %2hhx that this is not the part of the posix standard. so i'm considering to add the patch into vita newlib or replace sscanf codeset to another way.

Hello! Do you think you can fix this? Is it an easy patch to do?

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

No branches or pull requests

3 participants