Re: [PATCH v2 0/7] MR6517: mfsrcsnk: Read samples from the media source demuxer.

[Emil Velikov (＠xexaxo)]

Emil Velikov (@xexaxo) commented about dlls/winedmo/unix_demuxer.c:
> -
>    {
> -
>        TRACE( "Failed to read context %p, error %s.\n", ctx, debugstr_averr( ret ) );
> -
>        av_packet_free( &packet );
> -
>        if (ret == AVERROR_EOF) return STATUS_END_OF_FILE;
> -
>        return STATUS_UNSUCCESSFUL;
> -
>    }
> - }
> -
> - params->sample.size = packet->size;
> - if ((capacity < packet->size))
> - {
> -
>    ctx->opaque = packet;
> -
>    return STATUS_BUFFER_TOO_SMALL;
> - }
> -
> - memcpy( (void *)(UINT_PTR)sample->data, packet->data, packet->size );

Assuming this is going to be a hot path, would it make sense to wrap sample->data instead of copying? Not sure if libav has API (flexible) for this though.