On Sat, 28 Apr 2018 15:28:13 -0300 James Almer <jamr...@gmail.com> wrote:
> On 4/28/2018 2:05 PM, wm4 wrote: > > This can "demux" .vpy files. > > > > + pkt->data = pkt->buf->data; > > + pkt->size = pkt->buf->size; > > + pkt->flags |= AV_PKT_FLAG_TRUSTED; > > + > > + if (vs->is_cfr) > > + pkt->pts = vs->current_frame; > > + > > + vs->current_frame++; > > + > > +end: > > + if (err < 0) > > + av_packet_unref(pkt); > > Unneeded. Nothing after pkt->buf is initialized can fail right now. And > if av_buffer_create() fails, pkt will be untouched. Could be good for robustness reasons, but OK, removed. > > + av_frame_free(&frame); > > + vs->vsapi->freeFrame(vsframe); > > + return err; > > Shouldn't you set err to pkt->size right above the end label? I'm not sure what you mean. err is initialized to 0, and only becomes non-0 on errors, so it doesn't need to be set here. _______________________________________________ ffmpeg-devel mailing list ffmpeg-devel@ffmpeg.org http://ffmpeg.org/mailman/listinfo/ffmpeg-devel
