On 17 October 2013 19:41, David Kment <[email protected]> wrote:
>
> Here is the updated version.
> I changed the code according to the things mentioned.


Indeed it looks almost perfect.

The only thing beside stylistic I could spot is

if(hnm->buffer1) {av_freep(&hnm->buffer1);}

Don't be afraid to always call av_freep() - it checks for NULL pointers itself.
_______________________________________________
libav-devel mailing list
[email protected]
https://lists.libav.org/mailman/listinfo/libav-devel

Reply via email to