[Ffmpeg-devel] [PATCH] Move libavformat Doxygen and other comments to avformat.h

Michael Niedermayer michaelni
Sat Mar 3 20:18:19 CET 2007


Hi

On Sat, Mar 03, 2007 at 01:33:35PM +0100, Panagiotis Issaris wrote:
[...]
> >[...]
> >  
> >>+/**
> >>+ * Return the next frame of a stream.
> >>+ *
> >>+ * The returned packet is valid
> >>+ * until the next av_read_frame() or until av_close_input_file() and
> >>+ * must be freed with av_free_packet. For video, the packet contains
> >>+ * exactly one frame. For audio, it contains an integer number of
> >>+ * frames if each frame has a known fixed size (e.g. PCM or ADPCM
> >>+ * data). If the audio frames have a variable size (e.g. MPEG audio),
> >>+ * then it contains one frame.
> >>+ *
> >>+ * pkt->pts, pkt->dts and pkt->duration are always set to correct
> >>+ * values in AV_TIME_BASE unit (and guessed if the format cannot
> >>    
> >
> >this is wrong, they are in AVStream.timebase units
> >  
> So a simple change like this would be correct?
> 
> + * pkt->pts, pkt->dts and pkt->duration are always set to correct
> + * values in AVStream.timebase units (and guessed if the format cannot

yes


[...]
> Excellent. Applied the documentation moving patch and a patch correcting 
> the
> wrong docs is attached.

ok

[...]
-- 
Michael     GnuPG fingerprint: 9FF2128B147EF6730BADF133611EC787040B0FAB

The educated differ from the uneducated as much as the living from the
dead. -- Aristotle 
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 189 bytes
Desc: not available
URL: <http://lists.mplayerhq.hu/pipermail/ffmpeg-devel/attachments/20070303/ebaa8d48/attachment.pgp>



More information about the ffmpeg-devel mailing list