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
[VideoPlayer] FFmpegDemux: proper flush io context. #10722
Conversation
@@ -581,6 +581,9 @@ void CDVDDemuxFFmpeg::Reset() | |||
|
|||
void CDVDDemuxFFmpeg::Flush() | |||
{ | |||
if (m_ioContext) |
This comment was marked as spam.
This comment was marked as spam.
Sorry, something went wrong.
This comment was marked as spam.
This comment was marked as spam.
Sorry, something went wrong.
This comment was marked as spam.
This comment was marked as spam.
Sorry, something went wrong.
This comment was marked as spam.
This comment was marked as spam.
Sorry, something went wrong.
This comment was marked as spam.
This comment was marked as spam.
Sorry, something went wrong.
6596aa1
to
8f3df56
Compare
thanks, looks good now |
8f3df56
to
4d5c33a
Compare
now it better |
but the comment 2 lines above is outdated and wrong. those are no internal function anymore |
should we just drop the comment? |
yes please |
4d5c33a
to
3483b5c
Compare
jenkins build and merge |
see title.
@FernetMenta ping