On Sat, Apr 09, 2011 at 08:42:10AM -0400, Ronald S. Bultje wrote: > Hi, > > On Sat, Apr 9, 2011 at 7:50 AM, Reinhard Tartler <[email protected]> wrote: > > From: Georgi Chorbadzhiyski <[email protected]> > > > > This patch adds support for setting transport_stream_id, > > original_network_id, service_id, pmt_start_pid and start_pid > > in mpegts muxer. > > > > Signed-off-by: Ronald S. Bultje <[email protected]> > > (cherry picked from commit 445996aa51f4f1d9a26456a8511988291a720ba0) > > --- > > libavformat/mpegtsenc.c | 2 +- > > 1 files changed, 1 insertions(+), 1 deletions(-) > > > > diff --git a/libavformat/mpegtsenc.c b/libavformat/mpegtsenc.c > > index 40b906e..66ccfc6 100644 > > --- a/libavformat/mpegtsenc.c > > +++ b/libavformat/mpegtsenc.c > > @@ -421,7 +421,7 @@ static MpegTSService *mpegts_add_service(MpegTSWrite > > *ts, > > service = av_mallocz(sizeof(MpegTSService)); > > if (!service) > > return NULL; > > - service->pmt.pid = ts->pmt_start_pid + ts->nb_services - 1; > > + service->pmt.pid = ts->pmt_start_pid + ts->nb_services; > > service->sid = sid; > > service->provider_name = av_strdup(provider_name); > > service->name = av_strdup(name); > > The commit msg is wrong. I think some commits were mixed up here. The > commit msg should likely read: > > "mpegtsenc: fix setting start_pid, which was off by one." > > Then it's OK, please apply.
Actually no, the value in the original patch was wrong and was later fixed in 740ad0d1. This patch would undo the fix. -- Anton Khirnov
signature.asc
Description: Digital signature
_______________________________________________ libav-devel mailing list [email protected] https://lists.libav.org/mailman/listinfo/libav-devel
