On Thu, Jul 30, 2015 at 4:06 PM, Ganesh Ajjanagadde
<gajjanaga...@gmail.com> wrote:
> Signed-off-by: Ganesh Ajjanagadde <gajjanaga...@gmail.com>
> ---
> libavcodec/aacdec_template.c | 5 -----
> 1 file changed, 5 deletions(-)
>
> diff --git a/libavcodec/aacdec_template.c b/libavcodec/aacdec_template.c
> index 2f270bc..d7849da 100644
> --- a/libavcodec/aacdec_template.c
> +++ b/libavcodec/aacdec_template.c
> @@ -1907,11 +1907,6 @@ static int decode_ics(AACContext *ac,
> SingleChannelElement *sce,
> ac->oc[1].m4ac.object_type == AOT_ER_AAC_LD ||
> ac->oc[1].m4ac.object_type == AOT_ER_AAC_ELD;
>
> - /* This assignment is to silence a GCC warning about the variable being
> used
> - * uninitialized when in fact it always is.
> - */
> - pulse.num_pulse = 0;
> -
> global_gain = get_bits(gb, 8);
>
> if (!common_window && !scale_flag) {
> --
> 2.5.0
>
I guess I should have clarified the intent of this patch:
This particular code only existed (as shown in the comment) to work
around a compiler bug regarding diagnostics.
Said compiler bug does not affect GCC of any recent vintage (I run the
latest 5.2, but tested 4.8.1),
and I consider the code distracting.
When such things get fixed by the compiler,
hacks that we place to suppress warnings should also be removed in my opinion.
I am interested to know what others think about this.
_______________________________________________
ffmpeg-devel mailing list
ffmpeg-devel@ffmpeg.org
http://ffmpeg.org/mailman/listinfo/ffmpeg-devel
