Quoting Diego Biurrun (2015-04-07 14:32:17)
> On Mon, Apr 06, 2015 at 09:04:14PM +0200, Anton Khirnov wrote:
> > --- a/libavcodec/h264.c
> > +++ b/libavcodec/h264.c
> > @@ -1698,7 +1698,7 @@ av_cold void ff_h264_free_context(H264Context *h)
> > {
> > int i;
> >
> > - ff_h264_free_tables(h); // FIXME cleanup init stuff perhaps
> > + ff_h264_free_tables(h);
>
> LGTM, but could be squashed into 6/7 perhaps, I already noticed the
> comment in that patch.
Once upon a time we had a rule about no mixing of unrelated changes.
--
Anton Khirnov
_______________________________________________
libav-devel mailing list
[email protected]
https://lists.libav.org/mailman/listinfo/libav-devel