Re: [FFmpeg-devel] Setting default G0 character set for teletext decoder

2020-02-18 Thread k.savkov
I can also provide sample files, for which this option is needed, where can I upload them? On 18.02.2020 23:48, Marton Balint wrote: On Tue, 18 Feb 2020, k.savkov wrote: Some providers don't send info about character set, so default (latin) is used when decoding. I added option to force

Re: [FFmpeg-devel] build/fate fail

2020-02-18 Thread Andriy Gelman
On Tue, 18. Feb 14:33, Lou Logan wrote: > On Tue, Feb 18, 2020, at 2:00 PM, Patchwork wrote: > > Hello, > > > > Thank you for submitting a patch to ffmpeg-devel. > > > > An error occurred during an automated build/fate test. Please review > > the following link for more details: > >

Re: [FFmpeg-devel] [PATCH 3/3] avformat/dashenc: always attempt to enable prft on ldash mode

2020-02-18 Thread Jeyapal, Karthick
On 2/18/20 9:43 PM, James Almer wrote: > Signed-off-by: James Almer > --- > libavformat/dashenc.c | 5 + > 1 file changed, 5 insertions(+) > > diff --git a/libavformat/dashenc.c b/libavformat/dashenc.c > index b910cc22d0..045d2f4df6 100644 > --- a/libavformat/dashenc.c > +++

Re: [FFmpeg-devel] [PATCH] crash fixed: live stream.

2020-02-18 Thread mostafa namazi
Hello Andriyyou can see my patch in below link. please review that. thanks. [FFmpeg-devel] new patch - Patchwork | | | | [FFmpeg-devel] new patch - Patchwork | | | On Wednesday, February 19, 2020, 1:01:21 AM GMT+3:30, Andriy Gelman wrote: On Tue, 18. Feb 18:52, Mostafa

Re: [FFmpeg-devel] [PATCH] configure: unmark FDKAAC as nonfree

2020-02-18 Thread Jean-Baptiste Kempf
On Wed, Feb 19, 2020, at 01:46, rcombs wrote: > Red Hat legal believes the FDK license to be free when using their stripped > version (https://github.com/UnitedRPMs/fdk-aac-free/releases), as the patent > clause is a no-op now that the relevant patents on that version are expired: >

Re: [FFmpeg-devel] [PATCH 2/3] libavcodec/libx264.c: refine code to extract function x264_encode_set_roi

2020-02-18 Thread Guo, Yejun
> -Original Message- > From: Guo, Yejun > Sent: Friday, February 14, 2020 11:32 AM > To: ffmpeg-devel@ffmpeg.org > Cc: Guo, Yejun > Subject: [PATCH 2/3] libavcodec/libx264.c: refine code to extract function > x264_encode_set_roi > > Signed-off-by: Guo, Yejun > --- >

Re: [FFmpeg-devel] [PATCH 2/3] lavf/hlsdec: use public avformat_flush function

2020-02-18 Thread Steven Liu
> 2020年2月19日 上午8:21,rcombs 写道: > > --- > libavformat/hls.c | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) > > diff --git a/libavformat/hls.c b/libavformat/hls.c > index 1f58e745a7..8181a631b6 100644 > --- a/libavformat/hls.c > +++ b/libavformat/hls.c > @@ -2295,7 +2295,7 @@ static int

Re: [FFmpeg-devel] Setting default G0 character set for teletext decoder

2020-02-18 Thread Carl Eugen Hoyos
> Am 18.02.2020 um 15:23 schrieb k.savkov : > > Some providers don't send info about character set, so default (latin) is > used when decoding. I added option to force desired language with function > vbi_teletext_set_default_region. Please remove the re-indentation from your patch (assuming

Re: [FFmpeg-devel] [PATCH] configure: unmark FDKAAC as nonfree

2020-02-18 Thread Carl Eugen Hoyos
> Am 19.02.2020 um 03:00 schrieb Carl Eugen Hoyos : > > PS: This could be unrelated but please note that plex was or is a violator of > the copyrights of the FFmpeg developers. As reported by you... ___ ffmpeg-devel mailing list

Re: [FFmpeg-devel] [PATCH] configure: unmark FDKAAC as nonfree

2020-02-18 Thread Carl Eugen Hoyos
> Am 19.02.2020 um 01:46 schrieb rcombs : > > Plex legal believes that the license is GPL-compatible in general Instead please get a comment from the fsf. I object to this patch for the time being. Carl Eugen PS: This could be unrelated but please note that plex was or is a violator of the

[FFmpeg-devel] [PATCH] configure: unmark FDKAAC as nonfree

2020-02-18 Thread rcombs
Red Hat legal believes the FDK license to be free when using their stripped version (https://github.com/UnitedRPMs/fdk-aac-free/releases), as the patent clause is a no-op now that the relevant patents on that version are expired: https://bugzilla.redhat.com/show_bug.cgi?id=1501522 Plex legal

Re: [FFmpeg-devel] [PATCH 3/3] lavf/matroskadec: add read_flush function

2020-02-18 Thread Andreas Rheinhardt
rcombs: > --- > libavformat/matroskadec.c | 23 +-- > 1 file changed, 17 insertions(+), 6 deletions(-) > > diff --git a/libavformat/matroskadec.c b/libavformat/matroskadec.c > index 4d7fdab99f..68e424992a 100644 > --- a/libavformat/matroskadec.c > +++

Re: [FFmpeg-devel] [PATCH 1/3] lavf: add read_flush function to AVInputFormat vtable

2020-02-18 Thread James Almer
On 2/18/2020 9:21 PM, rcombs wrote: > --- > libavformat/avformat.h | 6 ++ > libavformat/utils.c| 4 > libavformat/version.h | 4 ++-- > 3 files changed, 12 insertions(+), 2 deletions(-) > > diff --git a/libavformat/avformat.h b/libavformat/avformat.h > index 9b9b634ec3..1412119342

[FFmpeg-devel] [PATCH 2/3] lavf/hlsdec: use public avformat_flush function

2020-02-18 Thread rcombs
--- libavformat/hls.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/libavformat/hls.c b/libavformat/hls.c index 1f58e745a7..8181a631b6 100644 --- a/libavformat/hls.c +++ b/libavformat/hls.c @@ -2295,7 +2295,7 @@ static int hls_read_seek(AVFormatContext *s, int stream_index,

[FFmpeg-devel] [PATCH 3/3] lavf/matroskadec: add read_flush function

2020-02-18 Thread rcombs
--- libavformat/matroskadec.c | 23 +-- 1 file changed, 17 insertions(+), 6 deletions(-) diff --git a/libavformat/matroskadec.c b/libavformat/matroskadec.c index 4d7fdab99f..68e424992a 100644 --- a/libavformat/matroskadec.c +++ b/libavformat/matroskadec.c @@ -3703,6 +3703,21

[FFmpeg-devel] [PATCH 1/3] lavf: add read_flush function to AVInputFormat vtable

2020-02-18 Thread rcombs
--- libavformat/avformat.h | 6 ++ libavformat/utils.c| 4 libavformat/version.h | 4 ++-- 3 files changed, 12 insertions(+), 2 deletions(-) diff --git a/libavformat/avformat.h b/libavformat/avformat.h index 9b9b634ec3..1412119342 100644 --- a/libavformat/avformat.h +++

[FFmpeg-devel] [PATCH 2/4] avcodec/txd: Check for input size against the header size.

2020-02-18 Thread Michael Niedermayer
Fixes: Timeout (21sec -> 80ms) Fixes: 20673/clusterfuzz-testcase-minimized-ffmpeg_AV_CODEC_ID_TXD_fuzzer-5177453863763968 Found-by: continuous fuzzing process https://github.com/google/oss-fuzz/tree/master/projects/ffmpeg Signed-off-by: Michael Niedermayer --- libavcodec/txd.c | 3 +++ 1 file

[FFmpeg-devel] [PATCH 4/4] libavcodec/svq: Remove ff_svq1_packet_checksum()

2020-02-18 Thread Michael Niedermayer
Signed-off-by: Michael Niedermayer --- libavcodec/Makefile | 4 +-- libavcodec/svq1.h| 3 -- libavcodec/svq13.c | 69 libavcodec/svq1dec.c | 6 ++-- libavcodec/svq3.c| 5 +++- 5 files changed, 9 insertions(+), 78 deletions(-) delete

[FFmpeg-devel] [PATCH 1/4] avcodec/qdm2: Check fft_coefs_index

2020-02-18 Thread Michael Niedermayer
Fixes: out of array access Fixes: 20660/clusterfuzz-testcase-minimized-ffmpeg_AV_CODEC_ID_QDM2_fuzzer-5658290216501248 Found-by: continuous fuzzing process https://github.com/google/oss-fuzz/tree/master/projects/ffmpeg Signed-off-by: Michael Niedermayer --- libavcodec/qdm2.c | 3 +++ 1 file

[FFmpeg-devel] [PATCH 3/4] avcodec/svq1dec: Check that there is data left after the hader

2020-02-18 Thread Michael Niedermayer
Fixes: Timeout (21sec -> 255ms) Fixes: 20709/clusterfuzz-testcase-minimized-ffmpeg_AV_CODEC_ID_SVQ1_fuzzer-5085075089915904 Found-by: continuous fuzzing process https://github.com/google/oss-fuzz/tree/master/projects/ffmpeg Signed-off-by: Michael Niedermayer --- libavcodec/svq1dec.c | 2 ++ 1

[FFmpeg-devel] FFmpeg dev work

2020-02-18 Thread Ryan Gilligan
Hi, I'm looking for a quote for some dev work involving FFmpeg. If anyone thinks they can help with this, please let me know. Happy to connect you with my lead developer to clarify anything and provide more details. Looking for the following: 1. Turning FFmpeg into a DLL that can run on

Re: [FFmpeg-devel] build/fate fail

2020-02-18 Thread Lou Logan
On Tue, Feb 18, 2020, at 2:00 PM, Patchwork wrote: > Hello, > > Thank you for submitting a patch to ffmpeg-devel. > > An error occurred during an automated build/fate test. Please review > the following link for more details: >

Re: [FFmpeg-devel] build/fate fail

2020-02-18 Thread Patchwork
Hello, Thank you for submitting a patch to ffmpeg-devel. An error occurred during an automated build/fate test. Please review the following link for more details: https://patchwork.ffmpeg.org/project/ffmpeg/patch/20200218224926.23103-1-andriy.gel...@gmail.com/ Thank you, ffmpeg-devel

[FFmpeg-devel] [PATCH] donotapply: Test automated email for build fail

2020-02-18 Thread Andriy Gelman
From: Andriy Gelman --- libavformat/libzmq.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/libavformat/libzmq.c b/libavformat/libzmq.c index 8c8b294c921..7d9b13ad64c 100644 --- a/libavformat/libzmq.c +++ b/libavformat/libzmq.c @@ -159,7 +159,7 @@ static int

Re: [FFmpeg-devel] [PATCH 001/244] Add a new channel layout API

2020-02-18 Thread Marton Balint
On Tue, 18 Feb 2020, Anton Khirnov wrote: Quoting Marton Balint (2020-02-05 19:55:24) On Tue, 7 Jan 2020, Anton Khirnov wrote: > Quoting Nicolas George (2019-12-31 16:17:49) >> Anton Khirnov (12019-12-29): >> > I do not agree. Duplicated channels in a layout are expected to be a >> >

[FFmpeg-devel] build/fate fail

2020-02-18 Thread ffmpeg . patchworks
Hello, Thank you for submitting a patch to ffmpeg-devel. An error occurred during an automated build/fate test. Please review the following link for more details: https://patchwork.ffmpeg.org/project/ffmpeg/patch/cagsmwna53lfupmnavgiax9waoysvhcyacsqi7mfcrvcgszy...@mail.gmail.com/ Thank you,

Re: [FFmpeg-devel] [PATCH] crash fixed: live stream.

2020-02-18 Thread Andriy Gelman
On Tue, 18. Feb 18:52, Mostafa Namazi fard wrote: > --- > libavformat/hls.c | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) > > diff --git a/libavformat/hls.c b/libavformat/hls.c > index 1f58e745a7..97b1a1db52 100644 > --- a/libavformat/hls.c > +++ b/libavformat/hls.c > @@ -2110,7

Re: [FFmpeg-devel] [PATCH 001/244] Add a new channel layout API

2020-02-18 Thread Anton Khirnov
Quoting Marton Balint (2020-02-05 19:55:24) > > > On Tue, 7 Jan 2020, Anton Khirnov wrote: > > > Quoting Nicolas George (2019-12-31 16:17:49) > >> Anton Khirnov (12019-12-29): > > >> > I do not agree. Duplicated channels in a layout are expected to be a > >> > fringe thing and how you handle

Re: [FFmpeg-devel] Setting default G0 character set for teletext decoder

2020-02-18 Thread Marton Balint
On Tue, 18 Feb 2020, k.savkov wrote: Some providers don't send info about character set, so default (latin) is used when decoding. I added option to force desired language with function vbi_teletext_set_default_region. You can see what character set code maps to what language in Table 32

Re: [FFmpeg-devel] dvbsub.c, change segment order to be in line with spec

2020-02-18 Thread Andriy Gelman
On Tue, 18. Feb 11:27, Alen Vrečko wrote: > On https://patchwork.ffmpeg.org/project/ffmpeg/list/ the new patch doesn't > show up. I used the same filename as before. > > Just in case sending it again with changed filename (and also changed commit > message). > Hello Alen, New patches are

Re: [FFmpeg-devel] [PATCH] Update HDR10+ metadata structure.

2020-02-18 Thread Mohammad Izadi
Hello, What's the status of the cl review? Is it good to submit? -- Best, Mohammad On Mon, Feb 10, 2020 at 11:44 AM Mohammad Izadi wrote: > From: Mohammad Izadi > > Trying to read HDR10+ metadata from HEVC/SEI and pass it to packet side > data in the follow-up CLs. > --- >

Re: [FFmpeg-devel] [PATCH 1/6] avformat/format: add av_demuxer_find_by_ext

2020-02-18 Thread Andreas Rheinhardt
Anton Khirnov: > Quoting Andreas Rheinhardt (2020-02-18 17:52:00) >> Anton Khirnov: >>> Quoting Andreas Rheinhardt (2020-02-18 16:01:00) Anton Khirnov: > Quoting Gyan Doshi (2020-02-01 06:15:30) >> >> >> On 31-01-2020 10:41 pm, Andreas Rheinhardt wrote: >>> Gyan Doshi:

Re: [FFmpeg-devel] [PATCH 1/6] avformat/format: add av_demuxer_find_by_ext

2020-02-18 Thread Anton Khirnov
Quoting Andreas Rheinhardt (2020-02-18 17:52:00) > Anton Khirnov: > > Quoting Andreas Rheinhardt (2020-02-18 16:01:00) > >> Anton Khirnov: > >>> Quoting Gyan Doshi (2020-02-01 06:15:30) > > > On 31-01-2020 10:41 pm, Andreas Rheinhardt wrote: > > Gyan Doshi: > >> Allows

Re: [FFmpeg-devel] [PATCH 1/6] avformat/format: add av_demuxer_find_by_ext

2020-02-18 Thread Gyan Doshi
On 18-02-2020 10:22 pm, Andreas Rheinhardt wrote: Anton Khirnov: Quoting Andreas Rheinhardt (2020-02-18 16:01:00) Anton Khirnov: Quoting Gyan Doshi (2020-02-01 06:15:30) On 31-01-2020 10:41 pm, Andreas Rheinhardt wrote: Gyan Doshi: Allows selecting demuxer by extension which are more

Re: [FFmpeg-devel] [PATCH 1/6] avformat/format: add av_demuxer_find_by_ext

2020-02-18 Thread Andreas Rheinhardt
Anton Khirnov: > Quoting Andreas Rheinhardt (2020-02-18 16:01:00) >> Anton Khirnov: >>> Quoting Gyan Doshi (2020-02-01 06:15:30) On 31-01-2020 10:41 pm, Andreas Rheinhardt wrote: > Gyan Doshi: >> Allows selecting demuxer by extension which are more widely recognized >>

Re: [FFmpeg-devel] [PATCH, v2 2/4] avc/avcodec: add AV_CODEC_CAP_VARIABLE_DIMENSIONS flag

2020-02-18 Thread Anton Khirnov
Quoting Fu, Linjie (2020-02-18 16:06:10) > > Is there even a sufficiently strong use case for this? Why not just > > Like transcoding reinit-large_420_8-to-small_420_8.h264 (from 352x288 to > 240x196) > and prefer to keep the resolution changing in the output stream. > > IMHO not modifying the

[FFmpeg-devel] [PATCH 3/3] avformat/dashenc: always attempt to enable prft on ldash mode

2020-02-18 Thread James Almer
Signed-off-by: James Almer --- libavformat/dashenc.c | 5 + 1 file changed, 5 insertions(+) diff --git a/libavformat/dashenc.c b/libavformat/dashenc.c index b910cc22d0..045d2f4df6 100644 --- a/libavformat/dashenc.c +++ b/libavformat/dashenc.c @@ -1395,6 +1395,11 @@ static int

Re: [FFmpeg-devel] [PATCH 1/6] avformat/format: add av_demuxer_find_by_ext

2020-02-18 Thread Anton Khirnov
Quoting Andreas Rheinhardt (2020-02-18 16:01:00) > Anton Khirnov: > > Quoting Gyan Doshi (2020-02-01 06:15:30) > >> > >> > >> On 31-01-2020 10:41 pm, Andreas Rheinhardt wrote: > >>> Gyan Doshi: > Allows selecting demuxer by extension which are more widely recognized > by users. > >

[FFmpeg-devel] [PATCH 2/3] avformat/dashenc: write a capture time Producer Reference Time element when none is provided by the encoder

2020-02-18 Thread James Almer
This way, the element will be present in any scenario when the write_prft option is used. Signed-off-by: James Almer --- libavformat/dashenc.c | 46 +-- 1 file changed, 36 insertions(+), 10 deletions(-) diff --git a/libavformat/dashenc.c

[FFmpeg-devel] [PATCH 1/3] avformat/dashenc: write the styp box when the first frame of a segment is ready

2020-02-18 Thread James Almer
This ensures it's written at the beginning of a segment in non streaming mode when segment duration differs from fragment duration. Signed-off-by: James Almer --- libavformat/dashenc.c | 10 +++--- 1 file changed, 3 insertions(+), 7 deletions(-) diff --git a/libavformat/dashenc.c

[FFmpeg-devel] new patch

2020-02-18 Thread Mostafa Namazi fard
I found this patch useful for prevent crash in http live stream. I'm using ffmpeg in my application for show http live when remote address send invalid data ffmpeg crash in this part and after my change it was fixed. From 73c61670d6e65ad9f7a354a785f73ff9ad5ffc12 Mon Sep 17 00:00:00 2001 From:

Re: [FFmpeg-devel] [PATCH 1/6] avformat/format: add av_demuxer_find_by_ext

2020-02-18 Thread Andreas Rheinhardt
Anton Khirnov: > Quoting Gyan Doshi (2020-02-01 06:15:30) >> >> >> On 31-01-2020 10:41 pm, Andreas Rheinhardt wrote: >>> Gyan Doshi: Allows selecting demuxer by extension which are more widely recognized by users. Conditional cast added since this function will usually be

[FFmpeg-devel] [PATCH] crash fixed: live stream.

2020-02-18 Thread Mostafa Namazi fard
--- libavformat/hls.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/libavformat/hls.c b/libavformat/hls.c index 1f58e745a7..97b1a1db52 100644 --- a/libavformat/hls.c +++ b/libavformat/hls.c @@ -2110,7 +2110,7 @@ static int hls_read_packet(AVFormatContext *s, AVPacket *pkt)

Re: [FFmpeg-devel] [PATCH, v2 2/4] avc/avcodec: add AV_CODEC_CAP_VARIABLE_DIMENSIONS flag

2020-02-18 Thread Fu, Linjie
> -Original Message- > From: ffmpeg-devel On Behalf Of > Anton Khirnov > Sent: Tuesday, February 18, 2020 21:32 > To: FFmpeg development discussions and patches de...@ffmpeg.org> > Subject: Re: [FFmpeg-devel] [PATCH, v2 2/4] avc/avcodec: add > AV_CODEC_CAP_VARIABLE_DIMENSIONS flag > >

Re: [FFmpeg-devel] [PATCH 1/6] avformat/format: add av_demuxer_find_by_ext

2020-02-18 Thread Anton Khirnov
Quoting Gyan Doshi (2020-02-01 06:15:30) > > > On 31-01-2020 10:41 pm, Andreas Rheinhardt wrote: > > Gyan Doshi: > >> Allows selecting demuxer by extension which are more widely recognized > >> by users. > >> > >> Conditional cast added since this function will usually be called after > >>

[FFmpeg-devel] Setting default G0 character set for teletext decoder

2020-02-18 Thread k.savkov
Some providers don't send info about character set, so default (latin) is used when decoding. I added option to force desired language with function vbi_teletext_set_default_region. You can see what character set code maps to what language in Table 32 ETS 300 706,  Section 15. >From

Re: [FFmpeg-devel] [PATCH] hwcontext_vaapi: Only accept a render node when deriving from DRM device

2020-02-18 Thread Anton Khirnov
Quoting Mark Thompson (2020-02-16 21:59:54) > If we are given a non-render node, try to find the matching render node and > fail if that isn't possible. > > libva will not accept a non-render device which is not DRM master, because > it requires legacy DRM authentication to succeed in that case:

Re: [FFmpeg-devel] [PATCH, v2 2/4] avc/avcodec: add AV_CODEC_CAP_VARIABLE_DIMENSIONS flag

2020-02-18 Thread Anton Khirnov
Quoting Michael Niedermayer (2020-02-17 23:38:31) > On Mon, Feb 17, 2020 at 06:28:23PM +, Fu, Linjie wrote: > > > > > > https://patchwork.ffmpeg.org/patch/14122/ > > iam hesitating because it feeds encoders with changing resolution resulting > in potentially undefined behavior ... > > > >

Re: [FFmpeg-devel] [PATCH 1/3] avutil/frame: add use_last_roi

2020-02-18 Thread Anton Khirnov
Quoting Guo, Yejun (2020-02-14 04:32:05) > For some cases, the regions of interest do not change, it is not > convenient to always prepare the roi data for every frame. So, add > use_last_roi to show it uses the same roi data as last frame. > > Since a new flag is added into AVFrame, the major

Re: [FFmpeg-devel] [PATCH 1/2] avformat/wtvdec: Forward errors when reading packet

2020-02-18 Thread Andreas Rheinhardt
Andreas Rheinhardt: > wtvfile_read_packet did not abide by the requirements of an > AVIOContext's read_packet-function: If it did not read anything, > it returned zero, which currently leads to a warning in read_packet_wrapper > in aviobuf.c. Said warning will be an av_assert2 as soon as >

[FFmpeg-devel] [PATCH v2 2/2] avformat: add demuxer for Rayman 2's APM format

2020-02-18 Thread Zane van Iperen
Adds support for the APM file format used by Ubisoft's Rayman 2. Signed-off-by: Zane van Iperen --- libavformat/Makefile | 1 + libavformat/allformats.c | 1 + libavformat/apm.c| 188 +++ libavformat/version.h| 4 +- 4 files changed, 192

[FFmpeg-devel] [PATCH v2 1/2] avcodec: add decoder for Rayman 2's ADPCM variant

2020-02-18 Thread Zane van Iperen
Adds support for the ADPCM variant used in Rayman 2's files. Signed-off-by: Zane van Iperen --- libavcodec/Makefile | 1 + libavcodec/adpcm.c | 24 libavcodec/allcodecs.c | 1 + libavcodec/avcodec.h| 1 + libavcodec/codec_desc.c | 7 +++

[FFmpeg-devel] [PATCH v2 0/2] Ubisoft Rayman 2 APM demuxer + decoder.

2020-02-18 Thread Zane van Iperen
Hi all, This patchset adds support for the APM files used by Rayman 2. It has been tested against all *.apm files in the game folder. v2: - Change extradata to use AV_{W,R}L32 instead of AV_{W,R}N32 - fix version - add probe function - removed an unnecessary `n = 0` Zane Zane van

Re: [FFmpeg-devel] dvbsub.c, change segment order to be in line with spec

2020-02-18 Thread Alen Vrečko
On https://patchwork.ffmpeg.org/project/ffmpeg/list/ the new patch doesn't show up. I used the same filename as before. Just in case sending it again with changed filename (and also changed commit message). Sorry for the extra message. Alen - Original Message - From: "Alen Vrečko"

Re: [FFmpeg-devel] dvbsub.c, change segment order to be in line with spec

2020-02-18 Thread Alen Vrečko
Sure thing. I attached a new patch. Thank you. Alen - Original Message - From: "Carl Eugen Hoyos" To: "ffmpeg-devel" Sent: Monday, February 17, 2020 6:28:17 PM Subject: Re: [FFmpeg-devel] dvbsub.c, change segment order to be in line with spec > Am 17.02.2020 um 16:10 schrieb Alen

Re: [FFmpeg-devel] [PATCH] ansi: process ESC[3m italics attribute

2020-02-18 Thread Paul B Mahol
lgtm On 2/18/20, Peter Ross wrote: > squelch unknown escape code warnings > --- > libavcodec/ansi.c | 3 ++- > 1 file changed, 2 insertions(+), 1 deletion(-) > > diff --git a/libavcodec/ansi.c b/libavcodec/ansi.c > index 5e1035ffd0..516d07db69 100644 > --- a/libavcodec/ansi.c > +++