This allows us to report the correct codec ID here
---
libavformat/allformats.c | 3 ++-
libavformat/mp3dec.c | 66 +++-
libavformat/rawenc.c | 13 ++
libavformat/utils.c | 4 +--
libavformat/version.h| 2 +-
5 files changed,
On 23 December 2016 at 05:12, Matt Oliver wrote:
>
> So I did a quick check over the FFmpeg source for all occurrences of DCE in
> the code base. So far I have found the following:
> 251 uses of DCE with ARCH_XXX
> 54 uses of DCE with HAVE_XXX
> 205 uses of DCE with CONFIG_XXX
>
>
> Basically all
On 23 December 2016 at 15:35, Matt Oliver wrote:
> On 21 December 2016 at 23:55, wm4 wrote:
>
>> On Fri, 16 Dec 2016 13:48:16 +1100
>> Matt Oliver wrote:
>>
>> > Recently we have again received several patches that are trying to add
>> > workarounds for ffmpegs use of DCE. This is not the first
On 21 December 2016 at 23:55, wm4 wrote:
> On Fri, 16 Dec 2016 13:48:16 +1100
> Matt Oliver wrote:
>
> > Recently we have again received several patches that are trying to add
> > workarounds for ffmpegs use of DCE. This is not the first time this has
> > happened and wont be the last until a de
On Thu, 15 Dec 2016, Marton Balint wrote:
Signed-off-by: Marton Balint
---
libavfilter/af_amerge.c | 11 ---
1 file changed, 8 insertions(+), 3 deletions(-)
diff --git a/libavfilter/af_amerge.c b/libavfilter/af_amerge.c
index 4a8c6d5..3bc7d89 100644
--- a/libavfilter/af_amerge.c
+++ b/
On Thu, 22 Dec 2016, Nicolas George wrote:
Le primidi 1er nivôse, an CCXXV, Marton Balint a écrit :
It seems after this patch I got an infinite loop if I try to convert the
attached file using this command line:
./ffmpeg -i amerge-test.mov -filter_complex "[0:a:0][0:a:1]amerge=2[aout]"
-map "
On Thu, Dec 22, 2016 at 03:39:05PM -0900, Lou Logan wrote:
> Signed-off-by: Lou Logan
> ---
> doc/scaler.texi | 3 ++-
> 1 file changed, 2 insertions(+), 1 deletion(-)
>
> diff --git a/doc/scaler.texi b/doc/scaler.texi
> index 3e115cd..7d482dd 100644
> --- a/doc/scaler.texi
> +++ b/doc/scaler.te
On Thu, Dec 22, 2016 at 04:12:36PM -0800, Thomas Turner wrote:
> Signed-off-by: Thomas Turner
> ---
> libavutil/tests/random_seed.c | 34 +-
> tests/ref/fate/random_seed| 1 +
> 2 files changed, 22 insertions(+), 13 deletions(-)
patch applied
thx
[...]
--
On 22.12.2016 22:52, Paul B Mahol wrote:
> ffmpeg | branch: master | Paul B Mahol | Fri Dec 2
> 20:30:50 2016 +0100| [73651090ca1183f37753ee30a7e206ca4fb9f4f0] | committer:
> Paul B Mahol
>
> avcodec: add Apple Pixlet decoder
>
> Signed-off-by: Paul B Mahol
>
>> http://git.videolan.org/gitw
Signed-off-by: Lou Logan
---
doc/scaler.texi | 3 ++-
1 file changed, 2 insertions(+), 1 deletion(-)
diff --git a/doc/scaler.texi b/doc/scaler.texi
index 3e115cd..7d482dd 100644
--- a/doc/scaler.texi
+++ b/doc/scaler.texi
@@ -13,7 +13,8 @@ FFmpeg tools. For programmatic use, they can be set
exp
On 11/27/2016 1:08 PM, Nicolas George wrote:
> A lot of changes happen at the same time:
>
> - Add a framequeue fifo to AVFilterLink.
>
> - split AVFilterLink.status into status_in and status_out: requires
> changes to the few filters and programs that use it directly
> (f_interleave, split,
On Thu, Dec 22, 2016 at 07:10:39PM -0300, James Almer wrote:
> On 12/22/2016 2:16 PM, Nicolas George wrote:
> > Le duodi 2 nivôse, an CCXXV, James Almer a écrit :
> >> You still have time to address the arguments from wm4's review you
> >> skipped in your previous reply. You only sort-of answered t
Signed-off-by: Thomas Turner
---
libavutil/tests/random_seed.c | 34 +-
tests/ref/fate/random_seed| 1 +
2 files changed, 22 insertions(+), 13 deletions(-)
diff --git a/libavutil/tests/random_seed.c b/libavutil/tests/random_seed.c
index ebe9b3e..fcd68bc 10064
Signed-off-by: Andreas Cadhalpun
---
libavcodec/mpeg12dec.c | 4
1 file changed, 4 insertions(+)
diff --git a/libavcodec/mpeg12dec.c b/libavcodec/mpeg12dec.c
index d3dc67ad6a..89aecd4de4 100644
--- a/libavcodec/mpeg12dec.c
+++ b/libavcodec/mpeg12dec.c
@@ -1468,6 +1468,10 @@ static void
mpe
Signed-off-by: Andreas Cadhalpun
---
libavcodec/mpeg12dec.c | 4
1 file changed, 4 insertions(+)
diff --git a/libavcodec/mpeg12dec.c b/libavcodec/mpeg12dec.c
index 89aecd4de4..964f498f3b 100644
--- a/libavcodec/mpeg12dec.c
+++ b/libavcodec/mpeg12dec.c
@@ -1473,6 +1473,10 @@ static void
mpe
Signed-off-by: Andreas Cadhalpun
---
libavcodec/mpeg12dec.c | 4
1 file changed, 4 insertions(+)
diff --git a/libavcodec/mpeg12dec.c b/libavcodec/mpeg12dec.c
index 63979079c8..d3dc67ad6a 100644
--- a/libavcodec/mpeg12dec.c
+++ b/libavcodec/mpeg12dec.c
@@ -1470,6 +1470,10 @@ static void
mpe
On 21.12.2016 13:46, wm4 wrote:
> On Wed, 21 Dec 2016 01:43:46 +0100
> Andreas Cadhalpun wrote:
>> On 20.12.2016 15:22, wm4 wrote:
>>> On Mon, 19 Dec 2016 23:36:11 +0100
>>> Andreas Cadhalpun wrote:
On 16.12.2016 17:22, wm4 wrote:
> Are you sure we need the message?
Yes,
Signed-off-by: James Almer
---
libavcodec/pixlet.c | 6 +++---
1 file changed, 3 insertions(+), 3 deletions(-)
diff --git a/libavcodec/pixlet.c b/libavcodec/pixlet.c
index c5d37bb..c1bd321 100644
--- a/libavcodec/pixlet.c
+++ b/libavcodec/pixlet.c
@@ -457,7 +457,7 @@ static void postprocess_luma
Le duodi 2 nivôse, an CCXXV, James Almer a écrit :
> Then you're not addressing them
I have: the complexity, in all its details, is necessary because of the
next patch. Including the "leaky abstraction", including the O(1)
implementation, including the lack of genericness.
This is the short and l
On 12/22/2016 2:16 PM, Nicolas George wrote:
> Le duodi 2 nivôse, an CCXXV, James Almer a écrit :
>> You still have time to address the arguments from wm4's review you
>> skipped in your previous reply. You only sort-of answered to the "Is
>> all this complexity really justified?" question.
>
> Al
From: Ganesh Ajjanagadde
Fixes Ticket 5389.
Signed-off-by: Ganesh Ajjanagadde
---
doc/ffplay.texi | 4
ffplay.c| 10 +-
2 files changed, 13 insertions(+), 1 deletion(-)
diff --git a/doc/ffplay.texi b/doc/ffplay.texi
index 073b457256..378a229d67 100644
--- a/doc/ffplay.te
On Thu, Dec 22, 2016 at 02:16:30PM +0100, Tobias Rapp wrote:
> Signed-off-by: Tobias Rapp
> ---
> libavformat/avidec.c | 14 +++---
> 1 file changed, 7 insertions(+), 7 deletions(-)
LGTM
thx
[...]
--
Michael GnuPG fingerprint: 9FF2128B147EF6730BADF133611EC787040B0FAB
The real eba
On Sat, Dec 17, 2016 at 10:48:50PM -0700, pkoshe...@gmail.com wrote:
> From: Pavel Koshevoy
>
> ---
> tests/fate/video.mak| 3 +++
> tests/ref/fate/mpeg2-ticket6024 | 27 +++
> 2 files changed, 30 insertions(+)
> create mode 100644 tests/ref/fate/mpeg2-ticke
On Thu, 22 Dec 2016 18:16:49 +0100
Nicolas George wrote:
> Le duodi 2 nivôse, an CCXXV, James Almer a écrit :
> > You still have time to address the arguments from wm4's review you
> > skipped in your previous reply. You only sort-of answered to the "Is
> > all this complexity really justified?"
On 12/22/16, Michael Niedermayer wrote:
> On Wed, Dec 21, 2016 at 09:16:40PM +0100, Paul B Mahol wrote:
>> Signed-off-by: Paul B Mahol
>> ---
>> Changelog | 1 +
>> doc/general.texi| 1 +
>> libavcodec/Makefile | 1 +
>> libavcodec/allcodecs.c | 1 +
>> libavco
Le duodi 2 nivôse, an CCXXV, James Almer a écrit :
> You still have time to address the arguments from wm4's review you
> skipped in your previous reply. You only sort-of answered to the "Is
> all this complexity really justified?" question.
All the rest was only sub-questions to the big one: deta
On 12/22/2016 1:44 PM, Nicolas George wrote:
> wm4:
>>
>
> I will not waste my and everybody's time answering these half-truths
> directly.
>
> I will, of course, continue to answer to people of good will.
You still have time to address the arguments from wm4's review you
skipped in your previo
Signed-off-by: Michael Niedermayer
---
libavcodec/flvdec.c| 1 +
libavcodec/h263dec.c | 2 ++
libavcodec/intelh263dec.c | 1 +
libavcodec/mpeg4videodec.c | 1 +
libavcodec/msmpeg4dec.c| 4
5 files changed, 9 insertions(+)
diff --git a/libavcodec/flvdec.c b/libavcodec/flvd
wm4:
>
I will not waste my and everybody's time answering these half-truths
directly.
I will, of course, continue to answer to people of good will.
Regards,
--
Nicolas George
signature.asc
Description: Digital signature
___
ffmpeg-devel mailing
On Thu, 22 Dec 2016 16:51:10 +0100
Nicolas George wrote:
> Le duodi 2 nivôse, an CCXXV, James Almer a écrit :
> > How's that addressing his concerns?
>
> The answer I would have posted directly to his remarks would have been:
>
> # > Is all this complexity really justified?
> #
> # Yes, all
Le duodi 2 nivôse, an CCXXV, James Almer a écrit :
> How's that addressing his concerns?
The answer I would have posted directly to his remarks would have been:
# > Is all this complexity really justified?
#
# Yes, all this complexity is really justified, including the "leaky
# abstraction", and
On 12/22/2016 12:16 PM, Nicolas George wrote:
> Thanks for stepping in.
>
> Le duodi 2 nivôse, an CCXXV, James Almer a écrit :
>> And you ignored his.
>
> No, the reply was there:
> http://ffmpeg.org/pipermail/ffmpeg-devel/2016-November/202768.html
>
> Not directly connected in the thread, but s
Thanks for stepping in.
Le duodi 2 nivôse, an CCXXV, James Almer a écrit :
> And you ignored his.
No, the reply was there:
http://ffmpeg.org/pipermail/ffmpeg-devel/2016-November/202768.html
Not directly connected in the thread, but still a reply. No news from
wm4 since, until after the fact.
Re
On 12/22/2016 11:59 AM, Nicolas George wrote:
> Le duodi 2 nivôse, an CCXXV, wm4 a écrit :
>> You ignored my review as well, for "a month" as you admit yourself
>> above.
>
> This is a lie. The last message in the discussion before the month gap
> was mine, not yours. You neglected to reply and wa
On Thu, 22 Dec 2016 15:59:19 +0100
Nicolas George wrote:
> Le duodi 2 nivôse, an CCXXV, wm4 a écrit :
> > You ignored my review as well, for "a month" as you admit yourself
> > above.
>
> This is a lie. The last message in the discussion before the month gap
> was mine, not yours. You neglecte
Le duodi 2 nivôse, an CCXXV, wm4 a écrit :
> You ignored my review as well, for "a month" as you admit yourself
> above.
This is a lie. The last message in the discussion before the month gap
was mine, not yours. You neglected to reply and waited, not me.
If you ever ignore a review from me on pu
On Dec 17, 2016 10:49 PM, wrote:
From: Pavel Koshevoy
---
tests/fate/video.mak| 3 +++
tests/ref/fate/mpeg2-ticket6024 | 27 +++
2 files changed, 30 insertions(+)
create mode 100644 tests/ref/fate/mpeg2-ticket6024
diff --git a/tests/fate/video.mak b/tests
On Thu, 22 Dec 2016 15:28:53 +0100
Nicolas George wrote:
> Le duodi 2 nivôse, an CCXXV, wm4 a écrit :
> > You didn't respond to my concerns. There was no indication that
> > repeating my concerns would have changed anything. It was up to you.
>
> Repeating them would have changed noting indeed
On 22.12.2016 14:50, wm4 wrote:
On Thu, 22 Dec 2016 14:16:30 +0100
Tobias Rapp wrote:
Signed-off-by: Tobias Rapp
---
libavformat/avidec.c | 14 +++---
1 file changed, 7 insertions(+), 7 deletions(-)
diff --git a/libavformat/avidec.c b/libavformat/avidec.c
index d465965..5b2d1bb 1006
Le duodi 2 nivôse, an CCXXV, wm4 a écrit :
> You didn't respond to my concerns. There was no indication that
> repeating my concerns would have changed anything. It was up to you.
Repeating them would have changed noting indeed: still rude and
nonconstructive.
Rewording them would have been the r
On Thu, 22 Dec 2016 14:52:23 +0100
Nicolas George wrote:
> Le duodi 2 nivôse, an CCXXV, wm4 a écrit :
> > That is not true. They were all technical and concerns and I was polite
> > enough.
>
> No. And you had ample time to make your point between 2016-11-14 and
> 2016-12-18.
You didn't respo
Le duodi 2 nivôse, an CCXXV, wm4 a écrit :
> That is not true. They were all technical and concerns and I was polite
> enough.
No. And you had ample time to make your point between 2016-11-14 and
2016-12-18.
> I will consider ignoring feature patch reviews from you as well, then.
Do not start a
On Thu, 22 Dec 2016 14:16:30 +0100
Tobias Rapp wrote:
> Signed-off-by: Tobias Rapp
> ---
> libavformat/avidec.c | 14 +++---
> 1 file changed, 7 insertions(+), 7 deletions(-)
>
> diff --git a/libavformat/avidec.c b/libavformat/avidec.c
> index d465965..5b2d1bb 100644
> --- a/libavforma
On Thu, 22 Dec 2016 13:33:20 +0100
Nicolas George wrote:
> Le duodi 2 nivôse, an CCXXV, wm4 a écrit :
> > So this was pushed without addressing any of my concerns?
>
> Indeed, and you were warned.
>
> For reference for other readers, "my concerns" were matter of personal
> taste formulated as
Signed-off-by: Tobias Rapp
---
libavformat/avidec.c | 14 +++---
1 file changed, 7 insertions(+), 7 deletions(-)
diff --git a/libavformat/avidec.c b/libavformat/avidec.c
index d465965..5b2d1bb 100644
--- a/libavformat/avidec.c
+++ b/libavformat/avidec.c
@@ -160,11 +160,11 @@ static int g
Le duodi 2 nivôse, an CCXXV, wm4 a écrit :
> So this was pushed without addressing any of my concerns?
Indeed, and you were warned.
For reference for other readers, "my concerns" were matter of personal
taste formulated as nonconstructive passive-aggressive questions, and my
warning was:
http://f
On Sun, 27 Nov 2016 17:08:51 +0100
Nicolas George wrote:
> Signed-off-by: Nicolas George
> ---
So this was pushed without addressing any of my concerns?
___
ffmpeg-devel mailing list
ffmpeg-devel@ffmpeg.org
http://ffmpeg.org/mailman/listinfo/ffmpeg-de
On Wed, Dec 21, 2016 at 09:16:40PM +0100, Paul B Mahol wrote:
> Signed-off-by: Paul B Mahol
> ---
> Changelog | 1 +
> doc/general.texi| 1 +
> libavcodec/Makefile | 1 +
> libavcodec/allcodecs.c | 1 +
> libavcodec/avcodec.h| 1 +
> libavcodec/codec_desc.
Le primidi 1er nivôse, an CCXXV, Marton Balint a écrit :
> It seems after this patch I got an infinite loop if I try to convert the
> attached file using this command line:
>
> ./ffmpeg -i amerge-test.mov -filter_complex "[0:a:0][0:a:1]amerge=2[aout]"
> -map "[aout]" out.wav
Can you confirm the a
On 12/20/16, Ronald S. Bultje wrote:
> ---
> libavcodec/wmavoice.c | 56
> +--
> 1 file changed, 28 insertions(+), 28 deletions(-)
>
> diff --git a/libavcodec/wmavoice.c b/libavcodec/wmavoice.c
> index ceac61f..4b3ab43 100644
> --- a/libavcodec/wmav
On 12/20/16, Ronald S. Bultje wrote:
> ---
> libavcodec/wmavoice.c | 17 +
> 1 file changed, 5 insertions(+), 12 deletions(-)
>
probably ok
___
ffmpeg-devel mailing list
ffmpeg-devel@ffmpeg.org
http://ffmpeg.org/mailman/listinfo/ffmpeg-
Le primidi 1er nivôse, an CCXXV, Nicolas George a écrit :
> I thought I had got them all :( I will look into it.
That was silly. Fixed.
Regards,
--
Nicolas George
signature.asc
Description: Digital signature
___
ffmpeg-devel mailing list
ffmpeg-de
On 12/20/16, Ronald S. Bultje wrote:
> ---
> libavcodec/wmavoice.c | 72
> +--
> 1 file changed, 36 insertions(+), 36 deletions(-)
>
> diff --git a/libavcodec/wmavoice.c b/libavcodec/wmavoice.c
> index ae100fb..f31c9d2 100644
> --- a/libavcodec/wmav
On 12/21/16, Andreas Cadhalpun wrote:
> On 20.12.2016 10:23, Muhammad Faiz wrote:
>> On 12/20/16, Andreas Cadhalpun wrote:
>>> On 16.12.2016 07:36, Muhammad Faiz wrote:
On 12/16/16, Andreas Cadhalpun wrote:
> Suggested-by: Rodger Combs
> Signed-off-by: Andreas Cadhalpun
> ---
From: Jan Sebechlebsky
---
doc/muxers.texi | 9 +
libavformat/tee.c | 20
2 files changed, 29 insertions(+)
diff --git a/doc/muxers.texi b/doc/muxers.texi
index ced223e..139ced0 100644
--- a/doc/muxers.texi
+++ b/doc/muxers.texi
@@ -1603,6 +1603,11 @@ outputs and
From: Jan Sebechlebsky
Hello,
I am re-sending the patch(es) adding support for fifo pseudo-muxer in tee muxer.
I will apply the first one in few days (since it already has been reviewed by
Nicolas,
and it differs only in issues Nicolas pointed out in review and these should be
fixed now).
The s
From: Jan Sebechlebsky
Signed-off-by: Jan Sebechlebsky
---
Changes since the last version of the patch:
- Refactoring based on Nicolas's comments
- Added TODO regarding boolean option parsing
doc/muxers.texi | 20 +
libavformat/tee.c | 87 ++
On 21.12.2016 20:50, compn wrote:
On Wed, 21 Dec 2016 12:51:18 -0600
Rodger Combs wrote:
On Dec 21, 2016, at 02:27, Carl Eugen Hoyos
wrote:
2016-12-21 5:48 GMT+01:00 Rodger Combs :
This allows us to report the correct codec ID here
Just curious: What does this fix?
Reporting in ffprob
58 matches
Mail list logo
