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
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 ++
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
---
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
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
> ---
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
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 | 17 +
> 1 file changed, 5 insertions(+), 12 deletions(-)
>
probably ok
___
ffmpeg-devel mailing list
ffmpeg-devel@ffmpeg.org
http://ffmpeg.org/mailman/listinfo/ffmpeg-
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
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 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.
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
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
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
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
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
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: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 :
> 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 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
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 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
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 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
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
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 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
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 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
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
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
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
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/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
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 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, 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
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 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
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
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
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: 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
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 d3dc67ad6a..89aecd4de4 100644
--- a/libavcodec/mpeg12dec.c
+++ b/libavcodec/mpeg12dec.c
@@ -1468,6 +1468,10 @@ static void
mpe
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
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
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,
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 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
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 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, 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, 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 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 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 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
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,
58 matches
Mail list logo
