diff mbox series

[FFmpeg-devel] avformat/ivfenc: remove unused var

Message ID 20240417113106.1754-1-ffmpeg@gyani.pro
State Accepted
Commit 38c322681e9715b288172eaed1e90205aa46e969
Headers show
Series [FFmpeg-devel] avformat/ivfenc: remove unused var | expand

Checks

Context Check Description
yinshiyou/make_loongarch64 success Make finished
yinshiyou/make_fate_loongarch64 success Make fate finished
andriy/make_x86 success Make finished
andriy/make_fate_x86 success Make fate finished

Commit Message

Gyan Doshi April 17, 2024, 11:31 a.m. UTC
sum_delta_pts is unused since 3b358f151d
---
 libavformat/ivfenc.c | 4 +---
 1 file changed, 1 insertion(+), 3 deletions(-)

Comments

James Almer April 17, 2024, 11:55 a.m. UTC | #1
On 4/17/2024 8:31 AM, Gyan Doshi wrote:
> sum_delta_pts is unused since 3b358f151d
> ---
>   libavformat/ivfenc.c | 4 +---
>   1 file changed, 1 insertion(+), 3 deletions(-)
> 
> diff --git a/libavformat/ivfenc.c b/libavformat/ivfenc.c
> index 09782eecd6..9feaea3516 100644
> --- a/libavformat/ivfenc.c
> +++ b/libavformat/ivfenc.c
> @@ -24,7 +24,7 @@
>   
>   typedef struct IVFEncContext {
>       unsigned frame_cnt;
> -    uint64_t last_pts, sum_delta_pts, last_pkt_duration;
> +    uint64_t last_pts, last_pkt_duration;
>   } IVFEncContext;
>   
>   static int ivf_init(AVFormatContext *s)
> @@ -80,8 +80,6 @@ static int ivf_write_packet(AVFormatContext *s, AVPacket *pkt)
>       avio_wl32(pb, pkt->size);
>       avio_wl64(pb, pkt->pts);
>       avio_write(pb, pkt->data, pkt->size);
> -    if (ctx->frame_cnt)
> -        ctx->sum_delta_pts += pkt->pts - ctx->last_pts;
>       ctx->last_pkt_duration = pkt->duration;
>       ctx->frame_cnt++;
>       ctx->last_pts = pkt->pts;

Ok.
Gyan Doshi April 17, 2024, 1:02 p.m. UTC | #2
On 2024-04-17 05:25 pm, James Almer wrote:
> On 4/17/2024 8:31 AM, Gyan Doshi wrote:
>> sum_delta_pts is unused since 3b358f151d
>> ---
>>   libavformat/ivfenc.c | 4 +---
>>   1 file changed, 1 insertion(+), 3 deletions(-)
>>
>> diff --git a/libavformat/ivfenc.c b/libavformat/ivfenc.c
>> index 09782eecd6..9feaea3516 100644
>> --- a/libavformat/ivfenc.c
>> +++ b/libavformat/ivfenc.c
>> @@ -24,7 +24,7 @@
>>     typedef struct IVFEncContext {
>>       unsigned frame_cnt;
>> -    uint64_t last_pts, sum_delta_pts, last_pkt_duration;
>> +    uint64_t last_pts, last_pkt_duration;
>>   } IVFEncContext;
>>     static int ivf_init(AVFormatContext *s)
>> @@ -80,8 +80,6 @@ static int ivf_write_packet(AVFormatContext *s, 
>> AVPacket *pkt)
>>       avio_wl32(pb, pkt->size);
>>       avio_wl64(pb, pkt->pts);
>>       avio_write(pb, pkt->data, pkt->size);
>> -    if (ctx->frame_cnt)
>> -        ctx->sum_delta_pts += pkt->pts - ctx->last_pts;
>>       ctx->last_pkt_duration = pkt->duration;
>>       ctx->frame_cnt++;
>>       ctx->last_pts = pkt->pts;
>
> Ok.

Thanks. Pushed as 38c322681e9715b288172eaed1e90205aa46e969

Regards,
Gyan
diff mbox series

Patch

diff --git a/libavformat/ivfenc.c b/libavformat/ivfenc.c
index 09782eecd6..9feaea3516 100644
--- a/libavformat/ivfenc.c
+++ b/libavformat/ivfenc.c
@@ -24,7 +24,7 @@ 
 
 typedef struct IVFEncContext {
     unsigned frame_cnt;
-    uint64_t last_pts, sum_delta_pts, last_pkt_duration;
+    uint64_t last_pts, last_pkt_duration;
 } IVFEncContext;
 
 static int ivf_init(AVFormatContext *s)
@@ -80,8 +80,6 @@  static int ivf_write_packet(AVFormatContext *s, AVPacket *pkt)
     avio_wl32(pb, pkt->size);
     avio_wl64(pb, pkt->pts);
     avio_write(pb, pkt->data, pkt->size);
-    if (ctx->frame_cnt)
-        ctx->sum_delta_pts += pkt->pts - ctx->last_pts;
     ctx->last_pkt_duration = pkt->duration;
     ctx->frame_cnt++;
     ctx->last_pts = pkt->pts;