[FFmpeg-devel] [PATCH 2/2] ffprobe: refactor frames decoding.
Nicolas George
nicolas.george at normalesup.org
Sun Jul 29 16:10:31 CEST 2012
L'octidi 8 thermidor, an CCXX, Stefano Sabatini a écrit :
> process_decoded_frame -> process_frame
>
> the latter is simpler and seems less misleading (since the frame is
> not yet decoded)
Locally changed.
> > + while (pkt1.size &&
> > + process_decoded_frame(w, fmt_ctx, &frame, &pkt1) > 0);
> nit: maybe drop "> 0".
process_(decoded)_frame() returning 0 means stop too, just not an error.
> Now this is doing:
> pkt->data += ret;
> pkt->size -= ret;
>
> and pkt->data is NULL.
>
> Verify that the decoder is returning 0 when the data is flushed (as I
> suppose it does).
The last patch of the patchset I just sent addresses that.
> Looks good otherwise, thanks.
Thanks, I'll push when the previous patch is updated.
Regards,
--
Nicolas George
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 198 bytes
Desc: Digital signature
URL: <http://ffmpeg.org/pipermail/ffmpeg-devel/attachments/20120729/56ff2616/attachment.asc>
More information about the ffmpeg-devel
mailing list