Re: [PATCH 4/5] winegstreamer: Rename wg_parser_get_next_read_offset to wg_parser_wait_request.

[Zebediah Figura]

On 6/14/22 02:27, Rémi Bernon wrote:
> static void handle_read_request(struct media_source *source, QWORD file_size,
> -
>    void **buffer, size_t *buffer_size, uint64_t offset, uint32_t size)
> -
>    void **buffer, size_t *buffer_size, struct wg_request *request)

This can be constified, right?
> diff --git a/dlls/winegstreamer/unixlib.h b/dlls/winegstreamer/unixlib.h
> index 860a8ab2a52..f767daed513 100644
> --- a/dlls/winegstreamer/unixlib.h
> +++ b/dlls/winegstreamer/unixlib.h
> @@ -135,6 +135,24 @@ struct wg_sample
>       BYTE *data;
>   };
>
> +struct wg_request
> +{
> - enum wg_request_type
> - {
> -
>    WG_REQUEST_NONE = 0,
> -
>    WG_REQUEST_READ = 1,
> - } type;
> -
> - union
> - {
> -
>    struct
> -
>    {
> -
>        UINT32 size;
> -
>        UINT64 offset;
> -
>    } read;
> - } u;
>
> +};
> - struct wg_parser_buffer
> {
>     /* pts and duration are in 100-nanosecond units. */

What's the purpose of WG_REQUEST_NONE?