[PATCH v3 1/1] winegstreamer: reintroduce flushing for wg_parser
zfigura at codeweavers.com
Tue Jun 7 16:56:13 CDT 2022
On 6/1/22 20:58, zlice wrote:
> From: zlice <zlice555 at gmail.com>
> Previous commit removed flushing but this causes issues in some games.
> - add flushing back
> - move variable creation after initial condition returns in wm_reader_get_stream_sample. This caused some slow down.
> - Remove EOS sets. Some are not what flushing did before. Had some seg faults on exiting Fallout 3 with them in.
As stated in my original reply to this patch :
I don't think "reintroduce flushing" is the right answer here, not
without understanding why it matters.
The point of 5144b2766 is that flushing should not make a difference. It
allows wg_parser_stream_get_event() to return more quickly, but that
same cost is added to the subsequent seek or stop request, so
GST_Seeking_SetPositions() or parser_cleanup_stream() will end up taking
just as long.
If flushing does make a difference, I think we need to understand why,
and quite likely solve this a different way.
More information about the wine-devel