Opened 6 years ago

Closed 6 years ago

#47 closed defect (fixed)

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

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


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 divVerent 6 years ago.

Download all attachments as: .zip

Change History (2)

Changed 6 years ago by divVerent

comment:1 Changed 6 years ago by cehoyos

  • Resolution set to fixed
  • Status changed from new to closed

Patch pushed in 0a962e61e956addb7cdc829e41b82fd76bea55db (April 14)

Note: See TracTickets for help on using tickets.