Opened 13 years ago

Closed 13 years ago

#47 closed defect (fixed)

mpegts muxer doesn't override time base, but assumes 1/90000

Reported by: Rudolf Polzer Owned by:
Priority: normal Component: avformat
Version: git Keywords: mpeg
Cc: Blocked By:
Blocking: Reproduced by developer: no
Analyzed by developer: no

Description

The subject says all: the mpegts muxer does not set the stream time base using av_set_pts_info, but expects it to have the default value of 1/90000. If the calling code changes stream pts before writing the header, other muxers override the time base at that point (like mpegenc.c). mpegtsenc.c does not, causing this issue.

In my code, I had to work around this by not setting the stream time base at all.

I attached a patch that adds the missing function call to mpegtsenc.c.

Attachments (1)

fix90000.diff (476 bytes ) - added by Rudolf Polzer 13 years ago.

Download all attachments as: .zip

Change History (2)

by Rudolf Polzer, 13 years ago

Attachment: fix90000.diff added

comment:1 by Carl Eugen Hoyos, 13 years ago

Resolution: fixed
Status: newclosed

Patch pushed in 0a962e61e956addb7cdc829e41b82fd76bea55db (April 14)

Note: See TracTickets for help on using tickets.