[libav-devel] [PATCH 2/2] avconv: remove now unneeded calls to avcodec_get_frame_defaults().

2012-10-22 Thread Anton Khirnov
--- avconv.c |4 1 file changed, 4 deletions(-) diff --git a/avconv.c b/avconv.c index 759e1c0..3188d61 100644 --- a/avconv.c +++ b/avconv.c @@ -1078,8 +1078,6 @@ static int decode_audio(InputStream *ist, AVPacket *pkt, int *got_output) if (!ist->decoded_frame && !(ist->decoded_f

[libav-devel] [PATCH 1/2] lavc: initialize output AVFrame before decoding.

2012-10-22 Thread Anton Khirnov
Avoids memleaks with audio when extended_data is nontrivial and the user doesn't reset the frame. Shouldn't have any effect for video for now, but might be useful in the future. --- libavcodec/utils.c |4 1 file changed, 4 insertions(+) diff --git a/libavcodec/utils.c b/libavcodec/utils.

Re: [libav-devel] [PATCH] Announce 0.8.4 Release

2012-10-22 Thread Reinhard Tartler
On Mon, Oct 22, 2012 at 11:49 PM, Diego Biurrun wrote: > On Mon, Oct 22, 2012 at 09:58:19PM +0200, Reinhard Tartler wrote: >> --- a/src/news >> +++ b/src/news >> @@ -1,5 +1,66 @@ >> + >> + >> +The following bug in our Bugzilla has been fixed: >> + >> +http://bugzilla.libav.org/show_bug.cgi?id=118"

[libav-devel] [PATCH 2/2] fate: check that dependencies actually exist

2012-10-22 Thread Mans Rullgard
This causes make to exit with an error message if a nonexistent dependency is specified rather than silently dropping the test. Signed-off-by: Mans Rullgard --- tests/Makefile | 4 +++- 1 file changed, 3 insertions(+), 1 deletion(-) diff --git a/tests/Makefile b/tests/Makefile index de4d928..26

[libav-devel] [PATCH 1/2] fate: fix ENCMUX macro

2012-10-22 Thread Mans Rullgard
This matches how it is actually being called. Signed-off-by: Mans Rullgard --- tests/Makefile | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/tests/Makefile b/tests/Makefile index 93eb548..de4d928 100644 --- a/tests/Makefile +++ b/tests/Makefile @@ -34,7 +34,7 @@ ENCDEC2 = $(

[libav-devel] [PATCH v2] build: Plan 9 support

2012-10-22 Thread Mans Rullgard
This adds support for building on Plan 9 x86-32. The compat/plan9 directory contains these items: - replacements for the 'head' and 'printf' shell commands - wrapper for main() to disable FPU exceptions Larger required changes to the system are described in the documentation. Signed-off-by: Man

Re: [libav-devel] [PATCH] mov: Do not apply dts shift from edit lists coming from data tracks.

2012-10-22 Thread Alex Converse
On Fri, Oct 12, 2012 at 4:07 AM, Måns Rullgård wrote: > > Alex Converse writes: > > > Some files in the wild have time code tracks with very negative initial > > offsets. > > Do you have a sample? > I can't share the samples I have. I tried to make one but I don't think I have a tool that can ge

Re: [libav-devel] [PATCH] Announce 0.8.4 Release

2012-10-22 Thread Diego Biurrun
On Mon, Oct 22, 2012 at 09:58:19PM +0200, Reinhard Tartler wrote: > --- a/src/news > +++ b/src/news > @@ -1,5 +1,66 @@ > + > + > +The following bug in our Bugzilla has been fixed: > + > +http://bugzilla.libav.org/show_bug.cgi?id=118";>#118: > zzufed H.261 file crashes avconv signal 11 (SIGSEGV) >

Re: [libav-devel] ffv1.3 support - "Unrecognized option 'slicecrc'"

2012-10-22 Thread Luca Barbato
On 10/20/2012 12:25 PM, Peter B. wrote: > On 10/20/2012 12:15 AM, Peter B. wrote: >> On 10/19/2012 08:30 PM, Luca Barbato wrote: >> >>> slicecrc should be supported. >> The version I've compiled now does. >> Before it was complaining that the commandline argument "slicecrc" is >> unknown. >> > Sor

Re: [libav-devel] [PATCH 1/2] mp3: exit on parsing error in mp_decode_frame

2012-10-22 Thread Luca Barbato
On 10/22/2012 08:02 PM, Diego Biurrun wrote: > On Mon, Oct 22, 2012 at 07:42:22PM +0200, Luca Barbato wrote: >> On 10/22/2012 07:30 PM, Diego Biurrun wrote: >>> On Mon, Oct 22, 2012 at 07:13:28PM +0200, Luca Barbato wrote: Properly forward mp_decode_layer3 errors. >>> >>> mp3: Properly fo

Re: [libav-devel] [PATCH] avconv: only apply presets when we have an encoder.

2012-10-22 Thread Luca Barbato
On 10/22/2012 10:42 PM, Anton Khirnov wrote: > Fixes a crash when using a preset with stream copy. > Ok. lu ___ libav-devel mailing list libav-devel@libav.org https://lists.libav.org/mailman/listinfo/libav-devel

[libav-devel] [PATCH] avconv: only apply presets when we have an encoder.

2012-10-22 Thread Anton Khirnov
Fixes a crash when using a preset with stream copy. CC: libav-sta...@libav.org --- avconv_opt.c | 55 --- 1 file changed, 28 insertions(+), 27 deletions(-) diff --git a/avconv_opt.c b/avconv_opt.c index 79444eb..0eb601b 100644 --- a/avconv_op

Re: [libav-devel] [PATCH] Announce 0.8.4 Release

2012-10-22 Thread Luca Barbato
On 10/22/2012 09:58 PM, Reinhard Tartler wrote: > --- > src/download | 22 ++--- > src/news | 61 > ++ > 2 files changed, 72 insertions(+), 11 deletions(-) > Looks fine. lu

[libav-devel] [PATCH] Announce 0.8.4 Release

2012-10-22 Thread Reinhard Tartler
--- src/download | 22 ++--- src/news | 61 ++ 2 files changed, 72 insertions(+), 11 deletions(-) diff --git a/src/download b/src/download index 9039cbd..9a53b21 100644 --- a/src/download +++ b/src/download @@ -225,9

Re: [libav-devel] [PATCH 2/2] avutil: Move memcpy_backptr() to mem.c

2012-10-22 Thread Måns Rullgård
Diego Biurrun writes: > On Mon, Oct 22, 2012 at 11:43:00AM -0700, Diego Elio Pettenò wrote: >> On 22/10/2012 10:18, Diego Biurrun wrote: >> > -#include "libavutil/lzo.h" // for av_memcpy_backptr >> > +#include "libavutil/lzo.h" >> > +#include "libavutil/mem.h" >> >> Is lzo.h still required here?

Re: [libav-devel] [PATCH 2/2] avutil: Move memcpy_backptr() to mem.c

2012-10-22 Thread Diego Biurrun
On Mon, Oct 22, 2012 at 11:43:00AM -0700, Diego Elio Pettenò wrote: > On 22/10/2012 10:18, Diego Biurrun wrote: > > -#include "libavutil/lzo.h" // for av_memcpy_backptr > > +#include "libavutil/lzo.h" > > +#include "libavutil/mem.h" > > Is lzo.h still required here? Yes, for AV_LZO_OUTPUT_PADDING

Re: [libav-devel] [PATCH 2/2] avutil: Move memcpy_backptr() to mem.c

2012-10-22 Thread Diego Elio Pettenò
On 22/10/2012 10:18, Diego Biurrun wrote: > -#include "libavutil/lzo.h" // for av_memcpy_backptr > +#include "libavutil/lzo.h" > +#include "libavutil/mem.h" Is lzo.h still required here? -- Diego Elio Pettenò — Flameeyes flamee...@flameeyes.eu — http://blog.flameeyes.eu/

[libav-devel] [PATCH] lzo: Use AV_WN*/AV_RN* where appropriate

2012-10-22 Thread Diego Biurrun
--- libavutil/lzo.c | 28 1 files changed, 8 insertions(+), 20 deletions(-) diff --git a/libavutil/lzo.c b/libavutil/lzo.c index 0d1ec55..cbb 100644 --- a/libavutil/lzo.c +++ b/libavutil/lzo.c @@ -23,6 +23,7 @@ #include "avutil.h" #include "common.h" +#inclu

Re: [libav-devel] [PATCH 1/2] mp3: exit on parsing error in mp_decode_frame

2012-10-22 Thread Diego Biurrun
On Mon, Oct 22, 2012 at 07:42:22PM +0200, Luca Barbato wrote: > On 10/22/2012 07:30 PM, Diego Biurrun wrote: > > On Mon, Oct 22, 2012 at 07:13:28PM +0200, Luca Barbato wrote: > >> > >> Properly forward mp_decode_layer3 errors. > > > > mp3: Properly forward mp_decode_layer3 parsing errors in mp_dec

Re: [libav-devel] [PATCH 1/2] mp3: exit on parsing error in mp_decode_frame

2012-10-22 Thread Måns Rullgård
Luca Barbato writes: > On 10/22/2012 07:30 PM, Diego Biurrun wrote: >> On Mon, Oct 22, 2012 at 07:13:28PM +0200, Luca Barbato wrote: >>> >>> Properly forward mp_decode_layer3 errors. >> >> mp3: Properly forward mp_decode_layer3 parsing errors in mp_decode_frame > > Anything longer than 70 col ma

Re: [libav-devel] [PATCH 1/2] mp3: exit on parsing error in mp_decode_frame

2012-10-22 Thread Luca Barbato
On 10/22/2012 07:30 PM, Diego Biurrun wrote: > On Mon, Oct 22, 2012 at 07:13:28PM +0200, Luca Barbato wrote: >> >> Properly forward mp_decode_layer3 errors. > > mp3: Properly forward mp_decode_layer3 parsing errors in mp_decode_frame Anything longer than 70 col makes miserable the life of those u

Re: [libav-devel] [PATCH 1/2] lzo: Use AV_COPY* where appropriate

2012-10-22 Thread Måns Rullgård
Diego Biurrun writes: > --- > libavutil/lzo.c | 28 > 1 files changed, 8 insertions(+), 20 deletions(-) > > diff --git a/libavutil/lzo.c b/libavutil/lzo.c > index 0d1ec55..a7fddc5 100644 > --- a/libavutil/lzo.c > +++ b/libavutil/lzo.c > @@ -23,6 +23,7 @@ > > #in

Re: [libav-devel] [PATCH 2/2] mp3: properly forward mp_decode_frame errors

2012-10-22 Thread Diego Biurrun
On Mon, Oct 22, 2012 at 07:13:29PM +0200, Luca Barbato wrote: > The function can return either a parsing error or a memory management > error. > --- > libavcodec/mpegaudiodec.c | 32 ++-- > 1 file changed, 18 insertions(+), 14 deletions(-) LGTM Diego _

Re: [libav-devel] [PATCH 1/2] mp3: exit on parsing error in mp_decode_frame

2012-10-22 Thread Diego Biurrun
On Mon, Oct 22, 2012 at 07:13:28PM +0200, Luca Barbato wrote: > > Properly forward mp_decode_layer3 errors. mp3: Properly forward mp_decode_layer3 parsing errors in mp_decode_frame > --- a/libavcodec/mpegaudiodec.c > +++ b/libavcodec/mpegaudiodec.c > @@ -1571,6 +1571,9 @@ static int mp_decode_fr

[libav-devel] [PATCH 2/2] avutil: Move memcpy_backptr() to mem.c

2012-10-22 Thread Diego Biurrun
The function is used elsewhere and does not belong with the LZO code. --- libavcodec/dfa.c |3 ++- libavcodec/eatgv.c|2 +- libavcodec/g723_1.c |2 +- libavcodec/lcldec.c |1 + libavcodec/wmavoice.c |2 +- libavcodec/xan.c |3 +-- libavcodec/xxan.c |

[libav-devel] [PATCH 1/2] lzo: Use AV_COPY* where appropriate

2012-10-22 Thread Diego Biurrun
--- libavutil/lzo.c | 28 1 files changed, 8 insertions(+), 20 deletions(-) diff --git a/libavutil/lzo.c b/libavutil/lzo.c index 0d1ec55..a7fddc5 100644 --- a/libavutil/lzo.c +++ b/libavutil/lzo.c @@ -23,6 +23,7 @@ #include "avutil.h" #include "common.h" +#inclu

[libav-devel] [PATCH 2/2] mp3: properly forward mp_decode_frame errors

2012-10-22 Thread Luca Barbato
The function can return either a parsing error or a memory management error. --- libavcodec/mpegaudiodec.c | 32 ++-- 1 file changed, 18 insertions(+), 14 deletions(-) diff --git a/libavcodec/mpegaudiodec.c b/libavcodec/mpegaudiodec.c index 5265ff8..5282235 100644 ---

[libav-devel] [PATCH 1/2] mp3: exit on parsing error in mp_decode_frame

2012-10-22 Thread Luca Barbato
From: Michael Niedermayer Properly forward mp_decode_layer3 errors. --- libavcodec/mpegaudiodec.c | 3 +++ 1 file changed, 3 insertions(+) diff --git a/libavcodec/mpegaudiodec.c b/libavcodec/mpegaudiodec.c index 80acdad..5265ff8 100644 --- a/libavcodec/mpegaudiodec.c +++ b/libavcodec/mpegaudiod

Re: [libav-devel] [PATCH] atrac3: return an error if extradata_size is not a specific known size

2012-10-22 Thread Kostya Shishkov
On Mon, Oct 22, 2012 at 12:03:04PM -0400, Justin Ruggles wrote: > Also fixes 3 compiler warnings about using uninitialized variables. > --- > libavcodec/atrac3.c |1 + > 1 files changed, 1 insertions(+), 0 deletions(-) > > diff --git a/libavcodec/atrac3.c b/libavcodec/atrac3.c > index 9cf9892

[libav-devel] [PATCH] atrac3: return an error if extradata_size is not a specific known size

2012-10-22 Thread Justin Ruggles
Also fixes 3 compiler warnings about using uninitialized variables. --- libavcodec/atrac3.c |1 + 1 files changed, 1 insertions(+), 0 deletions(-) diff --git a/libavcodec/atrac3.c b/libavcodec/atrac3.c index 9cf9892..5bf992f 100644 --- a/libavcodec/atrac3.c +++ b/libavcodec/atrac3.c @@ -912,6

Re: [libav-devel] [PATCH] mpegaudio: Refactor mp3on4 flush function

2012-10-22 Thread Justin Ruggles
On 10/22/2012 11:15 AM, Diego Biurrun wrote: > --- > libavcodec/mpegaudiodec.c | 17 + > 1 files changed, 9 insertions(+), 8 deletions(-) > > diff --git a/libavcodec/mpegaudiodec.c b/libavcodec/mpegaudiodec.c > index 80acdad..9c5ef26 100644 > --- a/libavcodec/mpegaudiodec.c > ++

[libav-devel] [PATCH] fate: aac: Place LATM tests and general AAC tests in different groups

2012-10-22 Thread Diego Biurrun
--- tests/fate/aac.mak | 17 +++-- 1 files changed, 11 insertions(+), 6 deletions(-) diff --git a/tests/fate/aac.mak b/tests/fate/aac.mak index 9d52dae..0368798 100644 --- a/tests/fate/aac.mak +++ b/tests/fate/aac.mak @@ -46,7 +46,7 @@ FATE_AAC += fate-aac-al_sbr_ps_06_ur fate-aac-

[libav-devel] [PATCH] fate: Add dependencies for aac, alac, amrnb, amrwb, atrac tests

2012-10-22 Thread Diego Biurrun
--- tests/fate/aac.mak | 15 +-- tests/fate/alac.mak |2 +- tests/fate/amrnb.mak |2 +- tests/fate/amrwb.mak |2 +- tests/fate/atrac.mak | 12 +++- 5 files changed, 19 insertions(+), 14 deletions(-) diff --git a/tests/fate/aac.mak b/tests/fate/aac.mak index

Re: [libav-devel] [PATCH] fate: Add dependencies for audio tests

2012-10-22 Thread Måns Rullgård
Diego Biurrun writes: > --- > tests/fate/audio.mak | 26 +- > 1 files changed, 13 insertions(+), 13 deletions(-) LGTM, did not verify component names. -- Måns Rullgård m...@mansr.com ___ libav-devel mailing list libav-devel

[libav-devel] [PATCH] fate: Add dependencies for audio tests

2012-10-22 Thread Diego Biurrun
--- tests/fate/audio.mak | 26 +- 1 files changed, 13 insertions(+), 13 deletions(-) diff --git a/tests/fate/audio.mak b/tests/fate/audio.mak index 86f78dd..7e08c2a 100644 --- a/tests/fate/audio.mak +++ b/tests/fate/audio.mak @@ -1,44 +1,44 @@ -FATE_BINKAUDIO += fate-bin

Re: [libav-devel] [PATCH] fate: aac: Place LATM tests and general AAC tests in different groups

2012-10-22 Thread Måns Rullgård
Diego Biurrun writes: > --- > tests/fate/aac.mak | 15 +-- > 1 files changed, 9 insertions(+), 6 deletions(-) > > diff --git a/tests/fate/aac.mak b/tests/fate/aac.mak > index 9d52dae..ddd9fe7 100644 > --- a/tests/fate/aac.mak > +++ b/tests/fate/aac.mak > @@ -46,7 +46,7 @@ FATE_AAC

Re: [libav-devel] [PATCH 2/2] fate: More fine-grained dependencies for libavcodec test programs

2012-10-22 Thread Måns Rullgård
Diego Biurrun writes: > --- > tests/fate/libavcodec.mak | 10 +- > 1 files changed, 5 insertions(+), 5 deletions(-) > > diff --git a/tests/fate/libavcodec.mak b/tests/fate/libavcodec.mak > index cba5eed..e0c565b 100644 > --- a/tests/fate/libavcodec.mak > +++ b/tests/fate/libavcodec.mak

[libav-devel] [PATCH] fate: aac: Place LATM tests and general AAC tests in different groups

2012-10-22 Thread Diego Biurrun
--- tests/fate/aac.mak | 15 +-- 1 files changed, 9 insertions(+), 6 deletions(-) diff --git a/tests/fate/aac.mak b/tests/fate/aac.mak index 9d52dae..ddd9fe7 100644 --- a/tests/fate/aac.mak +++ b/tests/fate/aac.mak @@ -46,7 +46,7 @@ FATE_AAC += fate-aac-al_sbr_ps_06_ur fate-aac-al_

Re: [libav-devel] [PATCH 1/2] fate: Declare avcodec/avformat deps in the respective Makefile snippets

2012-10-22 Thread Måns Rullgård
Diego Biurrun writes: > --- > tests/Makefile |3 --- > tests/fate/libavcodec.mak |1 + > tests/fate/libavformat.mak |1 + > 3 files changed, 2 insertions(+), 3 deletions(-) > > diff --git a/tests/Makefile b/tests/Makefile > index 1ef7a5e..1ec9b1a 100644 > --- a/tests/Mak

[libav-devel] [PATCH 1/2] fate: Declare avcodec/avformat deps in the respective Makefile snippets

2012-10-22 Thread Diego Biurrun
--- tests/Makefile |3 --- tests/fate/libavcodec.mak |1 + tests/fate/libavformat.mak |1 + 3 files changed, 2 insertions(+), 3 deletions(-) diff --git a/tests/Makefile b/tests/Makefile index 1ef7a5e..1ec9b1a 100644 --- a/tests/Makefile +++ b/tests/Makefile @@ -94,9 +94,6

[libav-devel] [PATCH 2/2] fate: More fine-grained dependencies for libavcodec test programs

2012-10-22 Thread Diego Biurrun
--- tests/fate/libavcodec.mak | 10 +- 1 files changed, 5 insertions(+), 5 deletions(-) diff --git a/tests/fate/libavcodec.mak b/tests/fate/libavcodec.mak index cba5eed..e0c565b 100644 --- a/tests/fate/libavcodec.mak +++ b/tests/fate/libavcodec.mak @@ -1,16 +1,16 @@ -FATE_LIBAVCODEC +=

Re: [libav-devel] [PATCH 1/2] mp3dec: Propagate error code from mp_decode_layer*() in mp_decode_frame()

2012-10-22 Thread Diego Biurrun
On Mon, Oct 22, 2012 at 05:08:52PM +0200, Luca Barbato wrote: > On 10/22/2012 05:02 PM, Diego Biurrun wrote: > > From: Michael Niedermayer > > > > --- > > libavcodec/mpegaudiodec.c |3 +++ > > 1 files changed, 3 insertions(+), 0 deletions(-) > > > > diff --git a/libavcodec/mpegaudiodec.c b/

[libav-devel] [PATCH] mpegaudio: Refactor mp3on4 flush function

2012-10-22 Thread Diego Biurrun
--- libavcodec/mpegaudiodec.c | 17 + 1 files changed, 9 insertions(+), 8 deletions(-) diff --git a/libavcodec/mpegaudiodec.c b/libavcodec/mpegaudiodec.c index 80acdad..9c5ef26 100644 --- a/libavcodec/mpegaudiodec.c +++ b/libavcodec/mpegaudiodec.c @@ -1681,11 +1681,15 @@ static

Re: [libav-devel] [PATCH 2/2] mpegaudiodec: Fix buffer handling on random access

2012-10-22 Thread Luca Barbato
On 10/22/2012 05:02 PM, Diego Biurrun wrote: > From: Michael Niedermayer > > --- > This fixes artifacts when seeking in > http://commondatastorage.googleapis.com/dalecurtis-shared/ironman2.mp3 > > libavcodec/mpegaudiodec.c |1 + > 1 files changed, 1 insertions(+), 0 deletions(-) > > diff -

Re: [libav-devel] [PATCH 1/2] mp3dec: Propagate error code from mp_decode_layer*() in mp_decode_frame()

2012-10-22 Thread Luca Barbato
On 10/22/2012 05:02 PM, Diego Biurrun wrote: > From: Michael Niedermayer > > --- > libavcodec/mpegaudiodec.c |3 +++ > 1 files changed, 3 insertions(+), 0 deletions(-) > > diff --git a/libavcodec/mpegaudiodec.c b/libavcodec/mpegaudiodec.c > index 9c5ef26..572f074 100644 > --- a/libavcodec/m

[libav-devel] [PATCH 1/2] mp3dec: Propagate error code from mp_decode_layer*() in mp_decode_frame()

2012-10-22 Thread Diego Biurrun
From: Michael Niedermayer --- libavcodec/mpegaudiodec.c |3 +++ 1 files changed, 3 insertions(+), 0 deletions(-) diff --git a/libavcodec/mpegaudiodec.c b/libavcodec/mpegaudiodec.c index 9c5ef26..572f074 100644 --- a/libavcodec/mpegaudiodec.c +++ b/libavcodec/mpegaudiodec.c @@ -1598,6 +1598,

[libav-devel] [PATCH 2/2] mpegaudiodec: Fix buffer handling on random access

2012-10-22 Thread Diego Biurrun
From: Michael Niedermayer --- This fixes artifacts when seeking in http://commondatastorage.googleapis.com/dalecurtis-shared/ironman2.mp3 libavcodec/mpegaudiodec.c |1 + 1 files changed, 1 insertions(+), 0 deletions(-) diff --git a/libavcodec/mpegaudiodec.c b/libavcodec/mpegaudiodec.c inde

Re: [libav-devel] [PATCH 1/4] rtmppkt: Escape backslash in Doxygen comment

2012-10-22 Thread Måns Rullgård
Diego Biurrun writes: > --- > libavformat/rtmppkt.h |2 +- > 1 files changed, 1 insertions(+), 1 deletions(-) > > diff --git a/libavformat/rtmppkt.h b/libavformat/rtmppkt.h > index 8932cac..cc35863 100644 > --- a/libavformat/rtmppkt.h > +++ b/libavformat/rtmppkt.h > @@ -253,7 +253,7 @@ int f

Re: [libav-devel] [PATCH] Improve wording and spelling of av_log_missing_feature messages.

2012-10-22 Thread Luca Barbato
On 10/22/2012 02:36 PM, Diego Biurrun wrote: > --- > libavcodec/aacdec.c|4 ++-- > libavcodec/aacsbr.c|4 ++-- > libavcodec/eac3dec.c |2 +- > libavcodec/flacdec.c |2 +- > libavcodec/flashsv.c |2 +- > libavcodec/proresdec.

[libav-devel] [PATCH 4/4] doxygen: Add av_alloc_size to list of predefined macros

2012-10-22 Thread Diego Biurrun
This avoids Doxygen believing the attribute is the function name. --- doc/Doxyfile |3 ++- 1 files changed, 2 insertions(+), 1 deletions(-) diff --git a/doc/Doxyfile b/doc/Doxyfile index 1b4e7d5..6fd0bd9 100644 --- a/doc/Doxyfile +++ b/doc/Doxyfile @@ -1381,7 +1381,8 @@ PREDEFINED

[libav-devel] [PATCH 2/4] doxygen: Build Doxygen documentation in the doc/ subdirectory

2012-10-22 Thread Diego Biurrun
--- .gitignore |2 +- Doxyfile |2 +- 2 files changed, 2 insertions(+), 2 deletions(-) diff --git a/.gitignore b/.gitignore index 8589127..3ed55b3 100644 --- a/.gitignore +++ b/.gitignore @@ -28,8 +28,8 @@ /doc/*.pod /doc/avoptions_codec.texi /doc/avoptions_format.texi +/doc/doxy/htm

[libav-devel] [PATCH 3/4] Move Doxyfile into the doc/ subdirectory

2012-10-22 Thread Diego Biurrun
--- Doxyfile => doc/Doxyfile |0 1 files changed, 0 insertions(+), 0 deletions(-) rename Doxyfile => doc/Doxyfile (100%) diff --git a/Doxyfile b/doc/Doxyfile similarity index 100% rename from Doxyfile rename to doc/Doxyfile -- 1.7.1 ___ libav-dev

[libav-devel] [PATCH 1/4] rtmppkt: Escape backslash in Doxygen comment

2012-10-22 Thread Diego Biurrun
--- libavformat/rtmppkt.h |2 +- 1 files changed, 1 insertions(+), 1 deletions(-) diff --git a/libavformat/rtmppkt.h b/libavformat/rtmppkt.h index 8932cac..cc35863 100644 --- a/libavformat/rtmppkt.h +++ b/libavformat/rtmppkt.h @@ -253,7 +253,7 @@ int ff_amf_read_number(GetByteContext *gbc, do

Re: [libav-devel] [PATCH 3/8] lavu: bump major to 52

2012-10-22 Thread Reinhard Tartler
On Mon, Oct 22, 2012 at 4:10 PM, Reinhard Tartler wrote: > On Mon, Oct 22, 2012 at 2:42 PM, Anton Khirnov wrote: >> In addition to the recent tables mess, the AVOption defaults behavior >> changed, so an old lavc used with a new lavu will get completely messed >> up defaults. >> --- >> doc/APIch

Re: [libav-devel] [PATCH 4/8] Revert "avutil: Add a copy of ff_sqrt_tab back into avutil to restore ABI compatibility"

2012-10-22 Thread Reinhard Tartler
On Mon, Oct 22, 2012 at 2:42 PM, Anton Khirnov wrote: > This reverts commit d15c21e5fa3961f10026da1a3080a3aa3cf4cec9. > > After the major bump this is no longer necessary. > --- > libavcodec/Makefile |1 - > libavcodec/mathtables.c | 11 +++ > libavcodec/sqrt_tab.c | 30

Re: [libav-devel] [PATCH 5/8] Revert "avutil: make some tables visible again"

2012-10-22 Thread Reinhard Tartler
On Mon, Oct 22, 2012 at 2:42 PM, Anton Khirnov wrote: > This reverts commit def8c94699877d015154a03d43e2b8f479cd89a0. > > After the major bump this is no longer necessary. > --- > libavutil/libavutil.v |2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) > > diff --git a/libavutil/libavutil

Re: [libav-devel] [PATCH 3/8] lavu: bump major to 52

2012-10-22 Thread Reinhard Tartler
On Mon, Oct 22, 2012 at 2:42 PM, Anton Khirnov wrote: > In addition to the recent tables mess, the AVOption defaults behavior > changed, so an old lavc used with a new lavu will get completely messed > up defaults. > --- > doc/APIchanges |2 +- > libavutil/version.h |4 ++-- > 2 file

Re: [libav-devel] [PATCH 3/4] pcmdec: use planar sample format for pcm_s16le_planar

2012-10-22 Thread Anton Khirnov
On Sun, 14 Oct 2012 17:40:07 -0400, Justin Ruggles wrote: > --- > libavcodec/pcm.c | 19 ++- > 1 files changed, 10 insertions(+), 9 deletions(-) > > diff --git a/libavcodec/pcm.c b/libavcodec/pcm.c > index c6c6bc0..da0e696 100644 > --- a/libavcodec/pcm.c > +++ b/libavcodec/pc

Re: [libav-devel] [PATCH] libxvid: remove useless doxy comments.

2012-10-22 Thread Diego Biurrun
On Mon, Oct 22, 2012 at 03:20:49PM +0200, Anton Khirnov wrote: > --- > libavcodec/libxvid.c | 27 --- > 1 file changed, 27 deletions(-) The changes itself do LGTM, but there is more Doxygen for purely file-internal stuff in there that should be converted to normal C comm

Re: [libav-devel] [PATCH] libxvid: remove useless doxy comments.

2012-10-22 Thread Justin Ruggles
On 10/22/2012 09:20 AM, Anton Khirnov wrote: > --- > libavcodec/libxvid.c | 27 --- > 1 file changed, 27 deletions(-) looks great -Justin ___ libav-devel mailing list libav-devel@libav.org https://lists.libav.org/mailman/list

[libav-devel] [PATCH] libxvid: remove useless doxy comments.

2012-10-22 Thread Anton Khirnov
--- libavcodec/libxvid.c | 27 --- 1 file changed, 27 deletions(-) diff --git a/libavcodec/libxvid.c b/libavcodec/libxvid.c index 89021e7..dd1e3e3 100644 --- a/libavcodec/libxvid.c +++ b/libavcodec/libxvid.c @@ -340,14 +340,6 @@ static void xvid_correct_framerate(AVCodec

Re: [libav-devel] [PATCH 01/16] atrac3: cosmetics: pretty-printing and renaming

2012-10-22 Thread Martin Storsjö
On Sat, 20 Oct 2012, Justin Ruggles wrote: also does some minor refactoring. --- libavcodec/atrac3.c | 1059 +++ libavcodec/atrac3data.h | 98 +++-- 2 files changed, 572 insertions(+), 585 deletions(-) LGTM // Martin

Re: [libav-devel] [PATCH 1/8] APIchanges: update lavr bump date

2012-10-22 Thread Justin Ruggles
On 10/22/2012 08:42 AM, Anton Khirnov wrote: > --- > doc/APIchanges |2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) > > diff --git a/doc/APIchanges b/doc/APIchanges > index ed479df..eed0f81 100644 > --- a/doc/APIchanges > +++ b/doc/APIchanges > @@ -6,7 +6,7 @@ libavcodec:2012-01-27

[libav-devel] [PATCH 7/8] lavu: remove disabled avoptions cruft

2012-10-22 Thread Anton Khirnov
--- libavutil/opt.c | 147 --- libavutil/opt.h | 76 -- libavutil/version.h |6 --- 3 files changed, 229 deletions(-) diff --git a/libavutil/opt.c b/libavutil/opt.c index b1ccf9b..61a72d0 100644 --- a/libavutil

[libav-devel] [PATCH 6/8] lavu: remove disabled FF_API_GET_BITS_PER_SAMPLE_FMT cruft

2012-10-22 Thread Anton Khirnov
--- libavutil/samplefmt.c |8 libavutil/samplefmt.h |8 libavutil/version.h |3 --- 3 files changed, 19 deletions(-) diff --git a/libavutil/samplefmt.c b/libavutil/samplefmt.c index 4f6dfd7..389f726 100644 --- a/libavutil/samplefmt.c +++ b/libavutil/samplefmt.c @@

[libav-devel] [PATCH 8/8] lavu: remove disabled FF_API_AV_FIFO_PEEK cruft

2012-10-22 Thread Anton Khirnov
--- libavutil/fifo.h| 11 --- libavutil/version.h |3 --- 2 files changed, 14 deletions(-) diff --git a/libavutil/fifo.h b/libavutil/fifo.h index e670175..ea30f5d 100644 --- a/libavutil/fifo.h +++ b/libavutil/fifo.h @@ -128,15 +128,4 @@ static inline uint8_t *av_fifo_peek2(const

[libav-devel] [PATCH 5/8] Revert "avutil: make some tables visible again"

2012-10-22 Thread Anton Khirnov
This reverts commit def8c94699877d015154a03d43e2b8f479cd89a0. After the major bump this is no longer necessary. --- libavutil/libavutil.v |2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/libavutil/libavutil.v b/libavutil/libavutil.v index c955347..e9f04cb 100644 --- a/libavu

[libav-devel] [PATCH 3/8] lavu: bump major to 52

2012-10-22 Thread Anton Khirnov
In addition to the recent tables mess, the AVOption defaults behavior changed, so an old lavc used with a new lavu will get completely messed up defaults. --- doc/APIchanges |2 +- libavutil/version.h |4 ++-- 2 files changed, 3 insertions(+), 3 deletions(-) diff --git a/doc/APIchang

[libav-devel] [PATCH 2/8] lavu: postpone recent deprecations until the next major bump

2012-10-22 Thread Anton Khirnov
--- libavutil/version.h |8 1 file changed, 4 insertions(+), 4 deletions(-) diff --git a/libavutil/version.h b/libavutil/version.h index 4de2a94..d21f532 100644 --- a/libavutil/version.h +++ b/libavutil/version.h @@ -74,16 +74,16 @@ #define FF_API_OLD_AVOPTIONS(LIBAVUTIL

[libav-devel] [PATCH 4/8] Revert "avutil: Add a copy of ff_sqrt_tab back into avutil to restore ABI compatibility"

2012-10-22 Thread Anton Khirnov
This reverts commit d15c21e5fa3961f10026da1a3080a3aa3cf4cec9. After the major bump this is no longer necessary. --- libavcodec/Makefile |1 - libavcodec/mathtables.c | 11 +++ libavcodec/sqrt_tab.c | 30 -- libavutil/Makefile |1 - libavu

[libav-devel] [PATCH 1/8] APIchanges: update lavr bump date

2012-10-22 Thread Anton Khirnov
--- doc/APIchanges |2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/doc/APIchanges b/doc/APIchanges index ed479df..eed0f81 100644 --- a/doc/APIchanges +++ b/doc/APIchanges @@ -6,7 +6,7 @@ libavcodec:2012-01-27 libavdevice: 2011-04-18 libavfilter: 2012-06-22 libavfo

[libav-devel] [PATCH] Improve wording and spelling of av_log_missing_feature messages.

2012-10-22 Thread Diego Biurrun
--- libavcodec/aacdec.c|4 ++-- libavcodec/aacsbr.c|4 ++-- libavcodec/eac3dec.c |2 +- libavcodec/flacdec.c |2 +- libavcodec/flashsv.c |2 +- libavcodec/proresdec.c |2 +- libavcodec/wmalosslessdec.c|4

Re: [libav-devel] [PATCH] lavc: remove stats_out from the options table.

2012-10-22 Thread Luca Barbato
On 10/19/2012 08:42 PM, Anton Khirnov wrote: > Since it is declared as a string AVOption, the generic freeing code > attempts to free it on codec close. Some codecs might have already freed > it elsewhere (or didn't even allocate it with av_malloc() in the first > place), so this might lead to an i