[FFmpeg-devel] [PATCH] Document that av_write_header can change stream time_base.
Reimar Döffinger
Reimar.Doeffinger
Sat Jan 29 14:47:52 CET 2011
On Sat, Jan 29, 2011 at 05:01:17AM -0800, Baptiste Coudurier wrote:
> On 1/29/11 4:54 AM, Reimar D?ffinger wrote:
> > ---
> > libavformat/avformat.h | 4 ++++
> > 1 files changed, 4 insertions(+), 0 deletions(-)
> >
> > diff --git a/libavformat/avformat.h b/libavformat/avformat.h
> > index f9f9be5..5e01922 100644
> > --- a/libavformat/avformat.h
> > +++ b/libavformat/avformat.h
> > @@ -507,6 +507,8 @@ typedef struct AVStream {
> > * This is the fundamental unit of time (in seconds) in terms
> > * of which frame timestamps are represented. For fixed-fps content,
> > * time base should be 1/framerate and timestamp increments should be 1.
> > + * decoding: set by libavformat
> > + * encoding: set by user, possibly modified by libavformat in av_write_header
> > */
>
> Setting time_base in AVStream when muxing doesn't work directly.
> User has to use AVCodecContext->time_base, which is used to specify the
> frame rate usually, for VFR set it to arbitrary large denominator value.
I have the feeling this is a huge hack that was used only because it worked
better in FFmpeg not because it makes sense. But whatever.
Is there a way to reply to review and send in a new patch without having to
send two emails? This seems annoyingly inefficent and discourages actually
discussing reviews.
More information about the ffmpeg-devel
mailing list