On 8/30/20, Andreas Rheinhardt <andreas.rheinha...@gmail.com> wrote: > av_new_packet() already sets the size. And if the packet is not > allocated by av_new_packet() (which seems to be impossible atm), both > pkt->size as well as size are 0, so setting it again is unnecessary in > this scenario, too. > > Signed-off-by: Andreas Rheinhardt <andreas.rheinha...@gmail.com> > --- > libavdevice/lavfi.c | 1 - > 1 file changed, 1 deletion(-) >
LGTM > diff --git a/libavdevice/lavfi.c b/libavdevice/lavfi.c > index 2b4e9290dc..4f05a1542e 100644 > --- a/libavdevice/lavfi.c > +++ b/libavdevice/lavfi.c > @@ -464,7 +464,6 @@ static int lavfi_read_packet(AVFormatContext *avctx, > AVPacket *pkt) > pkt->stream_index = stream_idx; > pkt->pts = frame->pts; > pkt->pos = frame->pkt_pos; > - pkt->size = size; > av_frame_unref(frame); > return size; > } > -- > 2.20.1 > > _______________________________________________ > ffmpeg-devel mailing list > ffmpeg-devel@ffmpeg.org > https://ffmpeg.org/mailman/listinfo/ffmpeg-devel > > To unsubscribe, visit link above, or email > ffmpeg-devel-requ...@ffmpeg.org with subject "unsubscribe". _______________________________________________ ffmpeg-devel mailing list ffmpeg-devel@ffmpeg.org https://ffmpeg.org/mailman/listinfo/ffmpeg-devel To unsubscribe, visit link above, or email ffmpeg-devel-requ...@ffmpeg.org with subject "unsubscribe".
