Fixes ticket #3862.
As a side effect, this also fixes aac_latm in wav.
Signed-off-by: James Almer jamr...@gmail.com
---
Maybe a check for channels = 0 should be also added to ff_get_wav_header()
right after the sample_rate one?
libavformat/wavdec.c | 3 ++-
1 file changed, 2 insertions(+), 1
On 08/16/2014 11:44 PM, wm4 wrote:
This reasoning may work when you have only a small amount of information
to read. When you are overwhelmed with it, having different places to do
different things is a much better approach. Sending patches to a list
simply doesn't scale.
Also, with a list,
On 08/16/2014 11:11 PM, Nicolas George wrote:
L'octidi 28 thermidor, an CCXXII, Bálint Réczey a écrit :
Using Gerrit and file ownersip are not mutually exclusive. Gerrit
can be configured to automatically invite the right people for review
based on the changed path. We recently migrated to
On 08/16/2014 11:30 PM, Nicolas George wrote:
So what about the code? Shall the FFmpeg developers discard three years of
work and start working on libav? Or shall the libav developers accept to
work with the code from FFmpeg that they do not like?
FFmpeg folks should rework the code to make it
On 08/17/2014 07:41 PM, Andreas Cadhalpun wrote:
Michael Niedermayer already volunteered to help with all security
related problems of FFmpeg in Debian.
So what should he do to relieve the impact on the security and release
teams?
Let's say he would take the role of patching stuff in Stable,
On Mon, Aug 18, 2014 at 10:44:03AM +0800, Andre Wolokita wrote:
Fixed parentheses mismatch.
Signed-off-by: Andre Wolokita andre.wolok...@analog.com
---
libavdevice/v4l2.c |2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
applied
with the previous patches commit message, as that was
Hi,
2014-08-17 23:53 GMT+02:00 Michael Niedermayer michae...@gmx.at:
i think these need a check for top = s-screen_height and
left = s-screen_width
[...]
0x007319ea in gif_fill_rect (picture=0x1a96a60, color=16777215, l=0,
t=65535, w=192, h=-65367) at libavcodec/gifdec.c:108
Sorry
On Mon, Aug 18, 2014 at 03:14:01AM -0300, James Almer wrote:
Fixes ticket #3862.
As a side effect, this also fixes aac_latm in wav.
Signed-off-by: James Almer jamr...@gmail.com
applied
---
Maybe a check for channels = 0 should be also added to ff_get_wav_header()
right after the
Hi,
here's the new version of the patch. Sorry for the delay.
James, I have not done 8-bit AVX versions because it requires unpacks that are
done differently in AVX.
Thanks for the feedback !
-Pierre-Edouard Leperecommit 414ebcfeb47ea99ac7e8281d2794996d8a2a09fc
Author: plepere
Hi,
2014-08-17 19:09 GMT+02:00 Christophe Gisquet christophe.gisq...@gmail.com:
It would possible to detect bitstreams before this change and achieve
correct decoding if need be.
So after further exploration, you can't really detect this:
- there's no metadata for the alac bitstream
- the
On Mon, Aug 18, 2014 at 09:42:08AM +0200, Christophe Gisquet wrote:
Hi,
2014-08-17 23:53 GMT+02:00 Michael Niedermayer michae...@gmx.at:
i think these need a check for top = s-screen_height and
left = s-screen_width
[...]
0x007319ea in gif_fill_rect (picture=0x1a96a60,
Hi,
Ticket #3846 is about an incorrect profile being used:
- The user wants alpha but the profile indicates to software there's no alpha
- Also, the encoder still encodes the alpha channel in spite of the profile
The attached patch is not the most user-friendly, but is preferred by
the encoder
On Sun, Aug 17, 2014 at 9:29 PM, Nicolas George geo...@nsup.org wrote:
Le decadi 30 thermidor, an CCXXII, Micah Galizia a écrit :
When new cookie values (with the same name as an existing cookie) are
returned in an HLS stream, the current implementation will append the
new cookie to the list
On Mon, Aug 18, 2014 at 07:59:11PM +1000, Micah Galizia wrote:
On Sun, Aug 17, 2014 at 9:29 PM, Nicolas George geo...@nsup.org wrote:
Le decadi 30 thermidor, an CCXXII, Micah Galizia a écrit :
When new cookie values (with the same name as an existing cookie) are
returned in an HLS stream,
On date Sunday 2014-08-17 19:04:35 -0400, compn encoded:
libav brought up the point again that it doesnt like libmpcodecs.
That is, they will reunite if we drop mp? Or is that one of the
conditional requirements?
is there anyone using the remaining libmpcodecs filters ?
most of the filters
On Sat, 16 Aug 2014, Thomas Goirand wrote:
Yes, the Linux kernel is a successful project. Does this mean using a
list for reviewing patches is a good thing? No! The workflow with a list
is simply horrible. Using git-review and gerrit is so much better.
Strong NAK here.
First: it breaks the
Le primidi 1er fructidor, an CCXXII, Stefano Sabatini a écrit :
That is, they will reunite if we drop mp?
And accept the features in FFmpeg that they have disparagingly called hacks?
I would not bet on that.
Regards,
--
Nicolas George
signature.asc
Description: Digital signature
Christophe Gisquet christophe.gisquet at gmail.com writes:
An alternative would be to select the profile based on
the pixel format.
What would be the disadvantage in this case?
If a user wants to encode without alpha, he would then
need to change the pixel format as well as specify the
Le primidi 1er fructidor, an CCXXII, Thomas Goirand a écrit :
The problem was enforcing patch review policies.
No, it never was.
There's been a very well commented technical reason stated here: the
release team don't want to deal with 2 of the same library that are
doing (nearly) the same
Andre Wolokita andre.wolokita at analog.com writes:
-if (ret == AVERROR(EINVAL)) {
+if (ret == AVERROR(EINVAL) || ret == AVERROR(ENODATA)) {
If I read fate correctly this broke compilation
on OpenBSD where ENODATA isn't defined.
Carl Eugen
On 07.08.2014 15:36, Michael Niedermayer wrote:
On Thu, Aug 07, 2014 at 01:58:55AM +0200, Lukasz Marek wrote:
PulseAudio expilitly requires name of the source.
This patch makes it use default source when not provided.
It simplifies programistic use.
Signed-off-by: Lukasz Marek
Andreas Cadhalpun andreas.cadhal...@googlemail.com schrieb:
Hi Thomas,
On 18.08.2014 08:36, Thomas Goirand wrote:
There's been a very well commented technical reason stated here: the
release team don't want to deal with 2 of the same library that are
doing (nearly) the same things, with
On 8/18/14, Carl Eugen Hoyos ceho...@ag.or.at wrote:
Ivan Kalvachev ikalvachev at gmail.com writes:
softpulldown - turns soft into hard telecine.
Do you remember how this filter was used with MEncoder?
I have a suspicion that it cannot work with FFmpeg.
I have written about it before:
On
---
libavfilter/af_apad.c | 29 +
1 file changed, 17 insertions(+), 12 deletions(-)
diff --git a/libavfilter/af_apad.c b/libavfilter/af_apad.c
index 15ba8c4..c58cacf 100644
--- a/libavfilter/af_apad.c
+++ b/libavfilter/af_apad.c
@@ -39,8 +39,8 @@ typedef struct {
---
doc/filters.texi | 47 +--
1 file changed, 45 insertions(+), 2 deletions(-)
diff --git a/doc/filters.texi b/doc/filters.texi
index 0ca1d6f..44eecca 100644
--- a/doc/filters.texi
+++ b/doc/filters.texi
@@ -742,8 +742,51 @@ Pass the audio source
Hi,
2014-06-19 6:23 GMT+02:00 Michael Niedermayer michae...@gmx.at:
patch applied
Am I missing something or was it not applied?
--
Christophe
___
ffmpeg-devel mailing list
ffmpeg-devel@ffmpeg.org
http://ffmpeg.org/mailman/listinfo/ffmpeg-devel
On 18 August 2014 02:26, Ivan Kalvachev ikalvac...@gmail.com wrote:
ilpack - interlaced yuv420- yuv422 converter. Scale should be able to
do that too.
Scale doesn't have much (any?) knowledge of interlaced chroma.
That said I could probably port this filter to lavfi because it would
be quite
Hi Moritz,
On 18.08.2014 14:05, Moritz Mühlenhoff wrote:
Andreas Cadhalpun andreas.cadhal...@googlemail.com schrieb:
On 18.08.2014 08:36, Thomas Goirand wrote:
There's been a very well commented technical reason stated here: the
release team don't want to deal with 2 of the same library that
On 8/18/14, Kieran Kunhya kier...@obe.tv wrote:
On 18 August 2014 02:26, Ivan Kalvachev ikalvac...@gmail.com wrote:
ilpack - interlaced yuv420- yuv422 converter. Scale should be able to
do that too.
Scale doesn't have much (any?) knowledge of interlaced chroma.
That said I could probably
On 18 August 2014 14:37, Ivan Kalvachev ikalvac...@gmail.com wrote:
On 8/18/14, Kieran Kunhya kier...@obe.tv wrote:
On 18 August 2014 02:26, Ivan Kalvachev ikalvac...@gmail.com wrote:
ilpack - interlaced yuv420- yuv422 converter. Scale should be able to
do that too.
Scale doesn't have much
Hi,
On Mon, Aug 18, 2014 at 9:48 AM, Kieran Kunhya kier...@obe.tv wrote:
On 18 August 2014 14:37, Ivan Kalvachev ikalvac...@gmail.com wrote:
On 8/18/14, Kieran Kunhya kier...@obe.tv wrote:
On 18 August 2014 02:26, Ivan Kalvachev ikalvac...@gmail.com wrote:
ilpack - interlaced yuv420-
How is it different? If you're interlacing-aware and call swscale_convert()
twice (once for each field, double stride each, alternate offset for second
call), isn't that the same?
That would be true in the 422 domain, yes. In 420, the chroma planes
are offset and this has to be taken into
Hi,
this is a proposal to allow decoding incorrect files. There is no
obvious and systematic way to detect them however.
--
Christophe
From feaacb09d8660b7e8784ad4a550dda455f172e24 Mon Sep 17 00:00:00 2001
From: Christophe Gisquet christophe.gisq...@gmail.com
Date: Mon, 18 Aug 2014 09:53:20
2014-08-18 15:59 GMT+02:00 Christophe Gisquet christophe.gisq...@gmail.com:
this is a proposal to allow decoding incorrect files. There is no
obvious and systematic way to detect them however.
Ignore the version.h stray change in your review, I thought I had removed it.
--
Christophe
On 8/18/14, Christophe Gisquet christophe.gisq...@gmail.com wrote:
Hi,
this is a proposal to allow decoding incorrect files. There is no
obvious and systematic way to detect them however.
--
Christophe
AFAIK it is missing AVClass *class in codec context.
On 2014-08-18 14:53, Stefano Sabatini wrote:
+@item whole_len
+Set the target number of sample in the audio stream. After the value
^^
samples, plural
signature.asc
Description: OpenPGP digital signature
___
On Mon, 18 Aug 2014 13:31:26 +0200
Stefano Sabatini stefa...@gmail.com wrote:
On date Sunday 2014-08-17 19:04:35 -0400, compn encoded:
is there anyone using the remaining libmpcodecs filters ?
Of the remaining filters, at least eq and eq2 are useful, and they are
actually used by real
On Mon, Aug 18, 2014 at 5:53 AM, Stefano Sabatini stefa...@gmail.com wrote:
---
doc/filters.texi | 47 +--
1 file changed, 45 insertions(+), 2 deletions(-)
diff --git a/doc/filters.texi b/doc/filters.texi
index 0ca1d6f..44eecca 100644
---
Hello,
Attached patch fixes segmentation fault in libavformat/a64.c
a64_write_header() when output stream's codec is not open, so
avctx-codec is NULL (in stream_copy use case for example). Correct
access to codec id is use of avctx-codec_id instead of
avctx-codec-id.
Additionally, in
Le primidi 1er fructidor, an CCXXII, Andrey Myznikov a écrit :
Additionally, in 'default' section of switch() statement here, I
propose to return AVERROR(ENOTSUP) instead of AVERROR_INVALIDDATA,
because it is more clear to get someting like
avformat_write_header() fails: Operation not
Hello,
Attached patch fixes segmentation fault in libavformat/a64.c
a64_write_header() when output stream's codec is not open, so
avctx-codec is NULL (in stream_copy use case for example). Correct
access to codec id is use of avctx-codec_id instead of
avctx-codec-id.
Regards,
Signed-off-by: Deti fliegl fli...@baycom.de
---
libavdevice/decklink_common.cpp | 227 ++
libavdevice/decklink_common.h | 98
libavdevice/decklink_common_c.h | 32 +++
libavdevice/decklink_dec.cpp| 520
Hi,
On Mon, Aug 18, 2014 at 9:58 AM, Kieran Kunhya kier...@obe.tv wrote:
How is it different? If you're interlacing-aware and call
swscale_convert()
twice (once for each field, double stride each, alternate offset for
second
call), isn't that the same?
That would be true in the 422
Deti Fliegl deti at fliegl.de writes:
+/* free() is needed for a string returned by the DeckLink SDL. */
+#undef free
I believe this is not needed anymore but ...
+free((void *) tmpDisplayName);
... please move the comment here.
Is the cast necessary?
+av_log(NULL,
2014-08-18 19:26 GMT+02:00 Christophe Gisquet christophe.gisq...@gmail.com:
Hi,
And with all patches.
--
Christophe
From 89b54fbd698eb9c68f0bf30105bcaf72c5575d27 Mon Sep 17 00:00:00 2001
From: Christophe Gisquet christophe.gisq...@gmail.com
Date: Mon, 18 Aug 2014 11:27:50 +0200
Subject:
Hi,
2014-08-18 13:40 GMT+02:00 Carl Eugen Hoyos ceho...@ag.or.at:
Christophe Gisquet christophe.gisquet at gmail.com writes:
If a profile was set, the automatic setting should
probably not overwrite it.
(If this is possible.)
But I consider the usecase where a user wants to encode
alpha
Hi,
2014-08-18 19:22 GMT+02:00 Ronald S. Bultje rsbul...@gmail.com:
You mean the centerpoint chroma location (top-left, top-middle,
middle-middle, etc.) respective to the set of luma pixels?
I don't think it's only that, but it's probably only slightly longer
to implement.
For interlaced
On Mon, Aug 18, 2014 at 08:05:41PM +0300, Andrey Myznikov wrote:
Hello,
Attached patch fixes segmentation fault in libavformat/a64.c
a64_write_header() when output stream's codec is not open, so
avctx-codec is NULL (in stream_copy use case for example).
Correct access to codec id is use
On 18.08.14 19:23, Carl Eugen Hoyos wrote:
Deti Fliegl deti at fliegl.de writes:
+/* free() is needed for a string returned by the DeckLink SDL. */
+#undef free
I believe this is not needed anymore but ...
removed it - but this has been part of the existing code which I simply
moved to a
On 18/08/14 5:01 AM, Pierre Edouard Lepere wrote:
Hi,
here's the new version of the patch. Sorry for the delay.
James, I have not done 8-bit AVX versions because it requires unpacks that
are done differently in AVX.
Aren't you thinking of AVX2 with 256bits wide registers? With AVX i mean an
Rebased commits to have all changes in one patch. Hopefully it's
complete now.
Deti
On 18.08.14 19:48, Deti Fliegl wrote:
On 18.08.14 19:23, Carl Eugen Hoyos wrote:
Deti Fliegl deti at fliegl.de writes:
+/* free() is needed for a string returned by the DeckLink SDL. */
+#undef free
I
On 18.08.14 20:05, Carl Eugen Hoyos wrote:
Deti Fliegl deti at fliegl.de writes:
+av_log(NULL, AV_LOG_ERROR,
Context should not be NULL.
Can hardly be done in this file unless you supply avctx as
argument in every function only for av_log purposes.
Please do this.
Done
On Mon, Aug 18, 2014 at 04:06:30PM +0200, Paul B Mahol wrote:
On 8/18/14, Christophe Gisquet christophe.gisq...@gmail.com wrote:
Hi,
this is a proposal to allow decoding incorrect files. There is no
obvious and systematic way to detect them however.
--
Christophe
AFAIK it is
patch requested by ubitux.
found in code dump from kierank:
+{ CODEC_ID_PCM_S24LE, MKTAG('n', 'i', '2', '4') }, /* BBC
typo fix for using ni24 instead of in24 (#3051) */ //GARYH added
(sorry for inline)
-compn
diff --git a/libavformat/isom.c b/libavformat/isom.c
index
This is adapted from Elemental/libavformat.isom.c.patch by a certain
GARYH. The original patch only includes the LE version.
See also
http://lists.apple.com/archives/quicktime-users/2009/Oct/msg00048.html
---
libavformat/isom.c | 2 ++
1 file changed, 2 insertions(+)
diff --git
On Mon, Aug 18, 2014 at 02:14:23PM +0200, Lukasz Marek wrote:
On 07.08.2014 15:36, Michael Niedermayer wrote:
On Thu, Aug 07, 2014 at 01:58:55AM +0200, Lukasz Marek wrote:
PulseAudio expilitly requires name of the source.
This patch makes it use default source when not provided.
It simplifies
On Mon, Aug 18, 2014 at 11:01:39PM +0200, Clément Bœsch wrote:
This is adapted from Elemental/libavformat.isom.c.patch by a certain
GARYH. The original patch only includes the LE version.
See also
http://lists.apple.com/archives/quicktime-users/2009/Oct/msg00048.html
---
I'm wondering if this patch request got missed? ... or if I neglected something
when submitting? ... Any feedback appreciated.
Cheers
GT
Graham Torn
graham.t...@gmail.com
Begin forwarded message:
From: Graham Torn graham.t...@gmail.com
Subject: [PATCH] libavformat: Add poster_time and
On Mon, Aug 18, 2014 at 11:01:39PM +0200, Clément Bœsch wrote:
This is adapted from Elemental/libavformat.isom.c.patch by a certain
GARYH. The original patch only includes the LE version.
See also
http://lists.apple.com/archives/quicktime-users/2009/Oct/msg00048.html
---
Hi
On Thu, Aug 14, 2014 at 01:24:37PM -0600, Graham Torn wrote:
From: gt-sdi gt...@straylightdigital.com
Extract poster_time and time_scale from quicktime mdhd atom
and add to metadata for output by ffprobe.
Signed-off-by: gt-sdi gt...@straylightdigital.com
---
libavformat/mov.c | 27
On Mon, Aug 18, 2014 at 7:03 AM, Timothy Gu timothyg...@gmail.com wrote:
On Sun, Aug 17, 2014 at 4:54 PM, Muhammad Faiz mfc...@gmail.com wrote:
This fontcolor option uses arithmetic expression, not color value,
so color names aren't available.
Thank's
You should use AV_OPT_TYPE_COLOR
Hi!
Attached patch removes a request for samples of which we already have several
that all work fine.
Please comment, Carl Eugendiff --git a/libavformat/mxfdec.c b/libavformat/mxfdec.c
index 7a4633f..aabae69 100644
--- a/libavformat/mxfdec.c
+++ b/libavformat/mxfdec.c
@@ -1603,9 +1603,6 @@
Hi!
Attached patch from MIKEH / Elemental is apparently meant to implement
setting h264 bitrate. It makes no difference for the sample from ticket
#3392.
I have no idea how to attribute the patch.
Please comment, Carl Eugendiff --git a/libavcodec/h264_ps.c b/libavcodec/h264_ps.c
index
Clément Bœsch u at pkh.me writes:
+{ AV_CODEC_ID_PCM_S24BE, MKTAG('n', 'i', '2', '4') },
BBC typo */
+{ AV_CODEC_ID_PCM_S24LE, MKTAG('n', 'i', '2', '4') },
/* BBC typo */
Doesn't this patch make absolutely sure that the same
files that didn't work before this patch (and
Am 19.08.2014 01:34 schrieb Carl Eugen Hoyos ceho...@ag.or.at:
Hi!
Attached patch from Elemental increases the maximum superframe size, I
don't know if this fixes any samples.
The original patchfile contains no further attribution.
Please comment, Carl Eugen
It makes no sense to send
On Mon, Aug 18, 2014 at 06:54:55AM +0700, Muhammad Faiz wrote:
This fontcolor option uses arithmetic expression, not color value,
so color names aren't available.
Thank's
---
doc/filters.texi | 20 +++
libavfilter/avf_showcqt.c | 84
66 matches
Mail list logo
