Re: [FFmpeg-devel] [PATCH] avcodec/h264: Declare the local variable decode_chroma as const.

2017-07-20 Thread Michael Niedermayer
On Thu, Jul 20, 2017 at 10:31:19AM -0700, Wan-Teh Chang wrote: > ff_h264_decode_mb_cabac() and ff_h264_decode_mb_cavlc() are very long > functions. Declaring decode_chroma as const makes it clear the variable > doesn't change after initialization. > > Signed-off-by: Wan-Teh Chang

[FFmpeg-devel] [PATCH] avcodec/h264: Declare the local variable decode_chroma as const.

2017-07-20 Thread Wan-Teh Chang
ff_h264_decode_mb_cabac() and ff_h264_decode_mb_cavlc() are very long functions. Declaring decode_chroma as const makes it clear the variable doesn't change after initialization. Signed-off-by: Wan-Teh Chang --- libavcodec/h264_cabac.c | 4 ++-- libavcodec/h264_cavlc.c | 4 ++--