[FFmpeg-devel,01/10] avformat/matroskadec: Remove an unused variable

Submitted by Andreas Rheinhardt on March 8, 2019, 9:25 a.m.

Details

Message ID 20190308092604.3752-2-andreas.rheinhardt@googlemail.com
State New
Headers show

Commit Message

Andreas Rheinhardt March 8, 2019, 9:25 a.m.
Signed-off-by: Andreas Rheinhardt <andreas.rheinhardt@googlemail.com>
---
 libavformat/matroskadec.c | 3 ---
 1 file changed, 3 deletions(-)

Comments

James Almer March 8, 2019, 10:38 p.m.
On 3/8/2019 6:25 AM, Andreas Rheinhardt wrote:
> Signed-off-by: Andreas Rheinhardt <andreas.rheinhardt@googlemail.com>
> ---
>  libavformat/matroskadec.c | 3 ---
>  1 file changed, 3 deletions(-)
> 
> diff --git a/libavformat/matroskadec.c b/libavformat/matroskadec.c
> index 0e3a6890c1..de27d63b17 100644
> --- a/libavformat/matroskadec.c
> +++ b/libavformat/matroskadec.c
> @@ -322,7 +322,6 @@ typedef struct MatroskaDemuxContext {
>      /* EBML stuff */
>      int num_levels;
>      MatroskaLevel levels[EBML_MAX_DEPTH];
> -    int level_up;
>      uint32_t current_id;
>  
>      uint64_t time_scale;
> @@ -1593,7 +1592,6 @@ static void matroska_convert_tags(AVFormatContext *s)
>  static int matroska_parse_seekhead_entry(MatroskaDemuxContext *matroska,
>                                           uint64_t pos)
>  {
> -    uint32_t level_up       = matroska->level_up;
>      uint32_t saved_id       = matroska->current_id;
>      int64_t before_pos = avio_tell(matroska->ctx->pb);
>      MatroskaLevel level;
> @@ -1629,7 +1627,6 @@ static int matroska_parse_seekhead_entry(MatroskaDemuxContext *matroska,
>      }
>      /* seek back */
>      avio_seek(matroska->ctx->pb, before_pos, SEEK_SET);
> -    matroska->level_up   = level_up;
>      matroska->current_id = saved_id;
>  
>      return ret;

LGTM.

Patch hide | download patch | download mbox

diff --git a/libavformat/matroskadec.c b/libavformat/matroskadec.c
index 0e3a6890c1..de27d63b17 100644
--- a/libavformat/matroskadec.c
+++ b/libavformat/matroskadec.c
@@ -322,7 +322,6 @@  typedef struct MatroskaDemuxContext {
     /* EBML stuff */
     int num_levels;
     MatroskaLevel levels[EBML_MAX_DEPTH];
-    int level_up;
     uint32_t current_id;
 
     uint64_t time_scale;
@@ -1593,7 +1592,6 @@  static void matroska_convert_tags(AVFormatContext *s)
 static int matroska_parse_seekhead_entry(MatroskaDemuxContext *matroska,
                                          uint64_t pos)
 {
-    uint32_t level_up       = matroska->level_up;
     uint32_t saved_id       = matroska->current_id;
     int64_t before_pos = avio_tell(matroska->ctx->pb);
     MatroskaLevel level;
@@ -1629,7 +1627,6 @@  static int matroska_parse_seekhead_entry(MatroskaDemuxContext *matroska,
     }
     /* seek back */
     avio_seek(matroska->ctx->pb, before_pos, SEEK_SET);
-    matroska->level_up   = level_up;
     matroska->current_id = saved_id;
 
     return ret;