On 01/09/2012 08:40 AM, Laurentiu Ion wrote:

> ---
>  libavcodec/kmvc.c |  122 
> ++++++++++++++++++++++++++---------------------------
>  1 files changed, 60 insertions(+), 62 deletions(-)


I have one general comment about the patch. There are still a lot of
explicit checks to see if there is at least one remaining byte. Couldn't
that be reduced with the use of the checked bytestream reader? At the
very least, some of those reads could be turned into unchecked reads
since they already have a check right above them...

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

Reply via email to