The patchset looks fine, push anytime.
lu
___
libav-devel mailing list
libav-devel@libav.org
https://lists.libav.org/mailman/listinfo/libav-devel
On Sat, Oct 06, 2012 at 02:44:41AM +0300, Martin Storsjö wrote:
On Sat, 6 Oct 2012, Diego Biurrun wrote:
On Sat, Oct 06, 2012 at 02:05:21AM +0300, Martin Storsjö wrote:
--- /dev/null
+++ b/libavformat/url-test.c
@@ -0,0 +1,55 @@
+/*
+ * Copyright (c) 2012 Martin Storsjo
No 'ö'?
I
---
libavformat/segment.c | 27 +++
1 file changed, 27 insertions(+)
diff --git a/libavformat/segment.c b/libavformat/segment.c
index 8ac04e2..07bb675 100644
--- a/libavformat/segment.c
+++ b/libavformat/segment.c
@@ -37,6 +37,7 @@ typedef struct {
AVFormatContext
An alpha specifier outside the valid range results in a conversion from
double to long with undefined result. Range-checking the double and
only converting it after it passes avoids this.
Fixes fate-parseutils errors on some systems.
Signed-off-by: Mans Rullgard m...@mansr.com
---
Diego Biurrun di...@biurrun.de writes:
On Sat, Oct 06, 2012 at 02:44:41AM +0300, Martin Storsjö wrote:
On Sat, 6 Oct 2012, Diego Biurrun wrote:
On Sat, Oct 06, 2012 at 02:05:21AM +0300, Martin Storsjö wrote:
--- /dev/null
+++ b/libavformat/url-test.c
@@ -0,0 +1,55 @@
+/*
+ * Copyright (c)
On Sat, Oct 06, 2012 at 10:39:51AM +0100, Måns Rullgård wrote:
Diego Biurrun di...@biurrun.de writes:
On Sat, Oct 06, 2012 at 02:44:41AM +0300, Martin Storsjö wrote:
On Sat, 6 Oct 2012, Diego Biurrun wrote:
On Sat, Oct 06, 2012 at 02:05:21AM +0300, Martin Storsjö wrote:
--- /dev/null
On Sat, 6 Oct 2012, Mans Rullgard wrote:
An alpha specifier outside the valid range results in a conversion from
double to long with undefined result. Range-checking the double and
only converting it after it passes avoids this.
Fixes fate-parseutils errors on some systems.
Signed-off-by:
On Sat, 6 Oct 2012, Luca Barbato wrote:
---
libavformat/segment.c | 27 +++
1 file changed, 27 insertions(+)
diff --git a/libavformat/segment.c b/libavformat/segment.c
index 8ac04e2..07bb675 100644
--- a/libavformat/segment.c
+++ b/libavformat/segment.c
@@ -37,6 +37,7 @@
On 10/06/2012 01:23 PM, Martin Storsjö wrote:
This doesn't look right. The #EXT-X-MEDIA-SEQUENCE tag indicates the
index in the sequence for the first entry in the list. So as long as you
just add more entries to the end of the list, this value should stay
unchanged - you should increment it
If the build_only variable is set in the configuration file, the
FATE client will skip running tests and just compile all targets.
---
This is a slightly cleaned-up version of an RFC patch I sent earlier.
This comes with documentation, so it should be good to go.
doc/fate.texi |1 +
Diego Biurrun di...@biurrun.de writes:
If the build_only variable is set in the configuration file, the
FATE client will skip running tests and just compile all targets.
---
This is a slightly cleaned-up version of an RFC patch I sent earlier.
This comes with documentation, so it should be
Make av_pix_fmt_descriptors table static on next major bump.
Making the table public is dangerous, since the caller has no way to
know how large it actually is. It also prevents adding new fields to
AVPixFmtDescriptor without a major bump.
---
libavutil/pixdesc.c | 30
---
doc/APIchanges |8 ++
libavutil/Makefile |1 +
libavutil/old_pix_fmts.h | 128
libavutil/pixfmt.h | 298 ++
libavutil/version.h |7 +-
5 files changed, 311 insertions(+), 131 deletions(-)
---
avprobe.c |6 +++---
cmdutils.c| 16 +++-
tools/graph2dot.c |4 ++--
3 files changed, 16 insertions(+), 10 deletions(-)
diff --git a/avprobe.c b/avprobe.c
index 16a5d29..3a3ae0f 100644
--- a/avprobe.c
+++ b/avprobe.c
@@ -584,6 +584,7 @@ static void
---
libswscale/output.c |3 +-
libswscale/swscale-test.c | 27 +
libswscale/swscale.c |6 +-
libswscale/swscale_internal.h | 130 +++--
libswscale/swscale_unscaled.c | 20 ---
libswscale/utils.c| 26
---
libavcodec/mpegvideo.c | 28 ++--
1 file changed, 14 insertions(+), 14 deletions(-)
diff --git a/libavcodec/mpegvideo.c b/libavcodec/mpegvideo.c
index f739c2d..995deb8 100644
--- a/libavcodec/mpegvideo.c
+++ b/libavcodec/mpegvideo.c
@@ -1527,20 +1527,20 @@ void
---
libavcodec/imgconvert.c | 30 ++
libavcodec/libopenjpegdec.c | 34 ++
libavcodec/libopenjpegenc.c | 13 ++---
libavcodec/mpegvideo.c | 10 ++
libavcodec/rawdec.c |3 ++-
---
libavfilter/avfilter.c |2 +-
libavfilter/buffersrc.c |2 +-
libavfilter/drawutils.c |2 +-
libavfilter/filtfmts.c |4 ++--
libavfilter/formats.c|6 --
libavfilter/vf_boxblur.c |2 +-
libavfilter/vf_crop.c| 10 +-
---
libavdevice/fbdev.c |2 +-
libavutil/imgutils.c | 24 +++-
2 files changed, 16 insertions(+), 10 deletions(-)
diff --git a/libavdevice/fbdev.c b/libavdevice/fbdev.c
index fffe655..7680b29 100644
--- a/libavdevice/fbdev.c
+++ b/libavdevice/fbdev.c
@@ -171,7 +171,7
---
Supporting either sample format for this particular encoder is trivial
and does not require additional processing in either case.
libavcodec/mpegaudioenc.c | 13 +
1 files changed, 9 insertions(+), 4 deletions(-)
diff --git a/libavcodec/mpegaudioenc.c
When there is only 1 channel, the planar and interleaved formats of the same
data type should be treated as identical.
---
libavcodec/utils.c |9 -
1 files changed, 8 insertions(+), 1 deletions(-)
diff --git a/libavcodec/utils.c b/libavcodec/utils.c
index a72c434..d1b9d0a 100644
---
When there is only 1 channel, the planar and interleaved formats of the same
data type should be treated as identical.
---
libavfilter/af_resample.c |7 ++-
1 files changed, 6 insertions(+), 1 deletions(-)
diff --git a/libavfilter/af_resample.c b/libavfilter/af_resample.c
index
On Sat, Oct 06, 2012 at 06:58:19PM +0200, Anton Khirnov wrote:
---
libavcodec/mpegvideo.c | 28 ++--
1 file changed, 14 insertions(+), 14 deletions(-)
OK
Diego
___
libav-devel mailing list
libav-devel@libav.org
On Sat, Oct 06, 2012 at 05:26:15PM +0100, Måns Rullgård wrote:
Diego Biurrun di...@biurrun.de writes:
If the build_only variable is set in the configuration file, the
FATE client will skip running tests and just compile all targets.
---
This is a slightly cleaned-up version of an RFC
On Sat, 6 Oct 2012 13:30:36 -0400, Justin Ruggles justin.rugg...@gmail.com
wrote:
When there is only 1 channel, the planar and interleaved formats of the same
data type should be treated as identical.
---
libavcodec/utils.c |9 -
1 files changed, 8 insertions(+), 1
On Sat, 6 Oct 2012 13:30:37 -0400, Justin Ruggles justin.rugg...@gmail.com
wrote:
When there is only 1 channel, the planar and interleaved formats of the same
data type should be treated as identical.
---
libavfilter/af_resample.c |7 ++-
1 files changed, 6 insertions(+), 1
On 10/06/2012 02:53 PM, Anton Khirnov wrote:
On Sat, 6 Oct 2012 13:30:37 -0400, Justin Ruggles justin.rugg...@gmail.com
wrote:
When there is only 1 channel, the planar and interleaved formats of the same
data type should be treated as identical.
---
libavfilter/af_resample.c |7
Hi,
On Sat, Oct 6, 2012 at 9:58 AM, Anton Khirnov an...@khirnov.net wrote:
[..]
Yay! OK.
(Didn't review at all, a sed replace can't be that hard, plus this is awesome.)
Ronald
___
libav-devel mailing list
libav-devel@libav.org
Hi,
On Sat, Oct 6, 2012 at 9:58 AM, Anton Khirnov an...@khirnov.net wrote:
---
avprobe.c |6 +++---
cmdutils.c| 16 +++-
tools/graph2dot.c |4 ++--
3 files changed, 16 insertions(+), 10 deletions(-)
diff --git a/avprobe.c b/avprobe.c
index
This is Andrew's Opus range decoder from his GSoC repo. This only includes
the functions required for SILK decoding. There are a few more that are
specific to CELT that are not included.
I have incldued Andrew's original code as a reference test in opusrac.c. I
assume
there will be optimizations
From: Andrew D'Addesio modchip...@gmail.com
Fully functional range decoder, tested against libopus's entdec.c
Includes opusrac-test in opusrac.c
---
libavcodec/Makefile |1 +
libavcodec/opusrac.c | 120 ++
libavcodec/opusrac.h | 90
---
tests/fate/libavcodec.mak |5 +
1 file changed, 5 insertions(+)
diff --git a/tests/fate/libavcodec.mak b/tests/fate/libavcodec.mak
index 2aa9596..71f1430 100644
--- a/tests/fate/libavcodec.mak
+++ b/tests/fate/libavcodec.mak
@@ -7,4 +7,9 @@ FATE_LIBAVCODEC += fate-iirfilter
On Sat, 6 Oct 2012 12:35:44 -0700, Ronald S. Bultje rsbul...@gmail.com
wrote:
Hi,
On Sat, Oct 6, 2012 at 9:58 AM, Anton Khirnov an...@khirnov.net wrote:
---
avprobe.c |6 +++---
cmdutils.c| 16 +++-
tools/graph2dot.c |4 ++--
3 files changed,
From: Ronald S. Bultje rsbul...@gmail.com
Use this in VP8/H264-8bit loopfilter functions so they can be used if
there is no aligned stack (e.g. MSVC 32bit or ICC 10.x).
---
libavcodec/x86/h264_deblock.asm | 27 ++-
libavcodec/x86/h264dsp_init.c | 4 +-
libavcodec/x86/vp8dsp.asm |
On 10/06/2012 06:58 PM, Anton Khirnov wrote:
Make av_pix_fmt_descriptors table static on next major bump.
Making the table public is dangerous, since the caller has no way to
know how large it actually is. It also prevents adding new fields to
AVPixFmtDescriptor without a major bump.
---
On 10/06/2012 06:58 PM, Anton Khirnov wrote:
---
avprobe.c |6 +++---
cmdutils.c| 16 +++-
tools/graph2dot.c |4 ++--
3 files changed, 16 insertions(+), 10 deletions(-)
Seems fine.
lu
___
libav-devel
On 10/06/2012 09:35 PM, Ronald S. Bultje wrote:
Hi,
@@ -629,9 +630,8 @@ static void show_stream(AVFormatContext *fmt_ctx, int
stream_idx)
rational_string(val_str, sizeof(val_str), :,
display_aspect_ratio));
}
-
On 10/06/2012 06:58 PM, Anton Khirnov wrote:
---
libswscale/output.c |3 +-
libswscale/swscale-test.c | 27 +
libswscale/swscale.c |6 +-
libswscale/swscale_internal.h | 130
+++--
libswscale/swscale_unscaled.c
On 10/06/2012 06:58 PM, Anton Khirnov wrote:
---
Ok.
___
libav-devel mailing list
libav-devel@libav.org
https://lists.libav.org/mailman/listinfo/libav-devel
Hi,
On Sat, Oct 6, 2012 at 1:28 PM, Anton Khirnov an...@khirnov.net wrote:
On Sat, 6 Oct 2012 12:35:44 -0700, Ronald S. Bultje rsbul...@gmail.com
wrote:
On Sat, Oct 6, 2012 at 9:58 AM, Anton Khirnov an...@khirnov.net wrote:
---
avprobe.c |6 +++---
cmdutils.c| 16
Should fix fate-acodec-adpcm-ima_wav with several compilers.
---
libavcodec/adpcmenc.c |4 ++--
1 files changed, 2 insertions(+), 2 deletions(-)
diff --git a/libavcodec/adpcmenc.c b/libavcodec/adpcmenc.c
index 5c95ad7..6816f60 100644
--- a/libavcodec/adpcmenc.c
+++ b/libavcodec/adpcmenc.c
@@
On 06/10/2012 17:01, Ronald S. Bultje wrote:
It prevents the caller from accessing all the descriptors when he's
using a newer library than the one he compiled against.
if (compile_ver runtime_ver) error(); on app init. Likely, you want
to do that anyway.
Actually Anton is referring to
On 06/10/2012 11:59 PM, Justin Ruggles wrote:
Should fix fate-acodec-adpcm-ima_wav with several compilers.
---
libavcodec/adpcmenc.c |4 ++--
1 files changed, 2 insertions(+), 2 deletions(-)
For the record, these are MSVC and PathScale EKOPath.
- Derek
On Sat, 6 Oct 2012 17:01:57 -0700, Ronald S. Bultje rsbul...@gmail.com
wrote:
Hi,
On Sat, Oct 6, 2012 at 1:28 PM, Anton Khirnov an...@khirnov.net wrote:
On Sat, 6 Oct 2012 12:35:44 -0700, Ronald S. Bultje rsbul...@gmail.com
wrote:
On Sat, Oct 6, 2012 at 9:58 AM, Anton Khirnov
Hi,
On Sat, Oct 6, 2012 at 9:19 PM, Anton Khirnov an...@khirnov.net wrote:
On Sat, 6 Oct 2012 17:01:57 -0700, Ronald S. Bultje rsbul...@gmail.com
wrote:
Hi,
On Sat, Oct 6, 2012 at 1:28 PM, Anton Khirnov an...@khirnov.net wrote:
On Sat, 6 Oct 2012 12:35:44 -0700, Ronald S. Bultje
On Sat, 6 Oct 2012 21:44:03 -0700, Ronald S. Bultje rsbul...@gmail.com
wrote:
Hi,
On Sat, Oct 6, 2012 at 9:19 PM, Anton Khirnov an...@khirnov.net wrote:
On Sat, 6 Oct 2012 17:01:57 -0700, Ronald S. Bultje rsbul...@gmail.com
wrote:
Hi,
On Sat, Oct 6, 2012 at 1:28 PM, Anton
Hi,
On Sat, Oct 6, 2012 at 9:47 PM, Anton Khirnov an...@khirnov.net wrote:
On Sat, 6 Oct 2012 21:44:03 -0700, Ronald S. Bultje rsbul...@gmail.com
wrote:
Hi,
On Sat, Oct 6, 2012 at 9:19 PM, Anton Khirnov an...@khirnov.net wrote:
On Sat, 6 Oct 2012 17:01:57 -0700, Ronald S. Bultje
Hi,
On Sat, Oct 6, 2012 at 9:58 AM, Anton Khirnov an...@khirnov.net wrote:
---
doc/APIchanges |8 ++
libavutil/Makefile |1 +
libavutil/old_pix_fmts.h | 128
libavutil/pixfmt.h | 298
++
On 06/10/2012 22:11, Ronald S. Bultje wrote:
The same way we can never have msvc support?
Right.
Shall we move on to more practical patches?
flamebait
Sure, are we going to stop reviewing MSVC fixes?
/flamebait
As I said this patch makes the ABI more stable, and distributions love
ABI
49 matches
Mail list logo