On 8/30/20, Andreas Rheinhardt <[email protected]> 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 <[email protected]> > --- > 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 > [email protected] > https://ffmpeg.org/mailman/listinfo/ffmpeg-devel > > To unsubscribe, visit link above, or email > [email protected] with subject "unsubscribe". _______________________________________________ ffmpeg-devel mailing list [email protected] https://ffmpeg.org/mailman/listinfo/ffmpeg-devel To unsubscribe, visit link above, or email [email protected] with subject "unsubscribe".
