[PATCH 6/6] winegstreamer: Add timestamp and duration to struct wg_sample.

Zebediah Figura zfigura at codeweavers.com
Mon Apr 4 18:07:39 CDT 2022


On 4/4/22 16:48, Rémi Bernon wrote:
> diff --git a/dlls/winegstreamer/unixlib.h b/dlls/winegstreamer/unixlib.h
> index f4e2ea4966b..ca361fe54bb 100644
> --- a/dlls/winegstreamer/unixlib.h
> +++ b/dlls/winegstreamer/unixlib.h
> @@ -114,10 +114,14 @@ struct wg_format
>   enum wg_sample_flag
>   {
>       WG_SAMPLE_FLAG_INCOMPLETE = 1,
> +    WG_SAMPLE_FLAG_HAS_TIMESTAMP = 2,
> +    WG_SAMPLE_FLAG_HAS_DURATION = 4,
>   };
>   
>   struct wg_sample
>   {
> +    /* timestamp and duration are in 100-nanosecond units. */
> +    LONGLONG timestamp, duration;
>       UINT32 flags;
>       UINT32 max_size;
>       UINT32 size;

I don't want to bikeshed the naming and types, and I'm willing to sign 
off on the patch regardless, but is there a reason to diverge from 
wg_parser_buffer here?

I do think the PTS and duration should be unsigned, at least.



More information about the wine-devel mailing list