On 6/29/2016 9:47 PM, James Almer wrote:
> On 6/29/2016 9:16 PM, Carl Eugen Hoyos wrote:
>> Hendrik Leppkes gmail.com> writes:
>>
>>> Plenty previous releases moved fields that were marked as "no direct
>>> access", usually because libav added a new field above the
>>> ffmpeg-exclusive ones (like
On 6/29/2016 9:27 PM, Carl Eugen Hoyos wrote:
> Ronald S. Bultje gmail.com> writes:
>
>> For the record: there was some discussion related to this on IRC.
>
> Would you mind adding some relevant points here?
> I was under the impression that this mailing list
> is the main place for FFmpeg patc
Fixes ticket #5623
TODO: bump lavf minor
---
Changelog| 1 +
configure| 4 +
libavformat/Makefile | 1 +
libavformat/allformats.c | 1 +
libavformat/libopenmpt.c | 200 +++
5 files changed, 207 insertions(+)
On 6/29/2016 9:16 PM, Carl Eugen Hoyos wrote:
> Hendrik Leppkes gmail.com> writes:
>
>> Plenty previous releases moved fields that were marked as "no direct
>> access", usually because libav added a new field above the
>> ffmpeg-exclusive ones (like AVFrame.channels, which is a common
>> candidat
Hendrik Leppkes gmail.com> writes:
> Plenty previous releases moved fields that were marked as "no direct
> access", usually because libav added a new field above the
> ffmpeg-exclusive ones (like AVFrame.channels, which is a common
> candidate), and we didn't do such a song and dance to accomoda
Ronald S. Bultje gmail.com> writes:
> For the record: there was some discussion related to this on IRC.
Would you mind adding some relevant points here?
I was under the impression that this mailing list
is the main place for FFmpeg patch discussions, no?
Carl Eugen
___
Hi,
On Wed, Jun 29, 2016 at 8:13 PM, Carl Eugen Hoyos wrote:
> Michael Niedermayer niedermayer.cc> writes:
>
> > -We strongly recommend users, distributors, and system integrators to
> > -upgrade unless they use current git master.
>
> This makes sense imo.
>
> Thank you for trying to f
Michael Niedermayer niedermayer.cc> writes:
> -We strongly recommend users, distributors, and system integrators to
> -upgrade unless they use current git master.
This makes sense imo.
Thank you for trying to fix the issues, Carl Eugen
___
ff
On Thu, Jun 30, 2016 at 01:25:59AM +0200, Hendrik Leppkes wrote:
> On Thu, Jun 30, 2016 at 12:50 AM, Timo Rothenpieler
> wrote:
> >>> While it's true that all fields after it are not public API/ABI, a lot
> >>> of software seems to misuse it, so to avoid too much trouble for now,
> >>> this should
James Almer gmail.com> writes:
> + NOTE: Users of FFmpeg 3.1 should avoid running
> the CLI tools ffmpeg and ffprobe from the 3.0 release with
> libraries from this release
Since nobody wants to do this, I don't think this warning
is necessary.
Carl Eugen
__
On Thu, Jun 30, 2016 at 12:50 AM, Timo Rothenpieler
wrote:
>>> While it's true that all fields after it are not public API/ABI, a lot
>>> of software seems to misuse it, so to avoid too much trouble for now,
>>> this should be fixed.
>>
>> How do you expect library users to start using the API cor
>> During an actual major bump, where such breakage is expected.
>> A lot of stuff gets it wrong, and it is indeed confusing, so putting all
>> blame on API users seems wrong to me. Specially as this issue will block
>> distributions from adapting 3.1.
>
> How? Just like they are updating to 3.1,
On 6/29/2016 7:50 PM, Timo Rothenpieler wrote:
>>> While it's true that all fields after it are not public API/ABI, a lot
>>> of software seems to misuse it, so to avoid too much trouble for now,
>>> this should be fixed.
>>
>> How do you expect library users to start using the API correctly if eve
>> While it's true that all fields after it are not public API/ABI, a lot
>> of software seems to misuse it, so to avoid too much trouble for now,
>> this should be fixed.
>
> How do you expect library users to start using the API correctly if every
> time they do something wrong we are the ones t
On 6/29/2016 7:39 PM, Timo Rothenpieler wrote:
> On 6/29/2016 10:31 PM, Michael Niedermayer wrote:
>> This is a similar ABI fix to 1eb43af1a0e542ad83dcbf327197785d815fc42d
>
> +1 for this.
>
> While it's true that all fields after it are not public API/ABI, a lot
> of software seems to misuse it,
On 6/29/2016 10:31 PM, Michael Niedermayer wrote:
> This is a similar ABI fix to 1eb43af1a0e542ad83dcbf327197785d815fc42d
+1 for this.
While it's true that all fields after it are not public API/ABI, a lot
of software seems to misuse it, so to avoid too much trouble for now,
this should be fixed.
Signed-off-by: James Almer
---
Suggestions about wording and such are of course welcome.
src/index | 4
1 file changed, 4 insertions(+)
diff --git a/src/index b/src/index
index eb66060..b67d281 100644
--- a/src/index
+++ b/src/index
@@ -92,6 +92,10 @@
We strongly recommend users, dist
On Mon, Jun 27, 2016 at 12:10:53PM +0200, Clément Bœsch wrote:
> From: Clément Bœsch
>
> TODO: bump lavf minor
> XXX: should i add a FF_API_NOCONST_GET_SIDE_DATA?
> ---
> libavformat/avformat.h | 2 +-
> libavformat/utils.c| 2 +-
> 2 files changed, 2 insertions(+), 2 deletions(-)
>
ping
The 3.1 release has some minor ABI incompatibilities compared to 3.0
The fixes of moving struct members or adding getters are disliked
bumping major soname also is disliked by the community
In case a consensus is found about how to deal with the issue, the
recommandition can be put back
---
src/i
On 6/29/2016 5:31 PM, Michael Niedermayer wrote:
> This is a similar ABI fix to 1eb43af1a0e542ad83dcbf327197785d815fc42d
> ---
> libavutil/frame.h | 11 +--
> 1 file changed, 5 insertions(+), 6 deletions(-)
>
> diff --git a/libavutil/frame.h b/libavutil/frame.h
> index 44adec4..2b5c332
On Wed, Jun 29, 2016 at 10:52 PM, Michael Niedermayer
wrote:
> On Wed, Jun 29, 2016 at 10:28:27PM +0200, Hendrik Leppkes wrote:
>> On Wed, Jun 29, 2016 at 10:18 PM, Michael Niedermayer
>> wrote:
>> > On Wed, Jun 29, 2016 at 09:41:50PM +0200, Clément Bœsch wrote:
>> >> On Wed, Jun 29, 2016 at 09:3
On Wed, Jun 29, 2016 at 10:28:27PM +0200, Hendrik Leppkes wrote:
> On Wed, Jun 29, 2016 at 10:18 PM, Michael Niedermayer
> wrote:
> > On Wed, Jun 29, 2016 at 09:41:50PM +0200, Clément Bœsch wrote:
> >> On Wed, Jun 29, 2016 at 09:36:39PM +0200, Michael Niedermayer wrote:
> >> > Signed-off-by: Micha
On Wed, Jun 29, 2016 at 10:31 PM, Michael Niedermayer
wrote:
> This is a similar ABI fix to 1eb43af1a0e542ad83dcbf327197785d815fc42d
All fields below this are documented as no direct access, and this
struct layout is in a release already.
- Hendrik
___
This is a similar ABI fix to 1eb43af1a0e542ad83dcbf327197785d815fc42d
---
libavutil/frame.h | 11 +--
1 file changed, 5 insertions(+), 6 deletions(-)
diff --git a/libavutil/frame.h b/libavutil/frame.h
index 44adec4..2b5c332 100644
--- a/libavutil/frame.h
+++ b/libavutil/frame.h
@@ -428,
On Wed, Jun 29, 2016 at 10:27 PM, Michael Niedermayer
wrote:
> On Wed, Jun 29, 2016 at 05:07:52PM -0300, James Almer wrote:
>> On 6/29/2016 4:36 PM, Michael Niedermayer wrote:
>> > Signed-off-by: Michael Niedermayer
>> > ---
>> > libavformat/avformat.h |4
>> > libavformat/utils.c|
On Wed, Jun 29, 2016 at 10:18 PM, Michael Niedermayer
wrote:
> On Wed, Jun 29, 2016 at 09:41:50PM +0200, Clément Bœsch wrote:
>> On Wed, Jun 29, 2016 at 09:36:39PM +0200, Michael Niedermayer wrote:
>> > Signed-off-by: Michael Niedermayer
>> > ---
>> > libavformat/avformat.h |4
>> > lib
On Wed, Jun 29, 2016 at 05:07:52PM -0300, James Almer wrote:
> On 6/29/2016 4:36 PM, Michael Niedermayer wrote:
> > Signed-off-by: Michael Niedermayer
> > ---
> > libavformat/avformat.h |4
> > libavformat/utils.c| 21 +
> > 2 files changed, 25 insertions(+)
> >
On 6/29/2016 4:36 PM, Michael Niedermayer wrote:
> Signed-off-by: Michael Niedermayer
> ---
> libavformat/avformat.h |4
> libavformat/utils.c| 21 +
> 2 files changed, 25 insertions(+)
>
> diff --git a/libavformat/avformat.h b/libavformat/avformat.h
> index 87
On Wed, Jun 29, 2016 at 09:41:50PM +0200, Clément Bœsch wrote:
> On Wed, Jun 29, 2016 at 09:36:39PM +0200, Michael Niedermayer wrote:
> > Signed-off-by: Michael Niedermayer
> > ---
> > libavformat/avformat.h |4
> > libavformat/utils.c| 21 +
> > 2 files changed
On Wed, Jun 29, 2016 at 09:36:39PM +0200, Michael Niedermayer wrote:
> Signed-off-by: Michael Niedermayer
> ---
> libavformat/avformat.h |4
> libavformat/utils.c| 21 +
> 2 files changed, 25 insertions(+)
>
> diff --git a/libavformat/avformat.h b/libavformat/a
Signed-off-by: Michael Niedermayer
---
ffprobe.c |2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/ffprobe.c b/ffprobe.c
index b9c3760..e8da344 100644
--- a/ffprobe.c
+++ b/ffprobe.c
@@ -2324,7 +2324,7 @@ static int show_stream(WriterContext *w, AVFormatContext
*fmt_ctx, int
Signed-off-by: Michael Niedermayer
---
libavfilter/src_movie.c |2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/libavfilter/src_movie.c b/libavfilter/src_movie.c
index 23dcb7b..0c4ef63 100644
--- a/libavfilter/src_movie.c
+++ b/libavfilter/src_movie.c
@@ -394,7 +394,7 @@ sta
Signed-off-by: Michael Niedermayer
---
tools/uncoded_frame.c |4 ++--
1 file changed, 2 insertions(+), 2 deletions(-)
diff --git a/tools/uncoded_frame.c b/tools/uncoded_frame.c
index 3ca2ba4..f0def44 100644
--- a/tools/uncoded_frame.c
+++ b/tools/uncoded_frame.c
@@ -169,8 +169,8 @@ int main(
Signed-off-by: Michael Niedermayer
---
ffmpeg.c | 37 +++--
ffmpeg_opt.c |4 ++--
2 files changed, 21 insertions(+), 20 deletions(-)
diff --git a/ffmpeg.c b/ffmpeg.c
index 9ffd833..c68461d 100644
--- a/ffmpeg.c
+++ b/ffmpeg.c
@@ -948,7 +948,7 @@ static v
Signed-off-by: Michael Niedermayer
---
libavformat/avformat.h |4
libavformat/utils.c| 21 +
2 files changed, 25 insertions(+)
diff --git a/libavformat/avformat.h b/libavformat/avformat.h
index 876f1e3..89f014b 100644
--- a/libavformat/avformat.h
+++ b/libavfor
On Tue, 28 Jun 2016 11:00:22 +0300, Dan Shamir wrote:
> Hi. A question
> We have developed a new kind of video file and feel that ffmpeg is a good
> tool to help spread usage for this format.
>
> How does the dev community feel about adding a new decoder/muxer format?
I can't really answer that
previously support little endian only because of fate problem
generally native endian code is faster
require 'tests/fate-run: support both le/be formats on pixfmts'
need someone tests it on BE machine
Signed-off-by: Muhammad Faiz
---
libavfilter/vf_lut.c | 85 +++
On Thu, Jun 30, 2016 at 1:24 AM, Muhammad Faiz wrote:
> previously support little endian only because of fate problem
> generally native endian code is faster
>
> require 'tests/fate-run: support both le/be formats on pixfmts'
> need someone tests it on BE machine
>
> Signed-off-by: Muhammad Faiz
previously support little endian only because of fate problem
generally native endian code is faster
require 'tests/fate-run: support both le/be formats on pixfmts'
need someone tests it on BE machine
Signed-off-by: Muhammad Faiz
---
libavfilter/vf_lut.c | 85 +++
Hello Michael,
On 06/29/2016 04:53 AM, Michael Niedermayer wrote:
On Tue, Jun 28, 2016 at 01:33:13PM +0200, Nicolas George wrote:
Le primidi 11 messidor, an CCXXIV, Nicolas George a écrit :
Well, looking at the code, I am thinking that the current design is flawed:
the extra alloc in ff_bsf_ge
Here are execution times of SIMD and non-SIMD functions. The times were
obtained using SystemTap probes at functions' entry and return points.
The dataset used was fate-filter-pixfmts-scale.
SIMD versions have suffix _vsx:
yuy2ToY_c_vsx.
no. of calls: 864. min: 1880 ns. avg: 2014 ns. max:
On 27 June 2016 at 22:38, James Almer wrote:
> On 6/27/2016 8:53 AM, Rostislav Pehlivanov wrote:
> > I've attached another patch which should work fine now.
> > I did this after the put_signed_rect so it does require the first patch,
> > but if this patch is okay I'll amend and tidy things before
On Wed, Jun 29, 2016 at 02:40:51PM +, Petru Rares Sincraian wrote:
> Hi there,
>
>
> Here is a new test, in this case for asetnsamples.
> fate/filter-audio.mak|5
> ref/fate/filter-asetnsamples | 522
> +++
> 2 files changed, 527 insert
I'm not sure where to post this message, but this list seemed the most
appropriate.
It seems like a better way to go about removing non-digital silence would be to
convert each sample to float or double, then compare that to the max dB
considered silence, then store the starting offset and endi
This patch addresses Trac ticket #5570. The optimized functions are in file
libswscale/ppc/input_vsx.c. Each optimized function name is a concatenation of
the
corresponding name in libswscale/input.c with suffix _vsx.
---
libswscale/ppc/Makefile | 1 +
libswscale/ppc/input_vsx.c| 437
Hi,
On 29/06/2016 17:57, Clément Bœsch wrote:
On Wed, Jun 29, 2016 at 05:54:37PM +0200, Benoit Fouet wrote:
From e514644033781cb431641ae088482f5a8aa2de42 Mon Sep 17 00:00:00 2001
From: Benoit Fouet
Date: Wed, 29 Jun 2016 17:53:50 +0200
Subject: [PATCH] git: ignore test binaries
---
.gitign
On Wed, Jun 29, 2016 at 05:54:37PM +0200, Benoit Fouet wrote:
>
> From e514644033781cb431641ae088482f5a8aa2de42 Mon Sep 17 00:00:00 2001
> From: Benoit Fouet
> Date: Wed, 29 Jun 2016 17:53:50 +0200
> Subject: [PATCH] git: ignore test binaries
>
> ---
> .gitignore | 1 +
> 1 file changed, 1 ins
From e514644033781cb431641ae088482f5a8aa2de42 Mon Sep 17 00:00:00 2001
From: Benoit Fouet
Date: Wed, 29 Jun 2016 17:53:50 +0200
Subject: [PATCH] git: ignore test binaries
---
.gitignore | 1 +
1 file changed, 1 insertion(+)
diff --git a/.gitignore b/.gitignore
index 524fb73..670d1d2 100644
--
On Wed, Jun 29, 2016 at 03:47:20PM +0200, Michael Niedermayer wrote:
> ---
> tests/fate/h264.mak |3 +++
> tests/ref/fate/h264-skip-nointra | 54
> ++
> tests/ref/fate/h264-skip-nokey | 54
> ++
> 3 fi
On Wed, Jun 29, 2016 at 03:47:12PM +0200, Timo Rothenpieler wrote:
> > (1) Iam not sure APIchanges, RELEASE_NOTES or Changelog is the best
> > place, iam fine with any of them ...
>
> APIchanges seems like the place where a distribution maintainer would
> most likely look for something like th
Hi there,
Here is a new test, in this case for asetnsamples.
From 350347097df129b82ec5d0d7ed59085f44acc605 Mon Sep 17 00:00:00 2001
From: Petru Rares Sincraian
Date: Wed, 29 Jun 2016 16:26:19 +0200
Subject: [PATCH] fate: add test for asetnsamples
---
tests/fate/filter-audio.mak| 5 +
---
tests/fate/h264.mak |3 +++
tests/ref/fate/h264-skip-nointra | 54 ++
tests/ref/fate/h264-skip-nokey | 54 ++
3 files changed, 111 insertions(+)
create mode 100644 tests/ref/fate/h264-skip-nointra
> (1) Iam not sure APIchanges, RELEASE_NOTES or Changelog is the best
> place, iam fine with any of them ...
APIchanges seems like the place where a distribution maintainer would
most likely look for something like this to me.
___
ffmpeg-devel mailin
On Wed, Jun 29, 2016 at 03:24:22PM +0200, Timo Rothenpieler wrote:
> This fixes an accidental ABI break introduced at 8688d3a.
> ---
> doc/APIchanges | 8
> libavfilter/avfilter.h | 14 +++---
> libavfilter/version.h | 4 ++--
> 3 files changed, 17 insertions(+), 9 dele
From: Sami Hult
NOTE
This is a new patch against the latest master branch. It fixes an error in code
where "file:-" was directed to file descriptor 2 instead of 1.
I changed mode=print file handling of metadata filter to accept any ffmpeg url.
This way multiple streams can for example be silenc
This fixes an accidental ABI break introduced at 8688d3a.
---
doc/APIchanges | 8
libavfilter/avfilter.h | 14 +++---
libavfilter/version.h | 4 ++--
3 files changed, 17 insertions(+), 9 deletions(-)
diff --git a/doc/APIchanges b/doc/APIchanges
index 6dd5ad7..47106c2 1
On Wed, Jun 29, 2016 at 02:43:52PM +0200, Timo Rothenpieler wrote:
> > This has to be documented in APIChanges i think
>
> Adding that field was documented there originally.
> It came in as a merge from libav in 8688d3a, and should have been merged
> to the end of the struct there.
> Technically i
On Wed, Jun 29, 2016 at 02:00:32PM +0200, Clément Bœsch wrote:
> On Wed, Jun 29, 2016 at 01:56:25PM +0200, Michael Niedermayer wrote:
> > On Wed, Jun 29, 2016 at 12:52:18PM +0200, Clément Bœsch wrote:
> > > From: Clément Bœsch
> > >
> > > This will simplify the next merge commit from Libav.
> > >
> This has to be documented in APIChanges i think
Adding that field was documented there originally.
It came in as a merge from libav in 8688d3a, and should have been merged
to the end of the struct there.
Technically it was an ABI break, but nothing seems to be affected by it.
So I'm not exactly
On Wed, Jun 29, 2016 at 01:56:25PM +0200, Michael Niedermayer wrote:
> On Wed, Jun 29, 2016 at 12:52:18PM +0200, Clément Bœsch wrote:
> > From: Clément Bœsch
> >
> > This will simplify the next merge commit from Libav.
> > ---
> > libavcodec/h264_slice.c | 6 +++---
> > 1 file changed, 3 inserti
On Wed, Jun 29, 2016 at 12:52:18PM +0200, Clément Bœsch wrote:
> From: Clément Bœsch
>
> This will simplify the next merge commit from Libav.
> ---
> libavcodec/h264_slice.c | 6 +++---
> 1 file changed, 3 insertions(+), 3 deletions(-)
all files i tried still work with this patch
they also all
thanks, seems the right clue to follow.
I don't know much about ffmpeg, after went through avplay.c. I think I
should do the following steps
1. discard all stream by default.
refer to decode_thread()
for (i = 0; i < ic->nb_streams; i++)
ic->streams[i]->discard = AVDISCARD_ALL;
2. ena
From: Clément Bœsch
This will simplify the next merge commit from Libav.
---
libavcodec/h264_slice.c | 6 +++---
1 file changed, 3 insertions(+), 3 deletions(-)
diff --git a/libavcodec/h264_slice.c b/libavcodec/h264_slice.c
index 474400b..0b5079c 100644
--- a/libavcodec/h264_slice.c
+++ b/libav
Hello,
in attach patch to add a new test for lossless webp
one with direct decoding argb
the second with rgb24 pix_fmt. the crc of this test is the same than the
CRC of the sample lena-rgb24 of png tests.
sample can be found here :
https://we.tl/XA18TMKjzg
and need to be put inside ./fate-suite/
On Wed, Jun 29, 2016 at 11:58:03AM +0200, Timo Rothenpieler wrote:
> ---
> libavfilter/avfilter.h | 14 +++---
> 1 file changed, 7 insertions(+), 7 deletions(-)
This has to be documented in APIChanges i think
[...]
--
Michael GnuPG fingerprint: 9FF2128B147EF6730BADF133611EC787040B0
On Wed, Jun 29, 2016 at 11:58:01AM +0200, Timo Rothenpieler wrote:
> ---
> ffplay.c | 2 +-
> 1 file changed, 1 insertion(+), 1 deletion(-)
LGTM
also please backport to release/3.1
thx
[...]
--
Michael GnuPG fingerprint: 9FF2128B147EF6730BADF133611EC787040B0FAB
The greatest way to live w
On Wed, Jun 29, 2016 at 11:58:02AM +0200, Timo Rothenpieler wrote:
> Even though this is not part of the public API, some external
> applications access fields after it, thus breaking after updating from
> ffmpeg 3.0 or earlier.
> Since it is not public, it can be freely moved to the end to avoid
>
---
libavfilter/avfilter.h | 14 +++---
1 file changed, 7 insertions(+), 7 deletions(-)
diff --git a/libavfilter/avfilter.h b/libavfilter/avfilter.h
index 8a7f791..757b81a 100644
--- a/libavfilter/avfilter.h
+++ b/libavfilter/avfilter.h
@@ -344,6 +344,13 @@ struct AVFilterContext {
Even though this is not part of the public API, some external
applications access fields after it, thus breaking after updating from
ffmpeg 3.0 or earlier.
Since it is not public, it can be freely moved to the end to avoid
that problem in the future.
---
libavfilter/avfilter.h | 12 ++--
1
The field in AVFilterLink is private, but a lot of applications, ffplay
included, still accessed some of them.
So this series first fixes ffplay, and then moves the new fields in
AVFilterLink and AVFilterContext to the end of their structs.
For AVFilterLink this should be unproblematic, as it's p
---
ffplay.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/ffplay.c b/ffplay.c
index f28e087..b0702eb 100644
--- a/ffplay.c
+++ b/ffplay.c
@@ -2725,7 +2725,7 @@ static int stream_component_open(VideoState *is, int
stream_index)
goto fail;
link =
On Mon, Jun 27, 2016 at 3:53 PM, Muhammad Faiz wrote:
> On Mon, Jun 27, 2016 at 3:28 PM, Hendrik Leppkes wrote:
>> On Mon, Jun 27, 2016 at 10:12 AM, Muhammad Faiz wrote:
>>> On Mon, Jun 27, 2016 at 2:59 PM, Hendrik Leppkes
>>> wrote:
On Mon, Jun 27, 2016 at 9:52 AM, Muhammad Faiz wrote:
On Tue, Jun 28, 2016 at 5:45 PM, Michael Niedermayer
wrote:
> On Tue, Jun 28, 2016 at 04:07:31PM +0700, Muhammad Faiz wrote:
>> On Mon, Jun 27, 2016 at 6:02 PM, Michael Niedermayer
>> wrote:
>> > On Mon, Jun 27, 2016 at 04:46:16PM +0700, Muhammad Faiz wrote:
>> >> previously support little endian
On Tue, Jun 28, 2016 at 12:31:33PM +0200, Matthieu Bouron wrote:
> From: Matthieu Bouron
>
> Fixes regression introduced by 0cd5e281df3f69c1ed8f2a72a5bcbf9691e1b5d5.
> ---
> libavcodec/mediacodecdec_h264.c | 21 +++--
> 1 file changed, 19 insertions(+), 2 deletions(-)
>
> diff -
74 matches
Mail list logo
