On Wed, Feb 1, 2017 at 11:34 AM, Anton Khirnov <an...@khirnov.net> wrote:
> Quoting Vittorio Giovara (2017-02-01 10:44:11)
>> On Wed, Feb 1, 2017 at 10:25 AM, Anton Khirnov <an...@khirnov.net> wrote:
>> > Video decoders always consume full packets.
>> > ---
>> >  libavcodec/h264dec.c | 20 +++-----------------
>> >  1 file changed, 3 insertions(+), 17 deletions(-)
>>
>> Do you mean that get_consumed_bytes() was always equal to buf_size?
>> Also there are several decoders that don't completely consume packets,
>> so the commit line could be changed to h264 only probably.
>
> No, I mean that the return value is always ignored by the generic code
> for video decoders. See
> https://git.libav.org/?p=libav.git;a=blob;f=libavcodec/decode.c;h=f4088cdae8d6a364f95ee3e04a601157fcfe16aa;hb=HEAD#l355
> So it doesn't matter what value is returned here.

ok, but IMO it's hard to get that from the current commit description.
Could you amend it in a more direct way?
-- 
Vittorio
_______________________________________________
libav-devel mailing list
libav-devel@libav.org
https://lists.libav.org/mailman/listinfo/libav-devel

Reply via email to