[FFmpeg-devel] [PATCH] doxy: Clarify what avpriv_set_pts_info does

Derek Buitenhuis derek.buitenhuis at gmail.com
Sun Dec 16 21:41:43 CET 2012


The "pts for a given stream" was nonsensical.

Signed-off-by: Derek Buitenhuis <derek.buitenhuis at gmail.com>
---
 libavformat/internal.h |    9 +++++----
 1 file changed, 5 insertions(+), 4 deletions(-)

diff --git a/libavformat/internal.h b/libavformat/internal.h
index 14f4cdb..840b9d2 100644
--- a/libavformat/internal.h
+++ b/libavformat/internal.h
@@ -292,14 +292,15 @@ int64_t ff_gen_search(AVFormatContext *s, int stream_index,
                       int64_t (*read_timestamp)(struct AVFormatContext *, int , int64_t *, int64_t ));
 
 /**
- * Set the pts for a given stream. If the new values would be invalid
- * (<= 0), it leaves the AVStream unchanged.
+ * Set the time base for a given stream. If the new time base is invalid
+ * (numerator or denominator are, or are less than zero), it leaves the stream
+ * unchanged.
  *
  * @param s stream
  * @param pts_wrap_bits number of bits effectively used by the pts
  *        (used for wrap control, 33 is the value for MPEG)
- * @param pts_num numerator to convert to seconds (MPEG: 1)
- * @param pts_den denominator to convert to seconds (MPEG: 90000)
+ * @param pts_num time base numerator (MPEG: 1)
+ * @param pts_den time base denominator (MPEG: 90000)
  */
 void avpriv_set_pts_info(AVStream *s, int pts_wrap_bits,
                          unsigned int pts_num, unsigned int pts_den);
-- 
1.7.10.4



More information about the ffmpeg-devel mailing list