Re: [libav-devel] [PATCH] swscale: remove unused xInc/srcW arguments from hScale().

2011-06-24 Thread Kostya
On Fri, Jun 24, 2011 at 09:49:42PM -0700, Ronald S. Bultje wrote: > --- > libswscale/ppc/swscale_altivec.c |3 +-- > libswscale/swscale.c | 11 +-- > libswscale/swscale_internal.h |6 ++ > libswscale/x86/swscale_template.c |3 +-- > 4 files changed, 9 in

[libav-devel] Commercial product violates GPL and distributes non-redistributable

2011-06-24 Thread The Informer
Hi, here's another violator of the GPL license, and the binaries distributed are even non-redistributable. The product is Ultimate File Viewer (http://www.ultimatefileviewer.com). Here's the config data from the distributed avcodec-52.dll: --prefix=/ffmpeg/Binaries --disable-debug --disable-sta

[libav-devel] [PATCH] swscale: remove unused xInc/srcW arguments from hScale().

2011-06-24 Thread Ronald S. Bultje
--- libswscale/ppc/swscale_altivec.c |3 +-- libswscale/swscale.c | 11 +-- libswscale/swscale_internal.h |6 ++ libswscale/x86/swscale_template.c |3 +-- 4 files changed, 9 insertions(+), 14 deletions(-) diff --git a/libswscale/ppc/swscale_altivec.c b/

Re: [libav-devel] [PATCH] aacenc: Mark psy_3gpp_window() as av_unused.

2011-06-24 Thread Nathan Caldwell
On Fri, Jun 24, 2011 at 3:48 AM, Diego Biurrun wrote: > It is intentionally left in to allow adding 3GPP-style windowing in the > future. > Marking it av_unused silences an annoying unused function warning. > --- >  libavcodec/aacpsy.c |    7 --- >  1 files changed, 4 insertions(+), 3 deletio

[libav-devel] [PATCH] build: Add DEP_LIBS dependency directly to the shared library build rule.

2011-06-24 Thread Diego Biurrun
The dependency was added conditional to a variable that is always defined, so it is safe to add it directly. --- subdir.mak |6 +- 1 files changed, 1 insertions(+), 5 deletions(-) diff --git a/subdir.mak b/subdir.mak index 6339860..e4d7377 100644 --- a/subdir.mak +++ b/subdir.mak @@ -39,1

Re: [libav-devel] [PATCH] mpegts: do not assemble incomplete PES

2011-06-24 Thread Luca Barbato
On 6/24/11 10:00 AM, Måns Rullgård wrote: That would depend on the decoder. Indeed. in the cases I had reports for (the patch had been in my github for a while) h264 was the only codec considered. lu ___ libav-devel mailing list libav-devel@libav.o

[libav-devel] [PATCH 2/2] build: Remove multiple inclusion guards from config.mak.

2011-06-24 Thread Diego Biurrun
config.mak is no longer included multiple times; they are pointless. --- configure |3 --- 1 files changed, 0 insertions(+), 3 deletions(-) diff --git a/configure b/configure index e20bb06..41e1a4d 100755 --- a/configure +++ b/configure @@ -3248,8 +3248,6 @@ config_files="$TMPH config.mak"

[libav-devel] [PATCH 1/2] build: Remove redundant config.mak subdirectory Makefile includes.

2011-06-24 Thread Diego Biurrun
Calling Make from subdirectories is not supported and config.mak has multiple inclusion guards anyway, so the top-level include is enough. --- libavcodec/Makefile |2 -- libavdevice/Makefile |2 -- libavfilter/Makefile |2 -- libavformat/Makefile |2 -- libavutil/Makefile |2

Re: [libav-devel] [PATCH] configure: Add -Wno-format-zero-length to CFLAGS.

2011-06-24 Thread Måns Rullgård
Diego Biurrun writes: > The C standard specifies that zero-length format strings are allowed. > --- > configure |1 + > 1 files changed, 1 insertions(+), 0 deletions(-) > > diff --git a/configure b/configure > index f557482..e20bb06 100755 > --- a/configure > +++ b/configure > @@ -2978,6 +29

Re: [libav-devel] [PATCH] configure: Add -Wno-format-zero-length to CFLAGS.

2011-06-24 Thread Måns Rullgård
Diego Biurrun writes: > The C standard specifies that zero-length format strings are allowed. > --- > configure |1 + > 1 files changed, 1 insertions(+), 0 deletions(-) > > diff --git a/configure b/configure > index f557482..e20bb06 100755 > --- a/configure > +++ b/configure > @@ -2978,6 +29

Re: [libav-devel] [PATCH] configure: Add -Wno-format-zero-length to CFLAGS.

2011-06-24 Thread Justin Ruggles
On 06/24/2011 03:21 PM, Diego Biurrun wrote: > The C standard specifies that zero-length format strings are allowed. > --- > configure |1 + > 1 files changed, 1 insertions(+), 0 deletions(-) > > diff --git a/configure b/configure > index f557482..e20bb06 100755 > --- a/configure > +++ b/con

[libav-devel] [PATCH] configure: Add -Wno-format-zero-length to CFLAGS.

2011-06-24 Thread Diego Biurrun
The C standard specifies that zero-length format strings are allowed. --- configure |1 + 1 files changed, 1 insertions(+), 0 deletions(-) diff --git a/configure b/configure index f557482..e20bb06 100755 --- a/configure +++ b/configure @@ -2978,6 +2978,7 @@ check_cflags -Wdeclaration-after-st

Re: [libav-devel] [PATCH] ffmpeg/ffplay: Fix zero-length gnu_printf format string warnings.

2011-06-24 Thread Justin Ruggles
On 06/21/2011 04:30 PM, Diego Biurrun wrote: > From: Clément Bœsch > > Signed-off-by: Diego Biurrun > --- > ffmpeg.c|2 +- > ffplay.c|2 +- > libavutil/log.h |2 +- > 3 files changed, 3 insertions(+), 3 deletions(-) > > diff --git a/ffmpeg.c b/ffmpeg.c > index 9f2d

Re: [libav-devel] [PATCH 2/2] swscale: implement >8bit scaling support.

2011-06-24 Thread Diego Biurrun
On Fri, Jun 24, 2011 at 07:43:20AM -0700, Ronald S. Bultje wrote: > This means that precision is retained when scaling between sample > formats with >8 bits per component (48bit RGB, 16bit grayscale, > 9/10/16bit YUV). Also document hscale functions. This patch is lrge and could possibly be sp

Re: [libav-devel] [PATCH 2/2] swscale: implement >8bit scaling support.

2011-06-24 Thread Kostya
On Fri, Jun 24, 2011 at 07:43:20AM -0700, Ronald S. Bultje wrote: > This means that precision is retained when scaling between sample > formats with >8 bits per component (48bit RGB, 16bit grayscale, > 9/10/16bit YUV). Also document hscale functions. > > Known problems: > - 1:1 scaling path (unsca

Re: [libav-devel] [PATCH 1/2] swscale: re-add support for non-native endianness.

2011-06-24 Thread Kostya
On Fri, Jun 24, 2011 at 07:43:19AM -0700, Ronald S. Bultje wrote: > This works through some non-obvious hacks in utils.c. > --- > libswscale/swscale.c | 18 -- > libswscale/utils.c | 18 -- > tests/ref/lavfi/pixdesc_le |4

Re: [libav-devel] [PATCH] udp: Receive on the remote port number, if no local port is set

2011-06-24 Thread Ronald S. Bultje
Hi, On Fri, Jun 17, 2011 at 12:47 AM, Martin Storsjö wrote: > From: Can Wu > > For a unicast udp stream to localhost:1234, currently > ffplay udp://:1234?localport=1234 > works, but > ffplay udp://:1234 > doesn't work. > --- >  libavformat/udp.c |    2 +- >  1 files changed, 1 insertions(+), 1 d

Re: [libav-devel] [PATCH] mpegts: do not assemble incomplete PES

2011-06-24 Thread Måns Rullgård
Luca Barbato writes: > On 6/24/11 4:38 AM, Måns Rullgård wrote: >> Luca Barbato writes: >> >>> If the Continuity Check fails drop the whole packet >> >> That's a bit harsh. There may still be useful data in the parts present. >> Consider the situation of a lost TS packet near the end of a frame

Re: [libav-devel] [PATCH] lavf: don't error on custom AVIOContex for an AVFMT_NOFILE format.

2011-06-24 Thread Luca Barbato
On 6/24/11 4:46 AM, Anton Khirnov wrote: On Fri, 24 Jun 2011 10:34:54 +0100, Måns Rullgård wrote: Luca Barbato writes: On 6/23/11 2:52 AM, Måns Rullgård wrote: If this is only to please *old* mplayer versions, it does not make sense. I'd consider it an example of a possible behaviour let

[libav-devel] [PATCH 1/2] swscale: re-add support for non-native endianness.

2011-06-24 Thread Ronald S. Bultje
This works through some non-obvious hacks in utils.c. --- libswscale/swscale.c | 18 -- libswscale/utils.c | 18 -- tests/ref/lavfi/pixdesc_le |4 tests/ref/lavfi/pixfmts_copy_le |4 tests/ref/lavfi/pixfmts_crop

Re: [libav-devel] [PATCH] mpegts: do not assemble incomplete PES

2011-06-24 Thread Luca Barbato
On 6/24/11 4:38 AM, Måns Rullgård wrote: Luca Barbato writes: If the Continuity Check fails drop the whole packet That's a bit harsh. There may still be useful data in the parts present. Consider the situation of a lost TS packet near the end of a frame (not necessarily end of a PES packet)

Re: [libav-devel] [PATCH] mpegts: do not assemble incomplete PES

2011-06-24 Thread Gil Pedersen
On 24/06/2011, at 09.33, Luca Barbato wrote: > If the Continuity Check fails drop the whole packet I can't see the direct consequences of this change, but you should at least remove the use of cc_ok in the remainder of the function. /Gil ___ libav-dev

[libav-devel] [PATCH] aacenc: Mark psy_3gpp_window() as av_unused.

2011-06-24 Thread Diego Biurrun
It is intentionally left in to allow adding 3GPP-style windowing in the future. Marking it av_unused silences an annoying unused function warning. --- libavcodec/aacpsy.c |7 --- 1 files changed, 4 insertions(+), 3 deletions(-) diff --git a/libavcodec/aacpsy.c b/libavcodec/aacpsy.c index

Re: [libav-devel] [PATCH] lavf: don't error on custom AVIOContex for an AVFMT_NOFILE format.

2011-06-24 Thread Anton Khirnov
On Fri, 24 Jun 2011 10:34:54 +0100, Måns Rullgård wrote: > Luca Barbato writes: > > > On 6/23/11 2:52 AM, Måns Rullgård wrote: > >> If this is only to please *old* mplayer versions, it does not make sense. > > > > I'd consider it an example of a possible behaviour let's be strict in > > the 0.8

Re: [libav-devel] [PATCH] mpegts: do not assemble incomplete PES

2011-06-24 Thread Måns Rullgård
Luca Barbato writes: > If the Continuity Check fails drop the whole packet That's a bit harsh. There may still be useful data in the parts present. Consider the situation of a lost TS packet near the end of a frame (not necessarily end of a PES packet). The decoder will probably skip to the ne

Re: [libav-devel] [PATCH] lavf: don't error on custom AVIOContex for an AVFMT_NOFILE format.

2011-06-24 Thread Måns Rullgård
Luca Barbato writes: > On 6/23/11 2:52 AM, Måns Rullgård wrote: >> If this is only to please *old* mplayer versions, it does not make sense. > > I'd consider it an example of a possible behaviour let's be strict in > the 0.8 about it. Did we change recently? If not, I see no reason to become le

Re: [libav-devel] [PATCH] Darwin/MacOS X: allow user-specified library install path

2011-06-24 Thread Måns Rullgård
Sean McGovern writes: > In Darwin/MacOS X, it is advantageous to be able to set a > custom library install path when embedding a library alongside or > in a path relative to it's calling binary application > This allows constructions such as: > @executable_path/lib > which is evaluated by dyld at

Re: [libav-devel] [PATCH] mpegts: do not assemble incomplete PES

2011-06-24 Thread Reinhard Tartler
On Fri, Jun 24, 2011 at 09:33:57 (CEST), Luca Barbato wrote: > If the Continuity Check fails drop the whole packet > --- > libavformat/mpegts.c |5 - > 1 files changed, 4 insertions(+), 1 deletions(-) > > diff --git a/libavformat/mpegts.c b/libavformat/mpegts.c > index f2ae567..05bc4dc 10

[libav-devel] [PATCH] mpegts: do not assemble incomplete PES

2011-06-24 Thread Luca Barbato
If the Continuity Check fails drop the whole packet --- libavformat/mpegts.c |5 - 1 files changed, 4 insertions(+), 1 deletions(-) diff --git a/libavformat/mpegts.c b/libavformat/mpegts.c index f2ae567..05bc4dc 100644 --- a/libavformat/mpegts.c +++ b/libavformat/mpegts.c @@ -1267,7 +1267