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

aml: add codecname / video dimensions to processinfo #10405

Merged
merged 1 commit into from Sep 5, 2016

Conversation

@stefansaraev
Copy link
Contributor

commented Sep 5, 2016

as title says.

@@ -216,6 +216,10 @@ bool CDVDVideoCodecAmlogic::Open(CDVDStreamInfo &hints, CDVDCodecOptions &option
}
}

m_processInfo.SetVideoDecoderName(m_pFormatName, true);
m_processInfo.SetVideoDimensions(m_hints.width, m_hints.height);

This comment has been minimized.

Copy link
@FernetMenta

FernetMenta Sep 5, 2016

Member

the definitions of processInfo's video dimensions is the uncropped size, the cropped size is already exposed by stream info. not sure if AML exposes this info. let me know if you want to change this or merge as is.

This comment has been minimized.

Copy link
@stefansaraev

stefansaraev Sep 5, 2016

Author Contributor

sory, I dont follow

push it as is if you want, close if inappropriate

side note: stefansaraev@3c0cfa0 fixes reported resolution when sw decoding for me.

@FernetMenta FernetMenta merged commit 0038ad4 into xbmc:master Sep 5, 2016
2 checks passed
2 checks passed
continuous-integration/appveyor/pr AppVeyor build succeeded
Details
continuous-integration/travis-ci/pr The Travis CI build passed
Details
@stefansaraev stefansaraev deleted the stefansaraev:aml-processinfo branch Sep 5, 2016
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
3 participants
You can’t perform that action at this time.