Le decadi 10 frimaire, an CCXXIV, Michael Niedermayer a écrit :
> this should mention in which timebase the pts is and possibly
> relative to what the timestamp is
> its rather obvious to write
> AVERROR(ENOMEM)
> pts = av_gettime()
> based on this documentation

Changed to this:

/**
 * Set the status field of a link from the source filter.
 * The pts should reflect the timestamp of the status change,
 * in link time base and relative to the frames timeline.
 * In particular, for AVERROR_EOF, it should reflect the
 * end time of the last frame.
 */

> s/unset/AV_NOPTS_VALUE/ ?

Changed to:

 * The pts should probably be left unset (AV_NOPTS_VALUE).

> rest of this patch LGTM, no need to resubmit for just above things
> if there are no other reasons to do so

Thanks for the review.

Regards,

-- 
  Nicolas George

Attachment: signature.asc
Description: Digital signature

_______________________________________________
ffmpeg-devel mailing list
ffmpeg-devel@ffmpeg.org
http://ffmpeg.org/mailman/listinfo/ffmpeg-devel

Reply via email to