> +static void videotoolbox_default_free(AVCodecContext *avctx)
> +{
> +AVVideotoolboxContext *videotoolbox = avctx->hwaccel_context;
> +
> +if (videotoolbox) {
> +if (videotoolbox->cm_fmt_desc)
> +CFRelease(videotoolbox->cm_fmt_desc);
> +
> +if (videotoolbox->se
On Tue, Jun 30, 2015 at 04:33:35PM -0600, Roger Pack wrote:
> On 6/30/15, Michael Niedermayer wrote:
> > On Tue, Jun 30, 2015 at 01:00:22PM -0600, Roger Pack wrote:
> >> On 6/30/15, Michael Niedermayer wrote:
> >> > On Mon, Jun 29, 2015 at 11:09:14PM -0600, Roger Pack wrote:
> >>
> >> > [...]
> >
On 30.06.2015 23:56, Michael Niedermayer wrote:
> On Tue, Jun 30, 2015 at 09:31:20PM +0200, Andreas Cadhalpun wrote:
>> This is the first part of the fix for ticket #4370.
>>
>> Signed-off-by: Andreas Cadhalpun
>> ---
>> libavcodec/pthread_frame.c | 11 +++
>> 1 file changed, 11 insertion
On 6/30/15, Michael Niedermayer wrote:
> On Tue, Jun 30, 2015 at 01:00:22PM -0600, Roger Pack wrote:
>> On 6/30/15, Michael Niedermayer wrote:
>> > On Mon, Jun 29, 2015 at 11:09:14PM -0600, Roger Pack wrote:
>>
>> > [...]
>>
>> >> +Sleep(0);
>>
>> >
>>
>> > the funky line ending is he
On Tue, Jun 30, 2015 at 03:13:12PM -0500, Rodger Combs wrote:
> ---
> ffmpeg_opt.c | 12 ++--
> 1 file changed, 10 insertions(+), 2 deletions(-)
this seems usefull
maybe doing it per map parameter would be better though so the
user could specify which maps he intends to be optional
should
This returns something like "N-73264-gb54ac84". This is much more useful
than the individual library versions, of which there are too much and
which are very hard to map back to releases or git commits.
---
This actually just returns the contents of the already semi-public
av_util_ffversion variabl
On Tue, Jun 30, 2015 at 09:31:20PM +0200, Andreas Cadhalpun wrote:
> This is the first part of the fix for ticket #4370.
>
> Signed-off-by: Andreas Cadhalpun
> ---
> libavcodec/pthread_frame.c | 11 +++
> 1 file changed, 11 insertions(+)
probably ok
[...]
--
Michael GnuPG fingerpr
On Tue, Jun 30, 2015 at 09:31:57PM +0200, Andreas Cadhalpun wrote:
> If threading is used, the first (thread_count - 1) packets are read
> before any frame/error is returned. Counting this as successful decoding
> is wrong, because it also happens when no single frame could be decoded.
>
> Signed-
On Tue, Jun 30, 2015 at 09:31:49PM +0200, Andreas Cadhalpun wrote:
> This is the second part of the fix for ticket #4370.
>
> Signed-off-by: Andreas Cadhalpun
> ---
> ffmpeg.c | 9 +
> 1 file changed, 9 insertions(+)
LGTM
thanks
[...]
--
Michael GnuPG fingerprint: 9FF2128B147EF67
On Tue, Jun 30, 2015 at 10:51:00PM +0200, wm4 wrote:
> On Tue, 30 Jun 2015 16:42:56 +0200
> Michael Niedermayer wrote:
>
>
> > > +#ifdef AVCODEC_QSV_LINUX_SESSION_HANDLE
> > > +#include
> > > +#include
> >
> > > +#include
> >
> > needs HAVE_UNISTD_H check if its needed
>
> There are Linux
On Tue, Jun 30, 2015 at 01:23:57PM +, Nicolas Derouineau wrote:
> Please find here enclosed the patch synthesizing all your comments.
applied
thanks
[...]
--
Michael GnuPG fingerprint: 9FF2128B147EF6730BADF133611EC787040B0FAB
The misfortune of the wise is better than the prosperity of
On Tue, 30 Jun 2015 16:42:56 +0200
Michael Niedermayer wrote:
> > +#ifdef AVCODEC_QSV_LINUX_SESSION_HANDLE
> > +#include
> > +#include
>
> > +#include
>
> needs HAVE_UNISTD_H check if its needed
There are Linux systems without unistd.h?
___
ffmpe
---
ffmpeg_opt.c | 12 ++--
1 file changed, 10 insertions(+), 2 deletions(-)
diff --git a/ffmpeg_opt.c b/ffmpeg_opt.c
index a746405..11511d3 100644
--- a/ffmpeg_opt.c
+++ b/ffmpeg_opt.c
@@ -114,6 +114,7 @@ static int override_ffserver = 0;
static int input_stream_potentially_available =
On Tue, Jun 30, 2015 at 01:00:22PM -0600, Roger Pack wrote:
> On 6/30/15, Michael Niedermayer wrote:
> > On Mon, Jun 29, 2015 at 11:09:14PM -0600, Roger Pack wrote:
>
> > [...]
>
> >> +Sleep(0);
>
> >
>
> > the funky line ending is here
>
> OK try this one.
> thanks!
> configure
Hi Paul,
> On Jun 29, 2015, at 7:40 PM, Paul B Mahol wrote:
>
> Signed-off-by: Paul B Mahol
> ---
> doc/filters.texi| 35 ++
> libavfilter/af_astats.c | 80 +++--
> 2 files changed, 113 insertions(+), 2 deletions(-)
I tested
If threading is used, the first (thread_count - 1) packets are read
before any frame/error is returned. Counting this as successful decoding
is wrong, because it also happens when no single frame could be decoded.
Signed-off-by: Andreas Cadhalpun
---
ffmpeg.c | 6 +++---
1 file changed, 3 insert
This is the second part of the fix for ticket #4370.
Signed-off-by: Andreas Cadhalpun
---
ffmpeg.c | 9 +
1 file changed, 9 insertions(+)
diff --git a/ffmpeg.c b/ffmpeg.c
index aac03bb..e3d0cc4 100644
--- a/ffmpeg.c
+++ b/ffmpeg.c
@@ -1873,6 +1873,9 @@ static int decode_audio(InputStrea
This is the first part of the fix for ticket #4370.
Signed-off-by: Andreas Cadhalpun
---
libavcodec/pthread_frame.c | 11 +++
1 file changed, 11 insertions(+)
diff --git a/libavcodec/pthread_frame.c b/libavcodec/pthread_frame.c
index a3fd1ff..d48ee33 100644
--- a/libavcodec/pthread_fram
On 6/30/15, Michael Niedermayer wrote:
> On Mon, Jun 29, 2015 at 11:09:14PM -0600, Roger Pack wrote:
> [...]
>> +Sleep(0);
>
> the funky line ending is here
OK try this one.
thanks!
0001-windows-respond-to-logoff-and-ctrl-break-messages-as.patch
Description: Binary data
Hello Michael,
Thank you very much, there is fixed patch attached, please look.
Tuesday, June 30, 2015, 7:21:53 PM, you wrote:
MN> On Tue, Jun 30, 2015 at 06:28:08PM +0300, Ivan Uskov wrote:
>> Hello Michael,
>>
>> I'm sorry, what exactly documentation should be added regarding this
>> patch? A
On Tue, Jun 30, 2015 at 10:55:45AM -0300, Claudio Freire wrote:
> LGTM, but it cannot be committed without the others, or a memset to
> zero of is_mask somewhere at the beginning of each frame (or encoder
> init).
ok, ill wait till the next patchset
@Rostislav, maybe mark it as "Reviewed-by:..."
On Mon, Jun 29, 2015 at 11:09:14PM -0600, Roger Pack wrote:
[...]
> +Sleep(0);
the funky line ending is here
[...]
--
Michael GnuPG fingerprint: 9FF2128B147EF6730BADF133611EC787040B0FAB
Concerning the gods, I have no means of knowing whether they exist or not
or of what sort t
On Mon, Jun 29, 2015 at 11:09:14PM -0600, Roger Pack wrote:
> On 6/25/15, Michael Niedermayer wrote:
> > On Wed, Jun 24, 2015 at 11:48:10PM -0600, Roger Pack wrote:
> >> On 6/24/15, Michael Niedermayer wrote:
> >> > On Wed, Jun 24, 2015 at 04:19:38AM -0600, Roger Pack wrote:
> >> >> On 7/5/12, Mi
On Tue, Jun 30, 2015 at 06:28:08PM +0300, Ivan Uskov wrote:
> Hello Michael,
>
> I'm sorry, what exactly documentation should be added regarding this
> patch? As I can see there is no any standalone topic about QSV into
> current /doc. Can modifications of Changelog be enough?
your patch adds com
Signed-off-by: Paul B Mahol
---
doc/filters.texi| 35 ++
libavfilter/af_astats.c | 80 +++--
2 files changed, 113 insertions(+), 2 deletions(-)
diff --git a/doc/filters.texi b/doc/filters.texi
index 46df5b5..ed33621 100644
-
Hello Michael,
I'm sorry, what exactly documentation should be added regarding this
patch? As I can see there is no any standalone topic about QSV into
current /doc. Can modifications of Changelog be enough?
Tuesday, June 30, 2015, 5:42:56 PM, you wrote:
MN> On Tue, Jun 30, 2015 at 05:24:54PM +0
The problem was that ms_mask altered the phase during distortion
calculations but did not alter it again during codebook selection (e.g.
INTENSITY_BT or INTENSITY_BT2), so the phases got swapped around and the
artifacts appeared. M/S is off by default so the artifacts were only
present if -stereo_m
On Tue, Jun 30, 2015 at 05:24:54PM +0300, Ivan Uskov wrote:
> Hello All,
>
> Unlike Windows, under linux a valid display handle should be specified
> to open a QSV/MFX session.
> The attached patch does find appropriate display handle and transfers
> it to Intel's MFX library.
>
>
>
>
> --
Hello All,
Unlike Windows, under linux a valid display handle should be specified
to open a QSV/MFX session.
The attached patch does find appropriate display handle and transfers
it to Intel's MFX library.
--
Best regards,
Ivan mailto:ivan.us...@nablet.com
0001-Ad
LGTM, but it cannot be committed without the others, or a memset to
zero of is_mask somewhere at the beginning of each frame (or encoder
init).
On Fri, Jun 26, 2015 at 5:16 PM, Rostislav Pehlivanov
wrote:
> This commit adds support for the coding of intensity stereo spectral
> coefficients. It a
Please find here enclosed the patch synthesizing all your comments.
Nicolas DEROUINEAU
Research Engineer
VITEC
T. +33 1 46 73 06 06
E. nicolas.derouin...@vitec.com
W. www.vitec.com
De : ffmpeg-devel-boun...@ffmpeg.org de la
part de Carl Eugen Hoyos
E
On Tue, Jun 30, 2015 at 02:05:37PM +0200, Carl Eugen Hoyos wrote:
> Hi!
>
> Attached patch makes the output of ffmpeg -i lena.pnm -strict -2 out.jp2
> compatible with Kakadu, fixes ticket #4689.
>
> Please comment, Carl Eugen
> j2kenc.c |1 +
> 1 file changed, 1 insertion(+)
> 7fe04b2a6ae6
On Tue, Jun 30, 2015 at 11:41:03AM +0200, Carl Eugen Hoyos wrote:
> Hi!
>
> I just tested a patch to make the j2k encoder lossless by default
> but I had not realized that the encoder does listen to -qscale which
> might make more sense.
>
> So instead I suggest the attached documentation updat
Hi!
Attached patch makes the output of ffmpeg -i lena.pnm -strict -2 out.jp2
compatible with Kakadu, fixes ticket #4689.
Please comment, Carl Eugen
diff --git a/libavcodec/j2kenc.c b/libavcodec/j2kenc.c
index dcdf411..82a75aa 100644
--- a/libavcodec/j2kenc.c
+++ b/libavcodec/j2kenc.c
@@ -962,6 +
On Tue, Jun 30, 2015 at 05:59:53AM -0400, Ronald S. Bultje wrote:
> Hi,
>
> On Tue, Jun 30, 2015 at 2:03 AM, James Zern wrote:
>
> > the max value of the lookup in expanded form is:
> > (((1 << 7) - 1) << 1) - 65 + 1 + 64 = 254
> >
> > add one entry of padding to inv_map_table[] to prevent out o
On Tuesday 30 June 2015 12:30:50 pm Nicolas Derouineau wrote:
> >> @@ -2610,6 +2610,7 @@ typedef struct AVCodecContext {
> >> #define FF_DEBUG_ER 0x0400
> >> #define FF_DEBUG_MMCO 0x0800
> >> #define FF_DEBUG_BUGS 0x1000
> >> +#define FF_DEBUG_GREEN_MD 0x000
>> @@ -475,6 +475,8 @@ per-block quantization parameter (QP)
>> motion vector
>> @item dct_coeff
>>
>> +@item green_metadata
>> +
>
>Shouldn't there be some explanation?
Corrected
>> @@ -2610,6 +2610,7 @@ typedef struct AVCodecContext {
>> #define FF_DEBUG_ER 0x0400
>> #define FF
Hi,
On Tue, Jun 30, 2015 at 2:03 AM, James Zern wrote:
> the max value of the lookup in expanded form is:
> (((1 << 7) - 1) << 1) - 65 + 1 + 64 = 254
>
> add one entry of padding to inv_map_table[] to prevent out of bounds
> access with non-conforming / fuzzed bitstreams
>
> Signed-off-by: James
From: Djordje Pesut
Add fixed point implementation.
Signed-off-by: Nedeljko Babic
---
libavcodec/Makefile | 14 ++-
libavcodec/aac_defines.h | 36 ++
libavcodec/aacps.c| 255 --
libavcodec/aacps.h|
From: Djordje Pesut
Add fixed poind code.
Signed-off-by: Nedeljko Babic
---
libavcodec/Makefile | 5 +-
libavcodec/aac.h | 52 +---
libavcodec/aac_defines.h | 78 ++
libavcodec/aacdec_template.c | 14 +-
libavcodec/aacsbr.c | 1 +
libavcodec/aacsbr.
From: Jovan Zelincevic
Move existing code to the new template files
Signed-off-by: Nedeljko Babic
---
libavcodec/aacdec.c| 3132 +---
libavcodec/{aacdec.c => aacdec_template.c} | 623 +---
libavcodec/cbrt_tablegen.c
From: Jovan Zelincevic
Signed-off-by: Nedeljko Babic
---
Changelog| 1 +
doc/general.texi | 2 +-
doc/mips.txt | 4
libavcodec/version.h | 2 +-
4 files changed, 7 insertions(+), 2 deletions(-)
diff --git a/Changelog b/Changelog
index 6e9ca8e..00f67de 100644
--- a/
Use macros from aac_defines.h for adding suffixes
instead of local macros.
Signed-off-by: Nedeljko Babic
---
libavcodec/cbrt_tablegen.h | 5 ++---
libavcodec/cbrt_tablegen_template.c| 2 +-
libavcodec/sinewin.h | 17 +
libavcodec/sinewin_table
Signed-off-by: Nedeljko Babic
---
tests/fate/aac.mak | 58 +-
1 file changed, 57 insertions(+), 1 deletion(-)
diff --git a/tests/fate/aac.mak b/tests/fate/aac.mak
index 34823be..7ebec45 100644
--- a/tests/fate/aac.mak
+++ b/tests/fate/aac.mak
@
From: Jovan Zelincevic
Add fixed point implementation of functions for generating tables.
Signed-off-by: Nedeljko Babic
---
libavcodec/aacps_fixed_tablegen.c| 24 +++
libavcodec/aacps_fixed_tablegen.h| 402 +++
libavcodec/aacps_tablegen.c | 73
From: Djordje Pesut
Add fixed point implementation
Signed-off-by: Nedeljko Babic
---
libavcodec/aac.h | 80 ++--
libavcodec/aacdec.c | 5 +
libavcodec/aacdec_fixed.c| 444 +++
libavcodec/aacdec_template.c | 421 +++
From: Jovan Zelincevic
Add fixed point implementation of functions for generating tables
Signed-off-by: Nedeljko Babic
---
libavcodec/aac.h | 23 +
libavcodec/aacdectab.h | 34 +-
libavcodec/aactab.c
From: Jovan Zelincevic
Create tables for fixed point code.
Signed-off-by: Nedeljko Babic
---
libavcodec/Makefile| 5 +-
.../{aacsbr_tablegen.c => aacsbr_fixed_tablegen.c} | 7 +-
.../{aacsbr_tablegen.c => aacsbr_fixed_tablegen.h} | 21 +-
libavcodec/aacsbr_
From: Djordje Pesut
Move the existing code to a new template file.
Signed-off-by: Nedeljko Babic
---
libavcodec/aacsbr.c| 1419 +---
libavcodec/aacsbr.h| 45 +
libavcodec/{aacsbr.c => aacsbr_template.c} | 377
In this patch set patches are formed using -C option of git format-patch in
order for the patches to be more readable.
Commit message for one of the patches is changed so some warnings are explained.
Patchset rebased to the newest code base.
Please take a look.
Thanks,
Nedeljko
___
From: Jovan Zelincevic
Build system modified
There are several warnings occurring during build after this patch is
applied. The cause of most of these warnings is in that some definitions
needed here are logical part of sbr module and are added in later patches.
When this patches are applied the
On Tuesday 30 June 2015 11:38:03 am Nicolas Derouineau wrote:
> From f7cff28a44adada6761233d59207b4a9c0373960 Mon Sep 17 00:00:00 2001
> From: Nicolas DEROUINEAU
> Date: Tue, 30 Jun 2015 10:59:07 +0200
> Subject: [PATCH] Patch to parse SEI GreenMetadata
>
> ---
> doc/codecs.texi |
Hi!
I just tested a patch to make the j2k encoder lossless by default
but I had not realized that the encoder does listen to -qscale which
might make more sense.
So instead I suggest the attached documentation update and the
removal of the experimental flag.
Please comment, Carl Eugen
diff --
Hello,
Thanks for your comments. Here's an updated version of the patch.
Did you have any troubles opening the bitstreams ?
BR,
Nicolas DEROUINEAU
Research Engineer
VITEC
T. +33 1 46 73 06 06
E. nicolas.derouin...@vitec.com
W. www.vitec.com
De : Mic
Hi,
On Sun, Jun 28, 2015 at 10:06 PM, Nicolas George wrote:
> Le decadi 10 messidor, an CCXXIII, Stephan Holljes a écrit :
>> Hi,
>> attached patches are the current state of work.
>> It's probably still a bit rough around the edges, but I think I made
>> some progress.
>> The sample code in doc/
>>
>> Cause for these warnings is similar as for the first ones: needed
>> definitions are added in later patches (that make changes in sbr module).
>> When all the patches are applied there are no warnings here.
>
>if it breaks nothing then temporary warnings are probably ok.
>Some people might ar
On Tuesday 19 May 2015 06:15:49 pm Michael Niedermayer wrote:
> > return AVPROBE_SCORE_EXTENSION + 1;
>
> i think this should be higher with that change
>
> also i think a somewhat lower score could still be output
> for less valid but still decodeable files
The patch was merged with the
Michael Niedermayer gmx.at> writes:
> > Makefile |1 +
> > allformats.c |1 +
> > img2dec.c| 12
> > 3 files changed, 14 insertions(+)
> > 694e8baeb11c153fd9b814eb22dabf5ad0c16c09 patchdds.diff
>
> LGTM
The patch was merged.
Carl Eugen
58 matches
Mail list logo
