On 28/04/2025 10:39, Andreas Rheinhardt wrote:
> Patches attached.
> 
> - Andreas
> 

> [PATCH 1/5] avformat/apvdec: Use ffio_read_size()

LGTM.

> [PATCH 2/5] avformat/apvdec: Check before access

LGTM.

>     pkt->flags        = AV_PKT_FLAG_KEY;

Not sure where the funny alignment on this line came from (setting some other 
field there?) but suggest fixing it at the same time as this patch since you 
edit the previous line and it's not really worth its own patch.

> [PATCH 3/5] avformat/apvdec: Fix seeking

This makes more sense, thank you.  LGTM.

> [PATCH 4/5] avformat/apvdec: Remove inappropriate INIT_CLEANUP flag

Sure.

> [PATCH 5/5] avcodec/apv_entropy: Remove ff_apv_read_vlc()

See patch sent a few minutes ago - suggestions for nicer answer welcome.  
(Copy/paste is not a nicer answer.)

Thanks,

- Mark

_______________________________________________
ffmpeg-devel mailing list
ffmpeg-devel@ffmpeg.org
https://ffmpeg.org/mailman/listinfo/ffmpeg-devel

To unsubscribe, visit link above, or email
ffmpeg-devel-requ...@ffmpeg.org with subject "unsubscribe".

Reply via email to