Re: [libav-devel] [PATCH v2] lavc, lavu: move frame cropping to a convenience function

2017-07-13 Thread Luca Barbato
On 7/12/17 5:02 PM, wm4 wrote: > TODO: APIchanges entry, version bump. > --- > Now with flags instead of a bool parameter. > --- > libavcodec/decode.c | 89 ++--- > libavutil/frame.c | 101 > >

Re: [libav-devel] [PATCH] rmdec: don't ignore the return value of av_get_packet()

2017-07-13 Thread Luca Barbato
On 7/13/17 2:30 AM, Sean McGovern wrote: > +av_packet_unref(pkt); This one is a no-op if I recall correctly. lu ___ libav-devel mailing list libav-devel@libav.org https://lists.libav.org/mailman/listinfo/libav-devel

[libav-devel] [PATCH v2] lavc, lavu: move frame cropping to a convenience function

2017-07-13 Thread wm4
TODO: APIchanges entry, version bump. --- Now with flags instead of a bool parameter. --- libavcodec/decode.c | 89 ++--- libavutil/frame.c | 101 libavutil/frame.h | 32 + 3 files

[libav-devel] [PATCH] rmdec: don't ignore the return value of av_get_packet()

2017-07-13 Thread Sean McGovern
--- As noted by Diego when I fixed NSV earlier. Sorry for dragging my feet on this one. Question: do I actually need to unref the packet here? It seems like we are really inconsistent on how to handle errors from av_get_packet(). Sometimes we just return the result of the call, other times we