Re: gst-plugins-bad: mpegvideoparse: Fix set-but-unused warnings

[Edward Hervey]

On Tue, 2011-07-12 at 15:23 -0700, Olivier Crête wrote:

> Module: gst-plugins-bad
> Branch: master
> Commit: 35f85f978e17cf4711c257735b7440e144a8f2ee
> URL:    http://cgit.freedesktop.org/gstreamer/gst-plugins-bad/commit/?id=35f85f978e17cf4711c257735b7440e144a8f2ee
>
> Author: Olivier Crête <[hidden email]>
> Date:   Tue Jul 12 17:56:25 2011 -0400
>
> mpegvideoparse: Fix set-but-unused warnings
>
> ---

  Comment out instead of removing. Might be needed in the future and
it's painful to guess again what they would be.

>
>  gst/mpegvideoparse/mpegpacketiser.c |    6 ------
>  1 files changed, 0 insertions(+), 6 deletions(-)
>
> diff --git a/gst/mpegvideoparse/mpegpacketiser.c b/gst/mpegvideoparse/mpegpacketiser.c
> index 694c476..2a95fa6 100644
> --- a/gst/mpegvideoparse/mpegpacketiser.c
> +++ b/gst/mpegvideoparse/mpegpacketiser.c
> @@ -511,8 +511,6 @@ mpeg_util_parse_extension_packet (MPEGSeqHdr * hdr, guint8 * data, guint8 * end)
>      case MPEG_PACKET_EXT_SEQUENCE:
>      {
>        /* Parse a Sequence Extension */
> -      gboolean low_delay;
> -      guint8 chroma_format;
>        guint8 horiz_size_ext, vert_size_ext;
>        guint8 fps_n_ext, fps_d_ext;
>  
> @@ -523,10 +521,8 @@ mpeg_util_parse_extension_packet (MPEGSeqHdr * hdr, guint8 * data, guint8 * end)
>        hdr->profile = data[0] & 0x0f;    /* profile (0:2) + escape bit (3) */
>        hdr->level = (data[1] >> 4) & 0x0f;
>        hdr->progressive = data[1] & 0x08;
> -      chroma_format = (data[1] >> 2) & 0x03;
>        horiz_size_ext = ((data[1] << 1) & 0x02) | ((data[2] >> 7) & 0x01);
>        vert_size_ext = (data[2] >> 5) & 0x03;
> -      low_delay = data[5] >> 7;
>        fps_n_ext = (data[5] >> 5) & 0x03;
>        fps_d_ext = data[5] & 0x1f;
>  
> @@ -549,7 +545,6 @@ mpeg_util_parse_sequence_hdr (MPEGSeqHdr * hdr, guint8 * data, guint8 * end)
>    guint32 code;
>    guint8 dar_idx, fps_idx;
>    guint32 sync_word = 0xffffffff;
> -  gboolean constrained_flag;
>    gboolean load_intra_flag;
>    gboolean load_non_intra_flag;
>  
> @@ -584,7 +579,6 @@ mpeg_util_parse_sequence_hdr (MPEGSeqHdr * hdr, guint8 * data, guint8 * end)
>      hdr->bitrate *= 400;
>    }
>  
> -  constrained_flag = (data[7] >> 2) & 0x01;
>    load_intra_flag = (data[7] >> 1) & 0x01;
>    if (load_intra_flag) {
>      if (G_UNLIKELY ((end - data) < 64))
>
> _______________________________________________
> gstreamer-commits mailing list
> [hidden email]
> http://lists.freedesktop.org/mailman/listinfo/gstreamer-commits

_______________________________________________
gstreamer-devel mailing list
[hidden email]
http://lists.freedesktop.org/mailman/listinfo/gstreamer-devel