[FFmpeg-cvslog] avcodec/bitstream: try to workaround internal compiler bug in gcc 4.2

Reimar Döffinger Reimar.Doeffinger at gmx.de
Sat Jun 21 09:16:31 CEST 2014



On 20.06.2014, at 21:59, git at videolan.org (Michael Niedermayer) wrote:

> ffmpeg | branch: master | Michael Niedermayer <michaelni at gmx.at> | Fri Jun 20 21:54:23 2014 +0200| [329898aa45f5f8e8b89386ecd40b8db96746d53c] | committer: Michael Niedermayer
> 
> avcodec/bitstream: try to workaround internal compiler bug in gcc 4.2
> 
> gcc 4.2 seems not maintained anymore so theres no option besides
> just working around it.
> 
> Signed-off-by: Michael Niedermayer <michaelni at gmx.at>
> 
>> http://git.videolan.org/gitweb.cgi/ffmpeg.git/?a=commit;h=329898aa45f5f8e8b89386ecd40b8db96746d53c
> ---
> 
> libavcodec/bitstream.c |    2 +-
> 1 file changed, 1 insertion(+), 1 deletion(-)
> 
> diff --git a/libavcodec/bitstream.c b/libavcodec/bitstream.c
> index e3e870f..eb36c6f 100644
> --- a/libavcodec/bitstream.c
> +++ b/libavcodec/bitstream.c
> @@ -163,7 +163,7 @@ static int build_table(VLC *vlc, int table_nb_bits, int nb_codes,
>     int table_size, table_index, index, code_prefix, symbol, subtable_bits;
>     int i, j, k, n, nb, inc;
>     uint32_t code;
> -    volatile VLC_TYPE (*table)[2];
> +    volatile VLC_TYPE (* volatile table)[2];

Could you add a comment about it as well?
I think it would be better if it could be seen that this strange double volatile is on purpose without going through the history.


More information about the ffmpeg-cvslog mailing list