diff mbox

[FFmpeg-devel,v1] avformat/rtpdec.h remove unused variable

Message ID 20190312212622.13569-1-junli1026@gmail.com
State Accepted
Commit c2a221c5ae5017e11654b9688ac97e9f5d3570b2
Headers show

Commit Message

Jun Li March 12, 2019, 9:26 p.m. UTC
Looks like the variable 'cur_timestamp' is not used anywhere.
So remove this variable.

Signed-off-by: Jun Li <junli1026@gmail.com>
---
 libavformat/rtpdec.h | 1 -
 1 file changed, 1 deletion(-)

Comments

Liu Steven March 12, 2019, 11:22 p.m. UTC | #1
> On Mar 13, 2019, at 05:26, Jun Li <junli1026@gmail.com> wrote:
> 
> Looks like the variable 'cur_timestamp' is not used anywhere.
> So remove this variable.
> 
> Signed-off-by: Jun Li <junli1026@gmail.com>
> ---
> libavformat/rtpdec.h | 1 -
> 1 file changed, 1 deletion(-)
> 
> diff --git a/libavformat/rtpdec.h b/libavformat/rtpdec.h
> index 5a47d6f79d..9144edbe8b 100644
> --- a/libavformat/rtpdec.h
> +++ b/libavformat/rtpdec.h
> @@ -154,7 +154,6 @@ struct RTPDemuxContext {
>     uint16_t seq;
>     uint32_t timestamp;
>     uint32_t base_timestamp;
> -    uint32_t cur_timestamp;
>     int64_t  unwrapped_timestamp;
>     int64_t  range_start_offset;
>     int max_payload_size;
> -- 
> 2.17.1
> 
> _______________________________________________
> ffmpeg-devel mailing list
> ffmpeg-devel@ffmpeg.org
> https://ffmpeg.org/mailman/listinfo/ffmpeg-devel

LGTM

Thanks
Steven
Michael Niedermayer March 13, 2019, 8:33 p.m. UTC | #2
On Wed, Mar 13, 2019 at 07:22:41AM +0800, Steven Liu wrote:
> 
> 
> > On Mar 13, 2019, at 05:26, Jun Li <junli1026@gmail.com> wrote:
> > 
> > Looks like the variable 'cur_timestamp' is not used anywhere.
> > So remove this variable.
> > 
> > Signed-off-by: Jun Li <junli1026@gmail.com>
> > ---
> > libavformat/rtpdec.h | 1 -
> > 1 file changed, 1 deletion(-)
> > 
> > diff --git a/libavformat/rtpdec.h b/libavformat/rtpdec.h
> > index 5a47d6f79d..9144edbe8b 100644
> > --- a/libavformat/rtpdec.h
> > +++ b/libavformat/rtpdec.h
> > @@ -154,7 +154,6 @@ struct RTPDemuxContext {
> >     uint16_t seq;
> >     uint32_t timestamp;
> >     uint32_t base_timestamp;
> > -    uint32_t cur_timestamp;
> >     int64_t  unwrapped_timestamp;
> >     int64_t  range_start_offset;
> >     int max_payload_size;
> > -- 
> > 2.17.1
> > 
> > _______________________________________________
> > ffmpeg-devel mailing list
> > ffmpeg-devel@ffmpeg.org
> > https://ffmpeg.org/mailman/listinfo/ffmpeg-devel
> 
> LGTM

will apply

thx

[...]
diff mbox

Patch

diff --git a/libavformat/rtpdec.h b/libavformat/rtpdec.h
index 5a47d6f79d..9144edbe8b 100644
--- a/libavformat/rtpdec.h
+++ b/libavformat/rtpdec.h
@@ -154,7 +154,6 @@  struct RTPDemuxContext {
     uint16_t seq;
     uint32_t timestamp;
     uint32_t base_timestamp;
-    uint32_t cur_timestamp;
     int64_t  unwrapped_timestamp;
     int64_t  range_start_offset;
     int max_payload_size;