[FFmpeg-devel] [PATCH 1/2] Fix DPX decoder Rewrite the DPX decoder to work with provided sample DPXs

mashiat.sarker at gmail.com mashiat.sarker at gmail.com
Thu Oct 4 23:58:57 CEST 2012


On 10/04/2012 01:48 PM, Georg Lippitsch wrote:
> @@ -180,68 +194,65 @@ static int decode_frame(AVCodecContext *avctx,
>   
>       for (i=0; i<AV_NUM_DATA_POINTERS; i++)
>           ptr[i] = p->data[i];
> -    stride = p->linesize[0];
>   
> -    if (source_packet_size*avctx->width*avctx->height > buf_end - buf) {
> +    if (elements*avctx->width*avctx->height > buf_end - buf) {

This line could use some spaces around the operators. A few similar 
instances can be found later too. I am not feeling like doing a cosmetic 
review. This is more of a nit.


More information about the ffmpeg-devel mailing list