Re: [PATCH v5 0/2] MR7623: winegstreamer: Use provided PTS and duration in video_decoder.

[Brendan McGrath (＠redmcg)]

On Wed Apr 16 01:25:38 2025 +0000, Brendan McGrath wrote:
> Oops - sorry. I got the GUID mixed up with `timestamp/x-wg-transform`.
> The GUID might not be necessary actually. I might be able to use the
> `preserve_timestamps` boolean instead. I think the only difference would
> be that if GStreamer lost the `timestamp/x-wg-transform` state, we would
> fall back to using the gstreamer values. Which is possibly better.

Oops (again). I just tried using the `preserve_timestamps` boolean and it doesn't work. The `timestamp/x-wg-transform` is a flag that lets `video_decoder.c` know that an input timestamp has been provided (and that it should use it). It will otherwise fall back to the behavior defined by `provide_timestamps`.

But if we want to get rid of the need for the GUID, I guess we could move the existing logic in `video_decoder.c` (for generating our own timestamp) in to `wg_transform.c`.