On 2015-04-06 21:04:10 +0200, Anton Khirnov wrote:
> Out of the three places where this function is called with free_rbsp=1,
> only during the call from free_context() it actually does something,
> it is a no-op in the other two cases.
> 
> Managing the DPB also does not logically belong in free_tables().
> Therefore, clear it in free_context() itself.

could please move DPB allocation/initialization to h264_init_context 
too?

Patch otherwise ok as are the ones I didn't reply explicitly.

Janne

_______________________________________________
libav-devel mailing list
[email protected]
https://lists.libav.org/mailman/listinfo/libav-devel

Reply via email to