[libav-devel] [PATCH 6/8] segment: Set the resend_headers flag for each segment

2012-10-03 Thread Martin Storsjö
This makes sure new inline headers are emitted when the next packet is written. This allows segmenting mpegts without calling write_header/write_trailer (nor freeing/reiniting the muxer) for each segment. --- libavformat/segment.c |3 +++ 1 file changed, 3 insertions(+) diff --git a/libavform

[libav-devel] [PATCH 7/8] segment: Flush buffered data before finishing a segment

2012-10-03 Thread Martin Storsjö
This makes sure any buffered data is written to the segment, for muxers that buffer up data internally (e.g. fragmented mp4). --- libavformat/segment.c |1 + 1 file changed, 1 insertion(+) diff --git a/libavformat/segment.c b/libavformat/segment.c index bdb5254..fb5a22e 100644 --- a/libavform

[libav-devel] [PATCH 8/8] segment: Add an option for omitting the first header and final trailer

2012-10-03 Thread Martin Storsjö
This allows writing totally bare segments, without any header/trailer included anywhere. --- libavformat/segment.c | 52 + 1 file changed, 48 insertions(+), 4 deletions(-) diff --git a/libavformat/segment.c b/libavformat/segment.c index fb5a22e..1

Re: [libav-devel] [PATCH] libspeexdec: handle NULL return value from speex_packet_to_header()

2012-10-03 Thread Martin Storsjö
On Wed, 3 Oct 2012, Justin Ruggles wrote: This will happen when the extradata is not a valid Speex header. --- libavcodec/libspeexdec.c |9 +++-- 1 files changed, 7 insertions(+), 2 deletions(-) diff --git a/libavcodec/libspeexdec.c b/libavcodec/libspeexdec.c index 760bfe2..0c93f05 10064

[libav-devel] [PATCH] h264probe: Don't error out on bits that no longer are reserved

2012-10-03 Thread Martin Storsjö
From: Michael Niedermayer --- libavformat/h264dec.c |2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/libavformat/h264dec.c b/libavformat/h264dec.c index 8070086..3076b7c 100644 --- a/libavformat/h264dec.c +++ b/libavformat/h264dec.c @@ -54,7 +54,7 @@ static int h264_probe(A

Re: [libav-devel] [PATCH 3/8] segment: Use the public av_write_header/av_write_trailer functions

2012-10-03 Thread Martin Storsjö
On Thu, 4 Oct 2012, Luca Barbato wrote: On 10/03/2012 11:37 PM, Martin Storsjö wrote: With this change, the segmenter muxer doesn't rely on anything not available/supported to libavformat external users, making the segmenter muxer do things just like a normal segmenter application

Re: [libav-devel] [PATCH 7/8] segment: Flush buffered data before finishing a segment

2012-10-03 Thread Martin Storsjö
On Thu, 4 Oct 2012, Luca Barbato wrote: On 10/03/2012 11:37 PM, Martin Storsjö wrote: This makes sure any buffered data is written to the segment, for muxers that buffer up data internally (e.g. fragmented mp4). --- libavformat/segment.c |1 + 1 file changed, 1 insertion(+) Ok. I wonder

Re: [libav-devel] [PATCH 2/8] segment: Properly create new AVStreams for the chained muxer

2012-10-04 Thread Martin Storsjö
On Thu, 4 Oct 2012, Anton Khirnov wrote: On Thu, 04 Oct 2012 00:12:38 +0200, Luca Barbato wrote: On 10/03/2012 11:37 PM, Martin Storsjö wrote: > Before, the chained muxer reused the AVStreams array from > the outer muxer, which made it impossible to use the proper > public functions

[libav-devel] [PATCH] segment: Add comments about calls that only are relevant for some muxers

2012-10-04 Thread Martin Storsjö
--- libavformat/segment.c |4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/libavformat/segment.c b/libavformat/segment.c index 1e928a7..375df58 100644 --- a/libavformat/segment.c +++ b/libavformat/segment.c @@ -98,7 +98,7 @@ static int segment_start(AVFormatContext *s, in

Re: [libav-devel] [PATCH 5/5] fate.sh: add support for build-only FATE instances

2012-10-04 Thread Martin Storsjö
On Tue, 20 Sep 2011, Måns Rullgård wrote: Diego Biurrun writes: If the "build_only" variable is set in the configuration file, the FATE client will skip running tests and just compile all targets. --- tests/fate.sh |1 + 1 files changed, 1 insertions(+), 0 deletions(-) diff --git a/test

[libav-devel] [PATCH 1/2] libspeexenc: Add an option for enabling DTX

2012-10-05 Thread Martin Storsjö
From: Dmitry Samonenko Discontinuous transmission is an addition to VAD/VBR operation, that allows to greatly reduce bitrate for silent chunks or stationary noises. --- libavcodec/libspeexenc.c | 10 ++ 1 file changed, 10 insertions(+) diff --git a/libavcodec/libspeexenc.c b/libavcode

[libav-devel] [PATCH 2/2] libspeexenc: Updated commentary to reflect recent changes

2012-10-05 Thread Martin Storsjö
From: Dmitry Samonenko --- libavcodec/libspeexenc.c | 17 + 1 file changed, 17 insertions(+) diff --git a/libavcodec/libspeexenc.c b/libavcodec/libspeexenc.c index 5a5079b..cbfa5be 100644 --- a/libavcodec/libspeexenc.c +++ b/libavcodec/libspeexenc.c @@ -62,6 +62,23 @@ *

[libav-devel] [PATCH 1/4] fate: Add tests of the ff_make_absolute_url function

2012-10-05 Thread Martin Storsjö
--- libavformat/Makefile |2 +- libavformat/url-test.c | 55 tests/Makefile |2 ++ tests/fate/libavformat.mak |5 tests/ref/fate/url | 10 5 files changed, 73 insertions(+), 1 deletion(-) crea

[libav-devel] [PATCH 2/4] url: Don't treat slashes in query parameters as directory separators

2012-10-05 Thread Martin Storsjö
From: Duncan Salerno Strip off query parameters from the original url before applying the new relative path. --- libavformat/url-test.c |1 + libavformat/utils.c|8 +++- tests/ref/fate/url |1 + 3 files changed, 9 insertions(+), 1 deletion(-) diff --git a/libavformat/url

[libav-devel] [PATCH 3/4] url: Handle relative urls being just a new query string

2012-10-05 Thread Martin Storsjö
From: Duncan Salerno --- libavformat/url-test.c |1 + libavformat/utils.c|6 ++ tests/ref/fate/url |1 + 3 files changed, 8 insertions(+) diff --git a/libavformat/url-test.c b/libavformat/url-test.c index aa07ffa..5c0bccb 100644 --- a/libavformat/url-test.c +++ b/libavfo

[libav-devel] [PATCH 4/4] url: Handle relative urls starting with two slashes

2012-10-05 Thread Martin Storsjö
From: Duncan Salerno This is defined by RFC 3986 section 5.4.1 to be handled this way. --- libavformat/url-test.c |1 + libavformat/utils.c| 14 ++ tests/ref/fate/url |1 + 3 files changed, 12 insertions(+), 4 deletions(-) diff --git a/libavformat/url-test.c b/liba

Re: [libav-devel] [PATCH] mingw/cygwin: Stop adding -fno-common to gcc CFLAGS

2012-10-05 Thread Martin Storsjö
On Fri, 5 Oct 2012, Derek Buitenhuis wrote: On 05/10/2012 6:16 PM, Diego Biurrun wrote: This was done to work around toolchain bugs that have long been fixed. --- configure |2 -- 1 files changed, 0 insertions(+), 2 deletions(-) It works on all MinGW-w64 chains I tested. I did not test t

[libav-devel] [PATCH 1/6] smoothstreamingenc: Check the output UrlContext before accessing it

2012-10-05 Thread Martin Storsjö
This code can be called with a NULL UrlContext if writing of the trailer involves seeking. --- libavformat/smoothstreamingenc.c |3 ++- 1 file changed, 2 insertions(+), 1 deletion(-) diff --git a/libavformat/smoothstreamingenc.c b/libavformat/smoothstreamingenc.c index 339a9bb..7ee640a 100644

[libav-devel] [PATCH 2/6] smoothstreamingenc: Properly return errors from ism_flush to the caller

2012-10-05 Thread Martin Storsjö
--- libavformat/smoothstreamingenc.c |7 +-- 1 file changed, 5 insertions(+), 2 deletions(-) diff --git a/libavformat/smoothstreamingenc.c b/libavformat/smoothstreamingenc.c index 7ee640a..d86e87f 100644 --- a/libavformat/smoothstreamingenc.c +++ b/libavformat/smoothstreamingenc.c @@ -551

[libav-devel] [PATCH 3/6] smoothstreamingenc: Move the output_chunk_list and write_manifest functions up

2012-10-05 Thread Martin Storsjö
This allows calling write_manifest from ism_write_header without a forward declaration. --- libavformat/smoothstreamingenc.c | 186 +++--- 1 file changed, 93 insertions(+), 93 deletions(-) diff --git a/libavformat/smoothstreamingenc.c b/libavformat/smoothstreaming

[libav-devel] [PATCH 4/6] smoothstreamingenc: Try writing a manifest when opening the muxer

2012-10-05 Thread Martin Storsjö
This allows failing cleaner and earlier if unable to write to the output directory. --- libavformat/smoothstreamingenc.c |1 + 1 file changed, 1 insertion(+) diff --git a/libavformat/smoothstreamingenc.c b/libavformat/smoothstreamingenc.c index cc34f7e..bafed70 100644 --- a/libavformat/smooth

[libav-devel] [PATCH 5/6] smoothstreamingenc: Ignore the return value from mkdir

2012-10-05 Thread Martin Storsjö
We don't need to return an error if the directory already existed. We make sure that the directory actually is properly writeable later within ism_write_header by writing a manifest anyway. --- libavformat/smoothstreamingenc.c |7 +-- 1 file changed, 1 insertion(+), 6 deletions(-) diff --

[libav-devel] [PATCH 6/6] smoothstreamingenc: Add a more verbose error message

2012-10-05 Thread Martin Storsjö
This helps tracking down the cause if this happens to be an issue. --- libavformat/smoothstreamingenc.c |4 +++- 1 file changed, 3 insertions(+), 1 deletion(-) diff --git a/libavformat/smoothstreamingenc.c b/libavformat/smoothstreamingenc.c index a8af711..69d555f 100644 --- a/libavformat/smoo

Re: [libav-devel] [PATCH 1/4] fate: Add tests of the ff_make_absolute_url function

2012-10-05 Thread Martin Storsjö
On Sat, 6 Oct 2012, Diego Biurrun wrote: On Sat, Oct 06, 2012 at 02:05:21AM +0300, Martin Storsjö wrote: --- a/libavformat/Makefile +++ b/libavformat/Makefile @@ -376,7 +376,7 @@ SKIPHEADERS-$(CONFIG_NETWORK)+= network.h rtsp.h EXAMPLES = metadata

Re: [libav-devel] [PATCH] parseutils: fix parsing of invalid alpha values

2012-10-06 Thread Martin Storsjö
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: Man

Re: [libav-devel] [PATCH] segment: support hls style playlist

2012-10-06 Thread Martin Storsjö
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 @@

Re: [libav-devel] [PATCH] adpcmenc: ensure calls to adpcm_ima_compress_sample() are in the right order

2012-10-07 Thread Martin Storsjö
On Sat, 6 Oct 2012, Justin Ruggles wrote: 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

[libav-devel] [PATCH] gxf: Include a header for the avpriv_frame_rate_tab declaration

2012-10-07 Thread Martin Storsjö
--- libavformat/gxf.c |2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/libavformat/gxf.c b/libavformat/gxf.c index f215329..04d75bd 100644 --- a/libavformat/gxf.c +++ b/libavformat/gxf.c @@ -23,6 +23,7 @@ #include "avformat.h" #include "internal.h" #include "gxf.h" +#inclu

Re: [libav-devel] [libav-commits] ppc: fmtconvert: kill VLA in float_to_int16_interleave_altivec()

2012-10-07 Thread Martin Storsjö
On Fri, 5 Oct 2012, Mans Rullgard wrote: Module: libav Branch: master Commit: 642b4efaf7b3055ab4b26bda252149eb35babc4b Author:Mans Rullgard Committer: Mans Rullgard Date: Thu Oct 4 01:26:50 2012 +0100 ppc: fmtconvert: kill VLA in float_to_int16_interleave_altivec() This seems t

Re: [libav-devel] [PATCH] doc: update the faq entry about custom I/O

2012-10-07 Thread Martin Storsjö
On Mon, 8 Oct 2012, Luca Barbato wrote: URLProtocols are private and custom AVIOContexts are a simple and easier solution for most of the situations. --- doc/faq.texi | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/doc/faq.texi b/doc/faq.texi index 79ef3b5..a23604c 100644

[libav-devel] [PATCH 1/2] rtpdec_vp8: Make the depacketizer implement the latest spec draft

2012-10-08 Thread Martin Storsjö
Not all details are implemented, but it's enough for proper playback as long as there is no packet loss. Tested to work with the packetizer in gstreamer (which although uses a different codec name, to clarify that it is still a spec draft). --- libavformat/rtpdec_vp8.c | 117

[libav-devel] [PATCH 2/2] rtpenc_vp8: Update the packetizer to the latest spec version

2012-10-08 Thread Martin Storsjö
Tested to work with the gstreamer depacketizer. --- libavformat/rtpenc.c |4 libavformat/rtpenc_vp8.c |6 -- 2 files changed, 4 insertions(+), 6 deletions(-) diff --git a/libavformat/rtpenc.c b/libavformat/rtpenc.c index dc257f6..36064ed 100644 --- a/libavformat/rtpenc.c +++

[libav-devel] [PATCH] rtp: Support packetization/depacketization of opus

2012-10-08 Thread Martin Storsjö
--- libavformat/rtpdec.c |7 +++ libavformat/rtpenc.c | 15 +++ libavformat/sdp.c|4 3 files changed, 26 insertions(+) diff --git a/libavformat/rtpdec.c b/libavformat/rtpdec.c index 0f4e1d1..dac367d 100644 --- a/libavformat/rtpdec.c +++ b/libavformat/rtpdec.c @@

Re: [libav-devel] [PATCH] rtp: Support packetization/depacketization of opus

2012-10-08 Thread Martin Storsjö
On Tue, 9 Oct 2012, Martin Storsjö wrote: --- libavformat/rtpdec.c |7 +++ libavformat/rtpenc.c | 15 +++ libavformat/sdp.c|4 3 files changed, 26 insertions(+) diff --git a/libavformat/rtpdec.c b/libavformat/rtpdec.c index 0f4e1d1..dac367d 100644 --- a

Re: [libav-devel] [PATCH] ppc: fix Altivec build with old compilers

2012-10-08 Thread Martin Storsjö
On Mon, 8 Oct 2012, Mans Rullgard wrote: The vec_splat() intrinsic requires a constant argument for the element number, and the code relies on the compiler unrolling the loop to provide this. Manually unrolling the loop avoids this reliance and works with all compilers. Signed-off-by: Mans Rul

[libav-devel] [PATCH] rtpdec_vp8: Mark unused variables with av_unused

2012-10-08 Thread Martin Storsjö
They still are useful since most of the header is parsed currently, even though not all fields are used. --- libavformat/rtpdec_vp8.c |4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/libavformat/rtpdec_vp8.c b/libavformat/rtpdec_vp8.c index a52be4b..efb9810 100644 --- a/l

[libav-devel] [PATCH] rtpdec_vp8: Don't parse fields that aren't used

2012-10-08 Thread Martin Storsjö
This avoids warnings about unused variables. --- libavformat/rtpdec_vp8.c | 25 ++--- 1 file changed, 6 insertions(+), 19 deletions(-) diff --git a/libavformat/rtpdec_vp8.c b/libavformat/rtpdec_vp8.c index a52be4b..999dd77 100644 --- a/libavformat/rtpdec_vp8.c +++ b/libavfor

Re: [libav-devel] [PATCH 2/2] Employ proper return values in case of missing features

2012-10-09 Thread Martin Storsjö
On Tue, 9 Oct 2012, Diego Biurrun wrote: --- Some cases I'm not sure about: * libavformat/rtpdec_qt.c: replacing ENOSYS is hopefully OK This one is ok with me. // Martin ___ libav-devel mailing list libav-devel@libav.org https://lists.libav.org/mai

Re: [libav-devel] [PATCH] file: Set the return value type for lseek to int64_t.

2012-10-09 Thread Martin Storsjö
On Tue, 9 Oct 2012, Yusuke Nakamura wrote: Attached patch fixes seeking to file offset exceeding 2G when int is defined as 32bit signed integer. This patch is similar withhttps://github.com/FFmpeg/FFmpeg/commit/d37044647f83f53a1cb711ea4655b944ff4 af0e5 but ffmpeg's one doesn't work on MinGW (no

Re: [libav-devel] [PATCH] rtp: Support packetization/depacketization of opus

2012-10-09 Thread Martin Storsjö
On Tue, 9 Oct 2012, Luca Barbato wrote: On 10/09/2012 12:06 AM, Martin Storsjö wrote: On Tue, 9 Oct 2012, Martin Storsjö wrote: --- libavformat/rtpdec.c |7 +++ libavformat/rtpenc.c | 15 +++ libavformat/sdp.c|4 3 files changed, 26 insertions(+) diff --git a

[libav-devel] [PATCH 1/4] url: Don't treat slashes in query parameters as directory separators

2012-10-09 Thread Martin Storsjö
From: Duncan Salerno Strip off query parameters from the original url before applying the new relative path. --- libavformat/utils.c |8 +++- 1 file changed, 7 insertions(+), 1 deletion(-) diff --git a/libavformat/utils.c b/libavformat/utils.c index bba5b2a..1fd0ba3 100644 --- a/libavfo

[libav-devel] [PATCH 2/4] url: Handle relative urls being just a new query string

2012-10-09 Thread Martin Storsjö
From: Duncan Salerno --- libavformat/utils.c |6 ++ 1 file changed, 6 insertions(+) diff --git a/libavformat/utils.c b/libavformat/utils.c index 1fd0ba3..9d0049a 100644 --- a/libavformat/utils.c +++ b/libavformat/utils.c @@ -3417,6 +3417,12 @@ void ff_make_absolute_url(char *buf, int si

[libav-devel] [PATCH 3/4] url: Handle relative urls starting with two slashes

2012-10-09 Thread Martin Storsjö
From: Duncan Salerno This is defined by RFC 3986 section 5.4.1 to be handled this way. --- libavformat/utils.c | 14 ++ 1 file changed, 10 insertions(+), 4 deletions(-) diff --git a/libavformat/utils.c b/libavformat/utils.c index 9d0049a..ca52469 100644 --- a/libavformat/utils.c +

[libav-devel] [PATCH 4/4] fate: Add tests of the ff_make_absolute_url function

2012-10-09 Thread Martin Storsjö
--- libavformat/Makefile |3 ++- libavformat/url-test.c | 58 tests/Makefile |2 ++ tests/fate/libavformat.mak |5 tests/ref/fate/url | 13 ++ 5 files changed, 80 insertions(+), 1 deletion(-) c

Re: [libav-devel] [PATCH] build: link test programs only against static libs

2012-10-09 Thread Martin Storsjö
On Mon, 8 Oct 2012, Mans Rullgard wrote: The test programs use internal symbols so cannot be linked against the shared libs. Linking against both shared and static is pointless and might do something strange depending on the linker. This changes the dependencies so the test programs are linked

Re: [libav-devel] [PATCH 2/2] rtpdec_mpeg4: au_headers is a single array, simple av_free is enough

2012-10-09 Thread Martin Storsjö
On Tue, 9 Oct 2012, Janne Grunau wrote: Fixes CID700204. --- libavformat/rtpdec_mpeg4.c | 10 +- 1 file changed, 1 insertion(+), 9 deletions(-) diff --git a/libavformat/rtpdec_mpeg4.c b/libavformat/rtpdec_mpeg4.c index d3e0754..cc92c88 100644 --- a/libavformat/rtpdec_mpeg4.c +++ b/libavf

Re: [libav-devel] [PATCH] avutil: Duplicate ff_log2_tab instead of sharing it across libs

2012-10-09 Thread Martin Storsjö
On Thu, 4 Oct 2012, Diego Biurrun wrote: The table is so small that the space gain is not worth the performance overhead of cross-library access. --- Now with each duplicated table in a separate file. libavcodec/Makefile |1 + libavcodec/log2_tab.c |1 + libavutil/Makefile |

Re: [libav-devel] [PATCH] build: link test programs only against static libs

2012-10-09 Thread Martin Storsjö
On Tue, 9 Oct 2012, Måns Rullgård wrote: Diego Biurrun writes: On Mon, Oct 08, 2012 at 08:39:39PM +0100, Mans Rullgard wrote: The test programs use internal symbols so cannot be linked against the shared libs. Linking against both shared and static is pointless and might do something strang

[libav-devel] [PATCH] build: Don't pass DEP_LIBS as parameters to the linker

2012-10-09 Thread Martin Storsjö
DEP_LIBS is part of $^, since DEP_LIBS is a dependency of the test tools. The libs, that originate in the FFLIBS variable, are added into FFEXTRALIBS already (normally in the form -lavutil), but via DEP_LIBS in the literal form as libavutil/libavutil.so. Previously, when building test tools, the m

Re: [libav-devel] [PATCH 3/4] g722enc: fix size argument in memset

2012-10-09 Thread Martin Storsjö
On Tue, 9 Oct 2012, Janne Grunau wrote: Fixes CID700725. --- libavcodec/g722enc.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/libavcodec/g722enc.c b/libavcodec/g722enc.c index cc87936..11d3f20 100644 --- a/libavcodec/g722enc.c +++ b/libavcodec/g722enc.c @@ -196,7 +196,7 @@

Re: [libav-devel] [PATCH] build: Don't pass DEP_LIBS as parameters to the linker

2012-10-09 Thread Martin Storsjö
On Tue, 9 Oct 2012, Måns Rullgård wrote: Martin Storsjö writes: DEP_LIBS is part of $^, since DEP_LIBS is a dependency of the test tools. The libs, that originate in the FFLIBS variable, are added into FFEXTRALIBS already (normally in the form -lavutil), but via DEP_LIBS in the literal form

[libav-devel] [PATCH] random_seed: Rewrite generic code

2012-10-09 Thread Martin Storsjö
From: Michael Niedermayer The new code is faster and reuses the previous state in case of multiple calls. The previous code could easily end up in near-infinite loops, if the difference between two clock() calls never was larger than 1. This makes fate-parseutils built with MSVC finish in finit

Re: [libav-devel] [PATCH] build: Don't pass DEP_LIBS as parameters to the linker

2012-10-09 Thread Martin Storsjö
On Tue, 9 Oct 2012, Måns Rullgård wrote: Martin Storsjö writes: On Tue, 9 Oct 2012, Måns Rullgård wrote: Martin Storsjö writes: DEP_LIBS is part of $^, since DEP_LIBS is a dependency of the test tools. The libs, that originate in the FFLIBS variable, are added into FFEXTRALIBS already

Re: [libav-devel] [PATCH] random_seed: Rewrite generic code

2012-10-09 Thread Martin Storsjö
On Tue, 9 Oct 2012, Måns Rullgård wrote: Martin Storsjö writes: From: Michael Niedermayer The new code is faster and reuses the previous state in case of multiple calls. The previous code could easily end up in near-infinite loops, if the difference between two clock() calls never was

[libav-devel] [PATCH] random_seed: Rewrite generic code

2012-10-09 Thread Martin Storsjö
From: Michael Niedermayer The new code is faster and reuses the previous state in case of multiple calls. The previous code could easily end up in near-infinite loops, if the difference between two clock() calls never was larger than 1. This makes fate-parseutils built with MSVC finish in finit

Re: [libav-devel] [PATCH 1/3] segment: support applehttp style list

2012-10-09 Thread Martin Storsjö
On Tue, 9 Oct 2012, Luca Barbato wrote: --- Should be ok now, Martin, did I overlook something else? libavformat/segment.c | 49 +++-- 1 file changed, 47 insertions(+), 2 deletions(-) diff --git a/libavformat/segment.c b/libavformat/segment.c index 8

Re: [libav-devel] [PATCH] build: sanitize linking of tools and test programs

2012-10-10 Thread Martin Storsjö
On Wed, 10 Oct 2012, Mans Rullgard wrote: This makes sure proper linker arguments are used for the tools and test programs when shared libraries are enabled. The tools are linked using the usual -l flag while for test programs the full name of the static library is used. Signed-off-by: Mans Ru

Re: [libav-devel] [PATCH 2/2] segment: support applehttp style list

2012-10-10 Thread Martin Storsjö
On Wed, 10 Oct 2012, Luca Barbato wrote: --- libavformat/segment.c | 85 +++ 1 file changed, 72 insertions(+), 13 deletions(-) Ok with me, assuming that it's tested. // Martin ___ libav-devel mailing li

[libav-devel] [PATCH] build: Rename the LIB variable for testprog targets

2012-10-10 Thread Martin Storsjö
On MSVC, the LIB environment variable is used to specify the default paths for libraries. When this variable is set in the makefile, it overwrites the original lib path variable for the linker. --- This slipped me before, since the env variables are set up slightly differently in my msvc/wine test

Re: [libav-devel] [PATCH] build: Rename the LIB variable for testprog targets

2012-10-10 Thread Martin Storsjö
On Wed, 10 Oct 2012, Måns Rullgård wrote: Martin Storsjö writes: On MSVC, the LIB environment variable is used to specify the default paths for libraries. When this variable is set in the makefile, it overwrites the original lib path variable for the linker. --- This slipped me before, since

Re: [libav-devel] [PATCH 3/3] build: do not use LIB as variable name

2012-10-10 Thread Martin Storsjö
On Wed, 10 Oct 2012, Mans Rullgard wrote: The Microsoft linker uses the LIB environment variable which clashes with a make variable of the same name. Signed-off-by: Mans Rullgard --- library.mak | 6 +++--- 1 file changed, 3 insertions(+), 3 deletions(-) diff --git a/library.mak b/library.mak

[libav-devel] [PATCH] random_seed: Support using CryptGenRandom on windows

2012-10-11 Thread Martin Storsjö
--- configure |2 ++ libavutil/random_seed.c | 14 ++ 2 files changed, 16 insertions(+) diff --git a/configure b/configure index 17f38ec..67f88eb 100755 --- a/configure +++ b/configure @@ -1139,6 +1139,7 @@ HAVE_LIST=" closesocket cmov cpunop +Cr

Re: [libav-devel] [PATCH] Move avutil tables only used in libavcodec to libavcodec.

2012-10-11 Thread Martin Storsjö
On Thu, 11 Oct 2012, Diego Biurrun wrote: --- Fixed placement of extern table declarations (for arm), they need to come before the arch-specific headers, as they are referenced there. libavcodec/Makefile|7 + libavcodec/arm/mathops.h |

[libav-devel] [PATCH 1/3] avutil: Add functions for allocating opaque contexts for algorithms

2012-10-11 Thread Martin Storsjö
The current API where the plain size is exposed is not of much use - in most cases it is allocated dynamically anyway. If allocated e.g. on the stack via an uint8_t array, there's no guarantee that the struct's members are aligned properly (unless the array is overallocated and the opaque pointer

[libav-devel] [PATCH 2/3] Use the new aes/md5/sha/tree allocation functions

2012-10-11 Thread Martin Storsjö
--- libavcodec/flacenc.c|2 +- libavformat/crypto.c|2 +- libavformat/httpauth.c |2 +- libavformat/mxfdec.c|2 +- libavformat/nut.c |2 +- libavformat/rtmpproto.c |2 +- libavutil/tree.c|2 +- 7 files changed, 7 insertions(+), 7 deletions(-)

[libav-devel] [PATCH 3/3] md5: Allocate a normal private context for the opaque md5 context pointer

2012-10-11 Thread Martin Storsjö
This avoids having to overestimate the md5 context size, which isn't known beforehand, allowing us to use the new allocate functions instead. --- libavformat/md5enc.c | 55 +--- libavformat/md5proto.c | 24 + 2 files changed, 53

Re: [libav-devel] [PATCH 1/2] parseutils: fix const removal warning

2012-10-11 Thread Martin Storsjö
On Thu, 11 Oct 2012, Mans Rullgard wrote: The const qualifier is still removed although it happens inside the strtol() function so no warning is generated. Fixes: libavutil/parseutils.c:110:11: warning: assignment discards qualifiers from pointer target type Signed-off-by: Mans Rullgard ---

Re: [libav-devel] [PATCH] Move av_reverse table to libavcodec; it is only used there.

2012-10-11 Thread Martin Storsjö
On Thu, 11 Oct 2012, Diego Biurrun wrote: --- I have duplicated the table with an ff_ prefix and marked it as to be removed on the next libavutil bump. Alternatively, one could play tricks with the preprocessor and avoid the source-level duplication, but I thought this was not worth the trouble

Re: [libav-devel] [PATCH 7/9] lavc: do not use av_pix_fmt_descriptors directly.

2012-10-11 Thread Martin Storsjö
On Sat, 6 Oct 2012, Anton Khirnov wrote: --- libavcodec/imgconvert.c | 30 ++ libavcodec/libopenjpegdec.c | 34 ++ libavcodec/libopenjpegenc.c | 13 ++--- libavcodec/mpegvideo.c | 10 ++ libavcodec/rawdec.c

Re: [libav-devel] [PATCH 9/9] lavu, lavd: do not use av_pix_fmt_descriptors directly.

2012-10-11 Thread Martin Storsjö
On Sat, 6 Oct 2012, Anton Khirnov wrote: --- libavdevice/fbdev.c |2 +- libavutil/imgutils.c | 24 +++- 2 files changed, 16 insertions(+), 10 deletions(-) LGTM // Martin ___ libav-devel mailing list libav-devel@libav.org htt

Re: [libav-devel] [PATCH] Move av_reverse table to libavcodec; it is only used there.

2012-10-11 Thread Martin Storsjö
On Thu, 11 Oct 2012, Diego Biurrun wrote: --- Now w/o accidentally renaming the copy in libavutil... libavcodec/asvdec.c |7 --- libavcodec/asvenc.c |7 --- libavcodec/bitstream.c |9 + libavcodec/indeo2.c |6 -- libavcodec/ivi_common.c |7 --

Re: [libav-devel] [PATCH 3/3] rtpdec_xiph: fix function return type

2012-10-11 Thread Martin Storsjö
On Thu, 11 Oct 2012, Mans Rullgard wrote: parse_packed_headers() returns either zero or a negative error code so its return type must be signed. Signed-off-by: Mans Rullgard --- libavformat/rtpdec_xiph.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/libavformat/rtpdec_xiph

Re: [libav-devel] [PATCH 2/3] smjpeg: fix type of 'ret' variable in smjpeg_read_packet()

2012-10-11 Thread Martin Storsjö
On Thu, 11 Oct 2012, Mans Rullgard wrote: The 'ret' variable is used for negative error codes so it should be a signed type. Signed-off-by: Mans Rullgard --- libavformat/smjpegdec.c | 3 ++- 1 file changed, 2 insertions(+), 1 deletion(-) diff --git a/libavformat/smjpegdec.c b/libavformat/smjpe

Re: [libav-devel] [PATCH 1/3] mpegvideo: remove write-only variable

2012-10-11 Thread Martin Storsjö
On Thu, 11 Oct 2012, Mans Rullgard wrote: Signed-off-by: Mans Rullgard --- libavcodec/mpegvideo.c | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/libavcodec/mpegvideo.c b/libavcodec/mpegvideo.c index f739c2d..a0ec3dc 100644 --- a/libavcodec/mpegvideo.c +++ b/libavcodec/mp

Re: [libav-devel] [PATCH] Move av_reverse table to libavcodec; it is only used there.

2012-10-12 Thread Martin Storsjö
On Thu, 11 Oct 2012, Diego Biurrun wrote: --- Now w/o accidentally renaming the copy in libavutil... libavcodec/asvdec.c |7 --- libavcodec/asvenc.c |7 --- libavcodec/bitstream.c |9 + libavcodec/indeo2.c |6 -- libavcodec/ivi_common.c |7

[libav-devel] [PATCH 1/2] rtspdec: Fix use of uninitialized byte

2012-10-12 Thread Martin Storsjö
From: Michael Niedermayer ffurl_read_complete can return 0 as well as negative error codes. --- libavformat/rtspdec.c |4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/libavformat/rtspdec.c b/libavformat/rtspdec.c index 7476c45..d46f1a4 100644 --- a/libavformat/rtspdec.c

[libav-devel] [PATCH 2/2] rtmpproto: Fix an out of array write

2012-10-12 Thread Martin Storsjö
From: Michael Niedermayer --- libavformat/rtmpproto.c |4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/libavformat/rtmpproto.c b/libavformat/rtmpproto.c index a5bc246..3ab2e57 100644 --- a/libavformat/rtmpproto.c +++ b/libavformat/rtmpproto.c @@ -1371,9 +1371,9 @@ stati

[libav-devel] [PATCH] rtspdec: Set the default port for listen mode, if none is specified

2012-10-12 Thread Martin Storsjö
--- libavformat/rtspdec.c |4 1 file changed, 4 insertions(+) diff --git a/libavformat/rtspdec.c b/libavformat/rtspdec.c index 7476c45..c0797ad 100644 --- a/libavformat/rtspdec.c +++ b/libavformat/rtspdec.c @@ -621,6 +621,10 @@ static int rtsp_listen(AVFormatContext *s) /* ff_url_jo

[libav-devel] [PATCH] tscc2: Fix an out of array access

2012-10-12 Thread Martin Storsjö
From: Michael Niedermayer --- libavcodec/tscc2.c |2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/libavcodec/tscc2.c b/libavcodec/tscc2.c index 1b2a3a4..3628713 100644 --- a/libavcodec/tscc2.c +++ b/libavcodec/tscc2.c @@ -173,7 +173,7 @@ static int tscc2_decode_mb(TSCC2Cont

Re: [libav-devel] [PATCH 1/2] rtspdec: Fix use of uninitialized byte

2012-10-12 Thread Martin Storsjö
On Fri, 12 Oct 2012, Luca Barbato wrote: On 10/12/2012 11:20 AM, Luca Barbato wrote: I wonder why 0 should appear here, we are asking 1, we should get an eagain. Should but does not, patch ok I guess. Yeah - you don't get eagain for blocking reads. For the normal ffurl_read, you can get l

Re: [libav-devel] [PATCH] avutil: Duplicate ff_log2_tab instead of sharing it across libs

2012-10-12 Thread Martin Storsjö
On Wed, 10 Oct 2012, Diego Biurrun wrote: The table is so small that the space gain is not worth the performance overhead of cross-library access. --- Now dropping an instance into libavformat as well, which also accesses the table directly. libavcodec/Makefile |1 + libavcodec/log2_tab.

Re: [libav-devel] [PATCH] Move av_reverse table to libavcodec; it is only used there.

2012-10-12 Thread Martin Storsjö
On Fri, 12 Oct 2012, Diego Biurrun wrote: --- Now with attribute_deprecated and w/o stupid copy and paste bug. libavcodec/asvdec.c |7 --- libavcodec/asvenc.c |7 --- libavcodec/bitstream.c |9 + libavcodec/indeo2.c |6 -- libavcodec/ivi_common.c |

Re: [libav-devel] [PATCH 1/3] configure: tms470: add mapping for -mfpu=vfpv3-d16 flag

2012-10-13 Thread Martin Storsjö
On Sat, 13 Oct 2012, Mans Rullgard wrote: Signed-off-by: Mans Rullgard --- configure | 1 + 1 file changed, 1 insertion(+) diff --git a/configure b/configure index 2890ab2..f770ca7 100755 --- a/configure +++ b/configure @@ -2219,6 +2219,7 @@ tms470_flags(){ -mfpu=neon) echo --fl

Re: [libav-devel] [PATCH 2/3] configure: improve tms470 compiler usage with glibc

2012-10-13 Thread Martin Storsjö
On Sat, 13 Oct 2012, Mans Rullgard wrote: Apply flags to work around glibc quirks only if glibc is detected, and add a few more such flags. Do not mess with as/ld settings in probe_cc. This is not the proper place. Signed-off-by: Mans Rullgard --- configure | 12 +--- 1 file changed,

Re: [libav-devel] [PATCH 3/3] build: tms470: work around glibc math.h problems

2012-10-13 Thread Martin Storsjö
On Sat, 13 Oct 2012, Mans Rullgard wrote: The glibc definitions of INFINITY and NAN do not work with the tms470 compiler, nor do our usual fallbacks. Signed-off-by: Mans Rullgard --- compat/tms470/math.h | 7 +++ configure| 1 + 2 files changed, 8 insertions(+) create mode 100644

Re: [libav-devel] [PATCH] configure: separate option-arguments from options to POSIX commands

2012-10-13 Thread Martin Storsjö
On Sat, 13 Oct 2012, Mans Rullgard wrote: POSIX says this about option syntax: If the SYNOPSIS of a standard utility shows an option with a mandatory option-argument, a conforming application shall use separate arguments for that option and its option-argument. However, a conforming impleme

Re: [libav-devel] [PATCH 1/2] configure: work around bug in ash shell

2012-10-13 Thread Martin Storsjö
On Sat, 13 Oct 2012, Mans Rullgard wrote: The ash 'test' builtin misbehaves if the first operand of a binary operator looks like a unary operator. Signed-off-by: Mans Rullgard --- configure | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/configure b/configure index bc3a218..

Re: [libav-devel] [PATCH 2/2] configure: recognise Minix as OS

2012-10-13 Thread Martin Storsjö
On Sat, 13 Oct 2012, Mans Rullgard wrote: No special setup is required for Minix. Signed-off-by: Mans Rullgard --- configure | 2 ++ 1 file changed, 2 insertions(+) diff --git a/configure b/configure index b076945..66806bf 100755 --- a/configure +++ b/configure @@ -2877,6 +2877,8 @@ case $targ

[libav-devel] [PATCH] Add support for building shared libraries with MSVC

2012-10-13 Thread Martin Storsjö
From: "Ronald S. Bultje" This requires the makedef perl script by Derek, which probably will go into the c99-to-c89 repo. That scripts produces a .def file, listing the symbols to be exported, based on the gcc version scripts and the built object files. To properly load non-function symbols from

Re: [libav-devel] [PATCH] configure: separate option-arguments from options to POSIX commands

2012-10-13 Thread Martin Storsjö
On Sat, 13 Oct 2012, Måns Rullgård wrote: Martin Storsjö writes: On Sat, 13 Oct 2012, Mans Rullgard wrote: POSIX says this about option syntax: If the SYNOPSIS of a standard utility shows an option with a mandatory option-argument, a conforming application shall use separate arguments

Re: [libav-devel] [PATCH] Add support for building shared libraries with MSVC

2012-10-13 Thread Martin Storsjö
On Sat, 13 Oct 2012, Diego Biurrun wrote: On Sat, Oct 13, 2012 at 01:46:38PM +0300, Martin Storsjö wrote: --- a/configure +++ b/configure @@ -2778,14 +2778,33 @@ case $target_os in shlibdir_default="$bindir_default" SLIBPREF="" SLIBSUF=".dll

Re: [libav-devel] [PATCH 2/2] avutil: Give symbols used across libraries avpriv_ instead of ff_ prefixes

2012-10-13 Thread Martin Storsjö
On Thu, 4 Oct 2012, Diego Biurrun wrote: --- libavcodec/bmpenc.c |2 +- libavcodec/rawdec.c |2 +- libavcodec/utils.c |2 +- libavutil/imgutils.c |4 ++-- libavutil/imgutils.h |2 +- 5 files changed, 6 insertions(+), 6 deletions(-) Ok with me, this at least makes the current

Re: [libav-devel] [PATCH 3/4] avutil: Do not make ff_ symbols globally visible.

2012-10-13 Thread Martin Storsjö
On Thu, 4 Oct 2012, Diego Biurrun wrote: --- libavutil/libavutil.v |2 +- 1 files changed, 1 insertions(+), 1 deletions(-) diff --git a/libavutil/libavutil.v b/libavutil/libavutil.v index eb16ae1..e9f04cb 100644 --- a/libavutil/libavutil.v +++ b/libavutil/libavutil.v @@ -1,4 +1,4 @@ LIBAVUTI

Re: [libav-devel] [PATCH 1/2] configure: check for mprotect

2012-10-13 Thread Martin Storsjö
On Sat, 13 Oct 2012, Mans Rullgard wrote: Signed-off-by: Mans Rullgard --- configure | 2 ++ 1 file changed, 2 insertions(+) diff --git a/configure b/configure index 7be62fe..700e99d 100755 --- a/configure +++ b/configure @@ -1198,6 +1198,7 @@ HAVE_LIST=" mkstemp mm_empty mmap +

Re: [libav-devel] [PATCH 2/2] swscale: try to use mmap only if available

2012-10-13 Thread Martin Storsjö
On Sat, 13 Oct 2012, Mans Rullgard wrote: Some systems, e.g. Minix, have sys/mman.h defining MAP_ANONYMOUS without providing (working) mmap and friends. The mmx filter generation code checks only for MAP_ANONYMOUS, not for availability of mmap itself which leads to build errors on aforementione

[libav-devel] [PATCH 1/2] configure: Split out msvc as a separate target OS

2012-10-13 Thread Martin Storsjö
The name mingw32 as target OS is both misleading, and very little of the target OS specific settings actually match. Since the target OS default is set based on uname, the default (which on MSYS is set to mingw) is overridden by --toolchain=msvc. --- configure | 12 1 file changed,

[libav-devel] [PATCH 2/2] Add support for building shared libraries with MSVC

2012-10-13 Thread Martin Storsjö
From: "Ronald S. Bultje" This requires the makedef perl script by Derek, which probably will go into the c99-to-c89 repo. That scripts produces a .def file, listing the symbols to be exported, based on the gcc version scripts and the built object files. To properly load non-function symbols from

Re: [libav-devel] [PATCH 1/2] configure: Split out msvc as a separate target OS

2012-10-13 Thread Martin Storsjö
On Sat, 13 Oct 2012, Hendrik Leppkes wrote: On Sat, Oct 13, 2012 at 7:46 PM, Derek Buitenhuis wrote: On 13/10/2012 1:29 PM, Måns Rullgård wrote: I don't really like calling the OS msvc. That is the compiler. How about "windows"? By that logic, shouldn't we also call mingw32 "windows"?

Re: [libav-devel] [PATCH 1/2] configure: Split out msvc as a separate target OS

2012-10-13 Thread Martin Storsjö
On Sat, 13 Oct 2012, Hendrik Leppkes wrote: On Sat, Oct 13, 2012 at 8:14 PM, Måns Rullgård wrote: Hendrik Leppkes writes: On Sat, Oct 13, 2012 at 7:46 PM, Derek Buitenhuis wrote: On 13/10/2012 1:29 PM, Måns Rullgård wrote: I don't really like calling the OS msvc. That is the compiler.

Re: [libav-devel] [PATCH 2/2] Add support for building shared libraries with MSVC

2012-10-13 Thread Martin Storsjö
On Sat, 13 Oct 2012, Måns Rullgård wrote: Martin Storsjö writes: diff --git a/configure b/configure index 667089b..362f8d1 100755 --- a/configure +++ b/configure @@ -2794,6 +2794,29 @@ case $target_os in add_cppflags -U__STRICT_ANSI__ ;; msvc) +if enabled

<    1   2   3   4   5   6   7   8   9   10   >