Re: [FFmpeg-devel] order T-shirts

On Sat, Oct 28, 2017, at 03:41 AM, Thilo Borgmann wrote: > Both designs come for 12€ each! Lou's design doesn't have a printing on > the neck. I'd choosen the green for our logo by their color chart - it > looks good color-wise to me. Looks good to me too. Thanks for all of your work on this.

Re: [FFmpeg-devel] [PATCH 1/5] lavc/avrndec: remove AV_CODEC_CAP_DR1, as it's broken

On 10/13/2017 1:59 PM, wm4 wrote: > In the is_mjpeg case, the user's get_buffer2 callback is not called, > thus completely breaking the API. > --- > libavcodec/avrndec.c | 1 - > 1 file changed, 1 deletion(-) > > diff --git a/libavcodec/avrndec.c b/libavcodec/avrndec.c > index

Re: [FFmpeg-devel] [PATCH 2/5] decode: avoid leaks on failure in ff_get_buffer()

On 10/13/2017 1:59 PM, wm4 wrote: > From: Anton Khirnov > > If the get_buffer() call fails, the frame might have some side data > already set. Make sure it gets freed. > > CC: libav-sta...@libav.org > > Merges Libav commit de77671438c24ffea93398c8dc885d4dd04477de. > --- >

Re: [FFmpeg-devel] [PATCH] libavformat/mov.c: use calculated dts offset when seeking in streams

On Sun, Oct 29, 2017 at 12:11:39PM +0100, Peter Große wrote: > From: Jonas Licht > > Subtract the calculated dts offset from the requested timestamp before > seeking. This fixes an error "Error while filtering: Operation not > permitted" observed with a short file

Re: [FFmpeg-devel] [PATCH] movenc: add m4b to ipod format extensions

On Sun, Oct 29, 2017 at 11:35:57AM +0100, Lukas Stabe wrote: > m4b is commonly used as extension for m4a audiobook files. > The format is exactly the same. The only thing that differs > is the extension. > --- > libavformat/movenc.c | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) will

Re: [FFmpeg-devel] checkasm/llviddsp : add tests for other func of the dsp

On Sun, Oct 29, 2017 at 06:07:09PM +0100, Martin Vignali wrote: > Hello, > > in attach patch to add a checkasm test for the uncover func of the dsp > > add_median_pred > > add_left_pred > for this one i add two tests : one with acc = 0 (used in several decoders) > and one with random acc > >

Re: [FFmpeg-devel] Added - HW accelerated H.264 and HEVC encoding for AMD GPUs based on AMF SDK

> -Original Message- > From: ffmpeg-devel [mailto:ffmpeg-devel-boun...@ffmpeg.org] On Behalf > Of Mark Thompson > Sent: October 29, 2017 5:51 PM > To: ffmpeg-devel@ffmpeg.org > Subject: Re: [FFmpeg-devel] Added - HW accelerated H.264 and HEVC > encoding for AMD GPUs based on AMF SDK > >

[FFmpeg-devel] [PATCH v5 1/2 ] avformat/mpegts: opus muxing for mapping family 255

Le 29/10/2017 à 4:44 PM, Michael Niedermayer a écrit : On Sat, Oct 28, 2017 at 03:49:13AM +0200, pkv.stream wrote: mpegtsenc.c | 67 +--- 1 file changed, 60 insertions(+), 7 deletions(-) 1423a193788547952e3c4cdcb424b4724b0f1157

Re: [FFmpeg-devel] [PATCH]fate: Add bitexact and idct simple to the mov-invalid-edst test

2017-10-29 22:48 GMT+01:00 Sasi Inguva : > looks good to me. Thanks for the fix. Patch applied. Thank you, Carl Eugen ___ ffmpeg-devel mailing list ffmpeg-devel@ffmpeg.org http://ffmpeg.org/mailman/listinfo/ffmpeg-devel

[FFmpeg-devel] [PATCH v5 2/2 ] avformat/mpegts: opus demuxing for mapping family 255

minor change, adding a test for number of channels for mapping_family 1 when channel_config_code is > 0x82 Passes fate + patcheck. Regards pkv From 2e0504728b81e0f35ba856c24cff14fa70680d8b Mon Sep 17 00:00:00 2001 From: pkviet Date: Sun, 29 Oct 2017 22:57:25 +0100 Subject:

Re: [FFmpeg-devel] [PATCH]fate: Add bitexact and idct simple to the mov-invalid-edst test

looks good to me. Thanks for the fix. On Sun, Oct 29, 2017 at 2:11 PM, Carl Eugen Hoyos wrote: > 2017-10-29 22:03 GMT+01:00 Carl Eugen Hoyos : > > Hi! > > > > Attached patch fixes a fate test here on a ppc system. > > The following is sufficient: -flags

Re: [FFmpeg-devel] Added - HW accelerated H.264 and HEVC encoding for AMD GPUs based on AMF SDK

On 29/10/17 20:48, Mironov, Mikhail wrote: >> -Original Message- >> From: ffmpeg-devel [mailto:ffmpeg-devel-boun...@ffmpeg.org] On Behalf >> Of Mark Thompson >> Sent: October 29, 2017 3:36 PM >> To: ffmpeg-devel@ffmpeg.org >> Subject: Re: [FFmpeg-devel] Added - HW accelerated H.264 and

Re: [FFmpeg-devel] [PATCH] avfilter/zscale: add explanation for error 3074

> Original Message >Subject: Re: [FFmpeg-devel] [PATCH] avfilter/zscale: add explanation for error >3074 >Local Time: October 29, 2017 6:50 AM >UTC Time: October 29, 2017 1:50 PM >From: barsn...@gmx.net >To: FFmpeg development discussions and patches >

Re: [FFmpeg-devel] [PATCH]fate: Add bitexact and idct simple to the mov-invalid-edst test

2017-10-29 22:03 GMT+01:00 Carl Eugen Hoyos : > Hi! > > Attached patch fixes a fate test here on a ppc system. The following is sufficient: -flags +bitexact The idct option is not necessary. Carl Eugen ___ ffmpeg-devel mailing list

Re: [FFmpeg-devel] Added - HW accelerated H.264 and HEVC encoding for AMD GPUs based on AMF SDK

> -Original Message- > From: ffmpeg-devel [mailto:ffmpeg-devel-boun...@ffmpeg.org] On Behalf > Of Carl Eugen Hoyos > Sent: October 29, 2017 5:01 PM > To: FFmpeg development discussions and patches de...@ffmpeg.org> > Subject: Re: [FFmpeg-devel] Added - HW accelerated H.264 and HEVC >

[FFmpeg-devel] [PATCH]fate: Add bitexact and idct simple to the mov-invalid-edst test

Hi! Attached patch fixes a fate test here on a ppc system. Please comment, Carl Eugen From 35a19a80efe319bc99c2efb8a8b79c5e865ab47f Mon Sep 17 00:00:00 2001 From: Carl Eugen Hoyos Date: Sun, 29 Oct 2017 22:01:40 +0100 Subject: [PATCH] fate: Add bitexact and idct simple to

Re: [FFmpeg-devel] Added - HW accelerated H.264 and HEVC encoding for AMD GPUs based on AMF SDK

2017-10-29 21:48 GMT+01:00 Mironov, Mikhail : > VAAPI on Linux is available in open source driver. AMF version will be > implemented via > Vulkan and will follow Vulkan implementation in the driver and in open > source policy. Does that mean that at least in theory,

Re: [FFmpeg-devel] Added - HW accelerated H.264 and HEVC encoding for AMD GPUs based on AMF SDK

> -Original Message- > From: ffmpeg-devel [mailto:ffmpeg-devel-boun...@ffmpeg.org] On Behalf > Of Mark Thompson > Sent: October 29, 2017 3:49 PM > To: ffmpeg-devel@ffmpeg.org > Subject: Re: [FFmpeg-devel] Added - HW accelerated H.264 and HEVC > encoding for AMD GPUs based on AMF SDK > >

Re: [FFmpeg-devel] Added - HW accelerated H.264 and HEVC encoding for AMD GPUs based on AMF SDK

> -Original Message- > From: ffmpeg-devel [mailto:ffmpeg-devel-boun...@ffmpeg.org] On Behalf > Of Mark Thompson > Sent: October 29, 2017 3:36 PM > To: ffmpeg-devel@ffmpeg.org > Subject: Re: [FFmpeg-devel] Added - HW accelerated H.264 and HEVC > encoding for AMD GPUs based on AMF SDK > >

Re: [FFmpeg-devel] [PATCH] ffmpeg.c: fix calculation of input file duration in seek_to_start()

On Sun, 29 Oct 2017 21:33:10 +0100 Michael Niedermayer wrote: > seems this doesnt apply > Yeah, it depends on the patch I started the thread with. I can swap the order of the two patches if you want, so that the style fix can go in first. But they will depend on each

Re: [FFmpeg-devel] [PATCH] ffmpeg.c: fix calculation of input file duration in seek_to_start()

On Sun, Oct 29, 2017 at 03:09:13PM +0100, Peter Große wrote: > On Sun, 29 Oct 2017 14:56:20 +0100 > Moritz Barsnick wrote: > > > On Sun, Oct 29, 2017 at 12:08:05 +0100, Peter Große wrote: > > > } else duration = 1; > > > > For style/readability, the statement

Re: [FFmpeg-devel] [PATCH] swscale: Reduce verbosity of misalignment reporting

On Sun, Oct 22, 2017 at 11:47:55AM +0200, Henrik Gramner wrote: > It's a bit overzealous to complain about misalignment with AV_LOG_WARNING, > especially since memory bandwidth is much more likely to be the bottleneck > compared to data alignment which the user may not even have control over. >

Re: [FFmpeg-devel] [PATCH] fate/fits: add new test & gen few samples automatically

2017-08-27 7:45 GMT+02:00 Paras Chadha : > Signed-off-by: Paras Chadha > --- > All other FITS patches need to be applied before this one. > > tests/fate/demux.mak| 3 +- > tests/fate/fits.mak | 57 >

Re: [FFmpeg-devel] Added - HW accelerated H.264 and HEVC encoding for AMD GPUs based on AMF SDK

On 29/10/17 14:54, Mironov, Mikhail wrote: >> -Original Message- >> From: ffmpeg-devel [mailto:ffmpeg-devel-boun...@ffmpeg.org] On Behalf >> Of Carl Eugen Hoyos >> Sent: October 28, 2017 3:19 PM >> To: FFmpeg development discussions and patches > de...@ffmpeg.org> >> Subject: Re:

Re: [FFmpeg-devel] Added - HW accelerated H.264 and HEVC encoding for AMD GPUs based on AMF SDK

On 29/10/17 18:39, Mironov, Mikhail wrote: >>> REGISTER_ENCODER(H263_V4L2M2M, h263_v4l2m2m); >>> REGISTER_ENCDEC (LIBOPENH264, libopenh264); >>> +REGISTER_ENCODER(H264_AMF, h264_amf); >>> + REGISTER_ENCODER(H264_AMF, h264_amf_d3d11va); >> >> No tabs.

[FFmpeg-devel] [PATCH] avcodec/avcodec.h: remove doxy from the old bsf API functions

Make it clear that these are deprecated and the new API should be used instead. As a side effect, this slightly reduces differences with libav. Signed-off-by: James Almer --- libavcodec/avcodec.h | 70 +++- 1 file changed, 14

Re: [FFmpeg-devel] [PATCH] avcodec/avcodec.h: remove doxy from the old bsf API functions

On 10/29/2017 3:49 PM, James Almer wrote: > Make it clear that these are deprecated and the new API should be > used instead. > > As a side effect, this reduces differences with libav. > > Signed-off-by: James Almer > --- > With this, existing users will get an extra

Re: [FFmpeg-devel] Added - HW accelerated H.264 and HEVC encoding for AMD GPUs based on AMF SDK

> > REGISTER_ENCODER(H263_V4L2M2M, h263_v4l2m2m); > > REGISTER_ENCDEC (LIBOPENH264, libopenh264); > > +REGISTER_ENCODER(H264_AMF, h264_amf); > > + REGISTER_ENCODER(H264_AMF, h264_amf_d3d11va); > > No tabs. Why is the d3d11 version separate? The

Re: [FFmpeg-devel] [PATCH 2/3] lavf/avio: temporarily accept 0 as EOF.

On Fri, Oct 27, 2017 at 9:56 PM, Jan Ekstrom wrote: > On Fri, Oct 27, 2017 at 9:46 PM, Nicolas George wrote: >> Print a warning to let applicatios fix their use. >> After a deprecation period, check with a low-level assert. >> Also make the constraint explicit

Re: [FFmpeg-devel] [PATCH 7/7] dashenc: copy stream frame rate to output stream

On Thu, Oct 26, 2017 at 05:58:40PM +0200, Peter Große wrote: > Leads to setting of DefaultDuration in Matroska muxer based on frame rate > instead of timebase. > Fixes playback in Chrome. > > Signed-off-by: Peter Große > --- > libavformat/dashenc.c | 1 + > 1 file changed, 1

Re: [FFmpeg-devel] [PATCH v2 6/7] dashenc: Don't output frameRates at both AS and Representation-level. Only output maxFrameRate at AS-level.

On Sun, Oct 29, 2017 at 04:10:35PM +0100, Peter Große wrote: > From: Anton Schubert > > Fix conformance regarding section "3.2.4. Presence of Attributes and > Element" of the "Guidelines for Implementation: DASH-IF Interoperability > Points V4.1"

Re: [FFmpeg-devel] [PATCH 5/7] dashenc: allow AdaptationSets to have different average frame rates

On Thu, Oct 26, 2017 at 06:17:31PM +0200, Peter Große wrote: > Signed-off-by: Peter Große > --- > libavformat/dashenc.c | 19 +-- > 1 file changed, 9 insertions(+), 10 deletions(-) will apply thanks [...] -- Michael GnuPG fingerprint:

Re: [FFmpeg-devel] [PATCH 4/7] dashenc: don't write header data before the first packet arrives

On Thu, Oct 26, 2017 at 05:58:37PM +0200, Peter Große wrote: > Fixes: 1b8ef01f04ab ("dashenc: add webm support") > Signed-off-by: Peter Große > --- > libavformat/dashenc.c | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) will apply thanks [...] -- Michael GnuPG

Re: [FFmpeg-devel] [PATCH RFC] libavdevice/decklink: Add support for EIA-708 output over SDI

> On Oct 25, 2017, at 2:23 PM, Marton Balint wrote: > > > On Fri, 6 Oct 2017, Devin Heitmueller wrote: > >> From: Devin Heitmueller > > >> >> Hook in libklvanc and use it for output of EIA-708 captions over >>

[FFmpeg-devel] checkasm/llviddsp : add tests for other func of the dsp

Hello, in attach patch to add a checkasm test for the uncover func of the dsp add_median_pred add_left_pred for this one i add two tests : one with acc = 0 (used in several decoders) and one with random acc add_left_pred16 Martin 0002-checkasm-llviddsp-add-test-for-other-dsp-func.patch

Re: [FFmpeg-devel] [PATCH v2 3/7] dashenc: set DASH related options for the subsequent matroska muxer when using webm

On Sun, Oct 29, 2017 at 04:10:36PM +0100, Peter Große wrote: > This patch is inspired by the ffmpeg webm_chunk muxer and fixes that all > resulting > tracks have the same track number. > > Signed-off-by: Peter Große > --- > v2: extended commit message > >

Re: [FFmpeg-devel] [PATCH v2 1/7] dashenc: fix bitrate estimation with correct scaling

On Sun, Oct 29, 2017 at 04:10:34PM +0100, Peter Große wrote: > From: Anton Schubert > > Signed-off-by: Anton Schubert > Signed-off-by: Peter Große > --- > v2: Fixed double rescaling. > > libavformat/dashenc.c | 3 ++- > 1 file

Re: [FFmpeg-devel] [PATCH]lavfi/palettegen: Allow setting the background colour

On Sun, Oct 29, 2017 at 01:47:03AM +0200, Carl Eugen Hoyos wrote: > 2017-10-28 23:50 GMT+02:00 Clément Bœsch : > > On Sat, Oct 28, 2017 at 10:57:32PM +0200, Carl Eugen Hoyos wrote: > >> 2017-10-17 23:42 GMT+02:00 Carl Eugen Hoyos : > >> > >> > Attached patch is

Re: [FFmpeg-devel] [PATCH 1/2] avcodec/jpeglsdec: Check ilv for being a supported value

On Thu, Oct 26, 2017 at 12:02:56AM +0200, Michael Niedermayer wrote: > Fixes: 1773/clusterfuzz-testcase-minimized-4832523987189760 > > Found-by: continuous fuzzing process > https://github.com/google/oss-fuzz/tree/master/projects/ffmpeg > Signed-off-by: Michael Niedermayer

Re: [FFmpeg-devel] fate/utvideo add test for decoding and encoding V2

On Sat, Oct 28, 2017 at 02:13:26PM +0200, Martin Vignali wrote: > Hello, > > New patch in attach, with more samples, in order to test most of the not > covered case > > Sample can be found here > https://we.tl/EV2lBmOXk0 > > And need to be put inside ./fate-suite/utvideo > > Decoding test add

Re: [FFmpeg-devel] Added - HW accelerated H.264 and HEVC encoding for AMD GPUs based on AMF SDK

2017-10-29 16:43 GMT+01:00 Mironov, Mikhail : >> >> > +const enum AVPixelFormat ff_amf_pix_fmts[] = { >> >> > +AV_PIX_FMT_NV12, >> >> >> >> > +AV_PIX_FMT_BGRA, >> >> > +AV_PIX_FMT_ARGB, >> >> > +AV_PIX_FMT_RGBA, >> >> >> >> This is wrong, your encoders do

[FFmpeg-devel] [PATCH] avcodec/avcodec.h: remove doxy from the old bsf API functions

Make it clear that these are deprecated and the new API should be used instead. As a side effect, this reduces differences with libav. Signed-off-by: James Almer --- With this, existing users will get an extra incentive to migrate, and potential new users will not start using

Re: [FFmpeg-devel] [PATCH v4 1/2 ] avformat/mpegts: opus muxing for mapping family 255

On Sat, Oct 28, 2017 at 03:49:13AM +0200, pkv.stream wrote: > > > mpegtsenc.c | 67 > +--- > 1 file changed, 60 insertions(+), 7 deletions(-) > 1423a193788547952e3c4cdcb424b4724b0f1157 >

Re: [FFmpeg-devel] [FFmpeg-cvslog] Merge commit '7cb1d9e2dbbe5bf4652be5d78cdd68e956fa3d63'

On 10/29/2017 12:04 PM, Carl Eugen Hoyos wrote: > 2017-10-12 2:22 GMT+02:00 Michael Niedermayer : >> On Wed, Oct 11, 2017 at 09:26:18PM +, James Almer wrote: >>> ffmpeg | branch: master | James Almer | Wed Oct 11 >>> 17:55:25 2017 -0300|

Re: [FFmpeg-devel] Added - HW accelerated H.264 and HEVC encoding for AMD GPUs based on AMF SDK

> -Original Message- > From: ffmpeg-devel [mailto:ffmpeg-devel-boun...@ffmpeg.org] On Behalf > Of Carl Eugen Hoyos > Sent: October 29, 2017 11:00 AM > To: FFmpeg development discussions and patches de...@ffmpeg.org> > Subject: Re: [FFmpeg-devel] Added - HW accelerated H.264 and HEVC >

[FFmpeg-devel] [PATCH v2 3/7] dashenc: set DASH related options for the subsequent matroska muxer when using webm

This patch is inspired by the ffmpeg webm_chunk muxer and fixes that all resulting tracks have the same track number. Signed-off-by: Peter Große --- v2: extended commit message libavformat/dashenc.c | 3 +++ 1 file changed, 3 insertions(+) diff --git a/libavformat/dashenc.c

[FFmpeg-devel] [PATCH v2 6/7] dashenc: Don't output frameRates at both AS and Representation-level. Only output maxFrameRate at AS-level.

From: Anton Schubert Fix conformance regarding section "3.2.4. Presence of Attributes and Element" of the "Guidelines for Implementation: DASH-IF Interoperability Points V4.1" (http://dashif.org/guidelines/) Signed-off-by: Anton Schubert

[FFmpeg-devel] [PATCH v2 1/7] dashenc: fix bitrate estimation with correct scaling

From: Anton Schubert Signed-off-by: Anton Schubert Signed-off-by: Peter Große --- v2: Fixed double rescaling. libavformat/dashenc.c | 3 ++- 1 file changed, 2 insertions(+), 1 deletion(-) diff --git a/libavformat/dashenc.c

Re: [FFmpeg-devel] fate/utvideo add test for decoding and encoding V2

On Sat, Oct 28, 2017 at 02:13:26PM +0200, Martin Vignali wrote: > Hello, > > New patch in attach, with more samples, in order to test most of the not > covered case > > Sample can be found here > https://we.tl/EV2lBmOXk0 uploaded thx [...] -- Michael GnuPG fingerprint:

Re: [FFmpeg-devel] [FFmpeg-cvslog] Merge commit '7cb1d9e2dbbe5bf4652be5d78cdd68e956fa3d63'

2017-10-12 2:22 GMT+02:00 Michael Niedermayer : > On Wed, Oct 11, 2017 at 09:26:18PM +, James Almer wrote: >> ffmpeg | branch: master | James Almer | Wed Oct 11 >> 17:55:25 2017 -0300| [6dfcbd80ad446ff163b47f2bf432bbf706436ea8] | committer: >>

Re: [FFmpeg-devel] Added - HW accelerated H.264 and HEVC encoding for AMD GPUs based on AMF SDK

2017-10-29 15:54 GMT+01:00 Mironov, Mikhail : >> -Original Message- >> From: ffmpeg-devel [mailto:ffmpeg-devel-boun...@ffmpeg.org] On Behalf >> Of Carl Eugen Hoyos >> Sent: October 28, 2017 3:19 PM >> To: FFmpeg development discussions and patches >

[FFmpeg-devel] [PATCH] tests/checkasm/float_dsp: Increase allowed difference for float_dsp.vector_dmul

The choosen value is the lowest power of 2 that allows 1000 iterations of fate-checkasm-float_dsp to pass on x86-32 Signed-off-by: Michael Niedermayer --- tests/checkasm/float_dsp.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git

Re: [FFmpeg-devel] Added - HW accelerated H.264 and HEVC encoding for AMD GPUs based on AMF SDK

> -Original Message- > From: ffmpeg-devel [mailto:ffmpeg-devel-boun...@ffmpeg.org] On Behalf > Of Mark Thompson > Sent: October 28, 2017 5:29 PM > To: ffmpeg-devel@ffmpeg.org > Subject: Re: [FFmpeg-devel] Added - HW accelerated H.264 and HEVC > encoding for AMD GPUs based on AMF SDK > >

Re: [FFmpeg-devel] Added - HW accelerated H.264 and HEVC encoding for AMD GPUs based on AMF SDK

> -Original Message- > From: ffmpeg-devel [mailto:ffmpeg-devel-boun...@ffmpeg.org] On Behalf > Of Carl Eugen Hoyos > Sent: October 28, 2017 3:19 PM > To: FFmpeg development discussions and patches de...@ffmpeg.org> > Subject: Re: [FFmpeg-devel] Added - HW accelerated H.264 and HEVC >

Re: [FFmpeg-devel] [PATCH]lavf/rtpenc: Add support for 24 bit pcm encoding

On 10/29/17, Carl Eugen Hoyos wrote: > 2017-10-24 23:10 GMT+02:00 Carl Eugen Hoyos : >> Hi! >> >> Attached patch allows sending pcm_s24be over rtp, defined in rfc 3190. > > Patch applied, Carl Eugen > ___ >

Re: [FFmpeg-devel] [PATCH]lavf/rtpenc: Add support for 24 bit pcm encoding

2017-10-24 23:10 GMT+02:00 Carl Eugen Hoyos : > Hi! > > Attached patch allows sending pcm_s24be over rtp, defined in rfc 3190. Patch applied, Carl Eugen ___ ffmpeg-devel mailing list ffmpeg-devel@ffmpeg.org

Re: [FFmpeg-devel] libavcodec/bswapdsp : add AVX2 for bswap_buf

On 2017-10-28 17:57, Martin Vignali wrote: >>> For this, i follow Henrik Gramner's code, in the discussion >>> libavcodec/exr : add x86 SIMD for predictor. >> >> That's fine. I will look at that on my own time. I don't want to hold >> up applying the patch. Would you like me to push it for

Re: [FFmpeg-devel] [PATCH] ffmpeg.c: fix calculation of input file duration in seek_to_start()

On Sun, 29 Oct 2017 14:56:20 +0100 Moritz Barsnick wrote: > On Sun, Oct 29, 2017 at 12:08:05 +0100, Peter Große wrote: > > } else duration = 1; > > For style/readability, the statement should be on a new line, perhaps > even with brackets. (Not necessarily as

Re: [FFmpeg-devel] [PATCH] lavf/mov.c: Fix parsing of edit list atoms with invalid elst entry count.

2017-10-29 13:39 GMT+01:00 Michael Niedermayer : > On Sat, Oct 28, 2017 at 08:26:16PM +0200, Michael Niedermayer wrote: >> On Thu, Oct 26, 2017 at 08:51:50PM -0700, Sasi Inguva wrote: >> > libavformat/mov.c | 21 +- >> >

Re: [FFmpeg-devel] [PATCH] ffmpeg.c: fix calculation of input file duration in seek_to_start()

On Sun, Oct 29, 2017 at 12:08:05 +0100, Peter Große wrote: > } else duration = 1; For style/readability, the statement should be on a new line, perhaps even with brackets. (Not necessarily as part of this patch though.) Moritz ___

Re: [FFmpeg-devel] [PATCH] avfilter/zscale: add explanation for error 3074

On Sat, Oct 28, 2017 at 20:54:02 -0400, Dxfhgwet wrote: -av_log(ctx, AV_LOG_ERROR, "code %d: %s\n", err_code, err_msg); +if (err_code == ZIMG_ERROR_NO_COLORSPACE_CONVERSION) { +av_log(ctx, AV_LOG_ERROR, "code %d: %s. Try specifying matrix/transfer/primaries.\n"; +} else { +

Re: [FFmpeg-devel] [PATCH] Fix signed integer overflow in mov_write_single_packet Detected with clang and -fsanitize=signed-integer-overflow

On Sat, Oct 28, 2017 at 21:36:10 -0700, Vitaly Buka wrote: > ping You didn't respond to Carl Eugen's review. Furthermore, the second line of your commit message ("Detected...") should be separated from the first with an empty line. Moritz ___

Re: [FFmpeg-devel] [PATCH] avutil/frame: deprecate getters and setters for AVFrame fields

On 10/28/2017 5:16 AM, Paul B Mahol wrote: > On 10/28/17, James Almer wrote: >> On 10/23/2017 11:13 AM, James Almer wrote: >>> The fields can be accessed directly, so these are not needed anymore. >>> >>> Signed-off-by: James Almer >>> --- >>>

Re: [FFmpeg-devel] [PATCH 1/2] avformat: move public AVStream fields up in the struct

On 10/28/2017 4:17 AM, Paul B Mahol wrote: > On 10/23/17, James Almer wrote: >> Remove the silly second notice. >> >> Signed-off-by: James Almer >> --- >> libavformat/avformat.h | 65 >> ++ >> 1 file changed,

Re: [FFmpeg-devel] [PATCH] lavf/mov.c: Fix parsing of edit list atoms with invalid elst entry count.

On Sat, Oct 28, 2017 at 08:26:16PM +0200, Michael Niedermayer wrote: > On Thu, Oct 26, 2017 at 08:51:50PM -0700, Sasi Inguva wrote: > > On Tue, Oct 24, 2017 at 2:48 PM, Michael Niedermayer > > wrote: > > > > > On Mon, Oct 23, 2017 at 04:18:28PM -0700, Sasi Inguva wrote:

[FFmpeg-devel] [PATCH] libavformat/mov.c: use calculated dts offset when seeking in streams

From: Jonas Licht Subtract the calculated dts offset from the requested timestamp before seeking. This fixes an error "Error while filtering: Operation not permitted" observed with a short file which contains only one key frame and starts with negative timestamps.

[FFmpeg-devel] [PATCH] ffmpeg.c: fix calculation of input file duration in seek_to_start()

Fixes looping files without audio or when using stream_copy, where ist->nb_samples is not set since no decoding is done. This fixes ticket #5719 and also fixes an endless loop with the sample in ticket #6139. Signed-off-by: Peter Große --- fftools/ffmpeg.c | 4 ++-- 1 file

[FFmpeg-devel] [PATCH] movenc: add m4b to ipod format extensions

m4b is commonly used as extension for m4a audiobook files. The format is exactly the same. The only thing that differs is the extension. --- libavformat/movenc.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/libavformat/movenc.c b/libavformat/movenc.c index

Re: [FFmpeg-devel] [PATCH] swscale: Reduce verbosity of misalignment reporting

On Sun, Oct 22, 2017 at 11:47 AM, Henrik Gramner wrote: > It's a bit overzealous to complain about misalignment with AV_LOG_WARNING, > especially since memory bandwidth is much more likely to be the bottleneck > compared to data alignment which the user may not even have

[FFmpeg-devel] [PATCH] lavf/img2enc: remove redundant option 'updatefirst'

'updatefirst' is an undocumented option redundant to older option 'update'. Since it doesn't serve as a shorthand, this patch removes it. Regards, Gyan From e845a4cc2d885db6bc27e3f69fbd7940b6131f52 Mon Sep 17 00:00:00 2001 From: Gyan Doshi Date: Sun, 29 Oct 2017 13:23:18