Re: [FFmpeg-devel] [PATCH 1/4] avformat/mpegtsenc: use standard pids for m2ts

2020-04-13 Thread Petri Hintukainen
pe, 2020-04-10 kello 21:44 +0200, Marton Balint kirjoitti: > Signed-off-by: Marton Balint > --- > doc/muxers.texi | 6 -- > libavformat/mpegts.h| 10 ++ > libavformat/mpegtsenc.c | 44 > +++- > 3 files changed, 57 insertions(+), 3 d

Re: [FFmpeg-devel] [PATCH 1/3] lavc/pgs_frame_merge_bsf: add bsf to merge PGS segments

2020-04-17 Thread Petri Hintukainen
to, 2020-04-16 kello 16:57 -0600, John Stebbins kirjoitti: > Required to remux m2ts to mkv > --- > libavcodec/Makefile | 1 + > libavcodec/bitstream_filters.c | 1 + > libavcodec/pgs_frame_merge_bsf.c | 164 > +++ > 3 files changed, 166 insertions(+)

Re: [FFmpeg-devel] [PATCH] pgssubdec: only set w/h/linesize when allocating data

2016-11-23 Thread Petri Hintukainen
ti, 2016-11-22 kello 23:21 +0100, Andreas Cadhalpun kirjoitti: > On 09.11.2016 23:27, Andreas Cadhalpun wrote: > > > > Rects with positive w/h/linesize but no data are invalid. > > > > Signed-off-by: Andreas Cadhalpun > > --- > >  libavcodec/pgssubdec.c | 9 + > >  1 file changed, 5 inser

Re: [FFmpeg-devel] [PATCH] avcodec: add HDMV Text Subtitle decoder

2017-01-31 Thread Petri Hintukainen
+= hapdec.o hap.o OBJS-$(CONFIG_HAP_ENCODER) += hapenc.o hap.o +OBJS-$(CONFIG_HDMV_TEXTST_DECODER) += hdmv_textstdec.o ass.o OBJS-$(CONFIG_HEVC_DECODER)+= hevc.o hevc_mvs.o hevc_ps.o hevc_sei.o \ hevc_cabac.o hevc_refs.o hevcpred.

Re: [FFmpeg-devel] [PATCH] avcodec: add HDMV Text Subtitle decoder

2017-02-01 Thread Petri Hintukainen
ti, 2017-01-31 kello 16:42 +0100, wm4 kirjoitti: > On Tue, 31 Jan 2017 15:22:17 +0100 > Paul B Mahol wrote: > > > Signed-off-by: Paul B Mahol > > --- > >  libavcodec/Makefile|   2 + > >  libavcodec/allcodecs.c |   2 + > >  libavcodec/textst_parser.c |  49 > >  li

Re: [FFmpeg-devel] [PATCH] avcodec: add HDMV Text Subtitle decoder

2017-02-02 Thread Petri Hintukainen
ti, 2017-01-31 kello 20:07 +0100, Paul B Mahol kirjoitti: > On 1/31/17, Petri Hintukainen wrote: > > Hello, > > > > I think I posted HDMV text subtitle decoder patch year+ ago. But I > > can't find it from list archives (?). > > > > I'll attach

[FFmpeg-devel] [PATCH] matroska: demux BluRay text subtitles

2017-02-05 Thread Petri Hintukainen
--- libavformat/matroska.c | 1 + 1 file changed, 1 insertion(+) diff --git a/libavformat/matroska.c b/libavformat/matroska.c index c8e5341..fda96fb 100644 --- a/libavformat/matroska.c +++ b/libavformat/matroska.c @@ -74,6 +74,7 @@ const CodecTags ff_mkv_codec_tags[]={ {"S_VOBSUB" ,

[FFmpeg-devel] [PATCH] Add SUP (raw BluRay PGS subtitle) muxer

2014-08-29 Thread Petri Hintukainen
From: Petri Hintukainen Fixes ticket #2208 --- libavformat/Makefile | 1 + libavformat/allformats.c | 1 + libavformat/supenc.c | 61 3 files changed, 63 insertions(+) create mode 100644 libavformat/supenc.c diff --git a/libavformat

Re: [FFmpeg-devel] [PATCH] Add SUP (raw BluRay PGS subtitle) muxer

2014-08-29 Thread Petri Hintukainen
On pe, 2014-08-29 at 12:51 +0200, wm4 wrote: > > Do you plan on writing a demuxer too? If not, then I'd give it a try. Probably not (at least in next week or two). It should be simple to parse :). After the header there's always segment type byte and length (16-bits BE). Without index seeking is

Re: [FFmpeg-devel] [PATCH] Add SUP (raw BluRay PGS subtitle) muxer

2014-08-29 Thread Petri Hintukainen
On pe, 2014-08-29 at 14:06 +0200, Hendrik Leppkes wrote: > On Fri, Aug 29, 2014 at 12:31 PM, Petri Hintukainen > wrote: > > From: Petri Hintukainen > > > > Fixes ticket #2208 > > Is the format officially documented somewhere, so interoperability > with other

Re: [FFmpeg-devel] [PATCH] Add SUP (raw BluRay PGS subtitle) muxer

2014-08-29 Thread Petri Hintukainen
On pe, 2014-08-29 at 15:02 +, Carl Eugen Hoyos wrote: > Petri Hintukainen gmail.com> writes: > > > From: Petri Hintukainen users.sourceforge.net> > > > > Fixes ticket #2208 > > Do I understand correctly that the produced files are > compatible with

Re: [FFmpeg-devel] [PATCH v5] Add SUP/PGS subtitle demuxer

2014-09-01 Thread Petri Hintukainen
On ma, 2014-09-01 at 00:34 +0200, wm4 wrote: > --- > Use AV_RB16 instead of memcpy. > Don't use AVPROBE_SCORE_EXTENSION. > --- > libavformat/Makefile | 1 + > libavformat/allformats.c | 1 + > libavformat/supdec.c | 107 > +++ > 3 files chan

Re: [FFmpeg-devel] [PATCH v5] Add SUP/PGS subtitle demuxer

2014-09-01 Thread Petri Hintukainen
On ma, 2014-09-01 at 10:05 +0200, Hendrik Leppkes wrote: > On Mon, Sep 1, 2014 at 10:01 AM, Petri Hintukainen wrote: > > On ma, 2014-09-01 at 00:34 +0200, wm4 wrote: > >> --- > >> Use AV_RB16 instead of memcpy. > >> Don't use AVPROBE_SCORE_EXTENSION. >

Re: [FFmpeg-devel] [PATCH v5] Add SUP/PGS subtitle demuxer

2014-09-03 Thread Petri Hintukainen
On ma, 2014-09-01 at 16:45 +0200, wm4 wrote: > On Mon, 1 Sep 2014 08:34:52 + (UTC) > Carl Eugen Hoyos wrote: > > > wm4 googlemail.com> writes: > > > +full_packet_size = AV_RB16(buf + 10 + 1) + 10 + 3; > > > +if (buf_size < full_packet_size) > > > +break; > > > > I

Re: [FFmpeg-devel] [PATCH v5] Add SUP/PGS subtitle demuxer

2014-09-03 Thread Petri Hintukainen
On ma, 2014-09-01 at 16:40 +0200, wm4 wrote: > On Mon, 01 Sep 2014 13:37:23 +0300 > Petri Hintukainen wrote: > > > On ma, 2014-09-01 at 10:05 +0200, Hendrik Leppkes wrote: > > > On Mon, Sep 1, 2014 at 10:01 AM, Petri Hintukainen > > > wrote: > > > >

Re: [FFmpeg-devel] [PATCH] Add SUP (raw BluRay PGS subtitle) muxer

2014-09-04 Thread Petri Hintukainen
On to, 2014-09-04 at 10:19 +0200, Hendrik Leppkes wrote: > On Fri, Aug 29, 2014 at 12:31 PM, Petri Hintukainen > wrote: > > From: Petri Hintukainen > > > > Fixes ticket #2208 > > --- > > libavformat/Makefile | 1 + > > libavformat/allformats.c

Re: [FFmpeg-devel] [PATCH v7] Add SUP/PGS subtitle demuxer

2014-09-26 Thread Petri Hintukainen
On pe, 2014-09-26 at 12:26 +0200, Clément Bœsch wrote: > On Fri, Sep 26, 2014 at 10:13:34AM +, Carl Eugen Hoyos wrote: > > Clément Bœsch pkh.me> writes: > > > > > Thank you for the patch. Looking forward the muxer, and a FATE test. > > > > Is there a reason why you didn't merge the muxer? >

Re: [FFmpeg-devel] [PATCH v7] Add SUP/PGS subtitle demuxer

2014-09-30 Thread Petri Hintukainen
Hello, Missing DTS combined with non-monotonic PTS seems to cause some problems. This is with the file from ticket #2208 (I added dumping timestamps to demuxer and muxer): $ ../ffmpeg -i orig/track_06\ -\ Subtitle.sup -scodec copy 6.sup ffmpeg version N-66546-gea74007 Copyright (c) 2000-2014 the

Re: [FFmpeg-devel] [PATCH v2] Add SUP/PGS subtitle muxer

2014-10-01 Thread Petri Hintukainen
On ti, 2014-09-30 at 15:56 +0200, wm4 wrote: > On Tue, 30 Sep 2014 11:01:17 +0300 > phint...@gmail.com wrote: > > > From: Petri Hintukainen > > > > Fixes ticket #2208 > > --- > > Changelog| 2 +- > > doc/general.texi

Re: [FFmpeg-devel] [PATCH v7] Add SUP/PGS subtitle demuxer

2014-10-01 Thread Petri Hintukainen
On ti, 2014-09-30 at 16:09 +0200, wm4 wrote: > On Tue, 30 Sep 2014 12:01:33 +0300 > Petri Hintukainen wrote: > > > Hello, > > > > Missing DTS combined with non-monotonic PTS seems to cause some > > problems. This is with the file from ticket #2208 (I added dump

[FFmpeg-devel] [PATCH] mpegts: fix demuxing PES private stream 2

2015-08-31 Thread Petri Hintukainen
PES header size is 6 bytes (00 00 01 bf XX XX), not 0. BluRay text subtitles use private stream 2. --- libavformat/mpegts.c | 1 + 1 file changed, 1 insertion(+) diff --git a/libavformat/mpegts.c b/libavformat/mpegts.c index eac6d9e..625d473 100644 --- a/libavformat/mpegts.c +++ b/libavformat/mpe

[FFmpeg-devel] [PATCH] mpegts: demux BluRay text subtitles

2015-08-31 Thread Petri Hintukainen
--- libavcodec/avcodec.h| 1 + libavcodec/codec_desc.c | 7 +++ libavformat/mpegts.c| 1 + 3 files changed, 9 insertions(+) diff --git a/libavcodec/avcodec.h b/libavcodec/avcodec.h index 12a6b54..b4cf79d 100644 --- a/libavcodec/avcodec.h +++ b/libavcodec/avcodec.h @@ -516,6 +516,7 @@

Re: [FFmpeg-devel] [PATCH] mpegts: demux BluRay text subtitles

2015-09-01 Thread Petri Hintukainen
On ma, 2015-08-31 at 09:25 +, Carl Eugen Hoyos wrote: > Petri Hintukainen gmail.com> writes: > > > +AV_CODEC_ID_HDMV_TEXT_SUBTITLE, > > No, I believe we still "maintain" the fork, you > have to force a specific tag, something like > "TSTX&quo

[FFmpeg-devel] [PATCH v2] mpegts: demux BluRay text subtitles

2015-09-01 Thread Petri Hintukainen
--- libavcodec/avcodec.h| 1 + libavcodec/codec_desc.c | 7 +++ libavformat/mpegts.c| 1 + 3 files changed, 9 insertions(+) diff --git a/libavcodec/avcodec.h b/libavcodec/avcodec.h index 12a6b54..9d38b59 100644 --- a/libavcodec/avcodec.h +++ b/libavcodec/avcodec.h @@ -530,6 +530,7 @@

Re: [FFmpeg-devel] [PATCH v2] mpegts: demux BluRay text subtitles

2015-09-01 Thread Petri Hintukainen
On ti, 2015-09-01 at 09:06 +, Paul B Mahol wrote: > On 9/1/15, Petri Hintukainen wrote: > > --- > > libavcodec/avcodec.h| 1 + > > libavcodec/codec_desc.c | 7 +++ > > libavformat/mpegts.c| 1 + > > 3 files changed, 9 insertions(+) > >

Re: [FFmpeg-devel] [PATCH] Fixed remuxing of HDMV PGS subtitles

2015-04-13 Thread Petri Hintukainen
On su, 2015-04-12 at 16:40 -0700, Philip Langdale wrote: > On Sun, 12 Apr 2015 08:13:39 + (UTC) > Carl Eugen Hoyos wrote: > > > > I tested with samples from the following directories: > > http://samples.ffmpeg.org/ffmpeg-bugs/trac/ticket598/ > > http://samples.ffmpeg.org/ffmpeg-bugs/trac/tick

Re: [FFmpeg-devel] [PATCH] Fixed remuxing of HDMV PGS subtitles

2015-04-21 Thread Petri Hintukainen
On ma, 2015-04-13 at 09:41 -0700, Philip Langdale wrote: > On 2015-04-13 03:39, Petri Hintukainen wrote: > > > > I don't know if it is a good idea to use HDMV stream types without HDMV > > program registration descriptor. If this is fixed incrementally, fixing >