Clément Bœsch pkh.me> writes:
> No it won't work because require_pkg_config dies if
> it fails, so you can fallback after it.
True.
> Of course we can add another require_pkg_config2()
> that takes some explicit linking flags in addition
> to the user ones as a fallback.
Sounds useful to me
On Fri, Aug 15, 2014 at 01:23:59AM +0200, Michael Niedermayer wrote:
> On Thu, Aug 14, 2014 at 11:05:11PM +0200, Clément Bœsch wrote:
> > ---
> > doc/APIchanges | 3 +++
> > libavutil/pixelutils.c | 21 +
> > libavutil/pixelutils.h | 23 +++
> > lib
On Thu, Aug 14, 2014 at 12:24:47PM +0200, Christophe Gisquet wrote:
> Hi,
>
> the attached patch seems to fix ticket #3773, but streamcopy is such a
> core feature of ffmpeg that I assume I'm missing a use case or
> incorrectly handle the buffers referencing.
>
> Another note is that this patch w
On Thu, Aug 14, 2014 at 11:05:11PM +0200, Clément Bœsch wrote:
> ---
> doc/APIchanges | 3 +++
> libavutil/pixelutils.c | 21 +
> libavutil/pixelutils.h | 23 +++
> libavutil/version.h| 2 +-
> 4 files changed, 48 insertions(+), 1 deletion(-)
>
On Tue, Aug 12, 2014 at 02:58:04PM -0700, Mark Reid wrote:
> UL values copied from FMbc version of mxf.c
> Fixes Ticket#1554, Ticket#3100 and Ticket#3450
> ---
> libavformat/mxf.c | 3 +++
> 1 file changed, 3 insertions(+)
missing signed-off
[...]
--
Michael GnuPG fingerprint: 9FF2128B147E
Not that much relevant from a performance PoV because not called often.
---
libavfilter/vf_fieldmatch.c | 42 ++
1 file changed, 22 insertions(+), 20 deletions(-)
diff --git a/libavfilter/vf_fieldmatch.c b/libavfilter/vf_fieldmatch.c
index e2aa60b..4935a30
Note: this isn't actually faster. pixelutils_sad_a_16x16_sse2() is ~500
decicyles. pixelutils_sad_8x8_mmxext() is ~240 decicyles. Still, the
overall decode is bigger when using the former. I don't get it.
---
libavfilter/f_select.c | 10 +++---
1 file changed, 7 insertions(+), 3 deletions(-)
~560 → ~500 decicycles
This is following the comments from Michael in
https://ffmpeg.org/pipermail/ffmpeg-devel/2014-August/160599.html
Using 2 registers for accumulator didn't help. On the other hand,
some re-ordering between the movs and psadbw allowed going ~538 to ~500.
---
libavutil/x86/pix
---
libavfilter/f_select.c | 20 +---
1 file changed, 5 insertions(+), 15 deletions(-)
diff --git a/libavfilter/f_select.c b/libavfilter/f_select.c
index 546a940..e1837db 100644
--- a/libavfilter/f_select.c
+++ b/libavfilter/f_select.c
@@ -251,22 +251,12 @@ static double get_scene
---
doc/APIchanges | 3 +++
libavutil/pixelutils.c | 21 +
libavutil/pixelutils.h | 23 +++
libavutil/version.h| 2 +-
4 files changed, 48 insertions(+), 1 deletion(-)
diff --git a/doc/APIchanges b/doc/APIchanges
index 067f60f..72cfe96 100644
On Tue, 2014-08-12 at 14:58 -0700, Mark Reid wrote:
> UL values copied from FMbc version of mxf.c
> Fixes Ticket#1554, Ticket#3100 and Ticket#3450
> ---
> libavformat/mxf.c | 3 +++
> 1 file changed, 3 insertions(+)
>
> diff --git a/libavformat/mxf.c b/libavformat/mxf.c
> index d20ed94..8ebfdf2 1
From: gt-sdi
Extract poster_time and time_scale from quicktime mdhd atom
and add to metadata for output by ffprobe.
Signed-off-by: gt-sdi
---
libavformat/mov.c | 27 +--
1 file changed, 25 insertions(+), 2 deletions(-)
diff --git a/libavformat/mov.c b/libavformat/mov.c
After further searching and understanding the makefile I see that the
lpc code is tested by the complete fate sample suite. This patch is
unnecessary.
Anyway to answer the questions...
On 2014-08-14 18:14, Christophe Gisquet wrote:
> Hi,
>
> 2014-08-14 18:08 GMT+02:00 James Darnley :
>> Minimum
On Thu, Aug 14, 2014 at 06:01:49PM +0200, Christophe Gisquet wrote:
> Hi,
>
> 2014-08-14 17:14 GMT+02:00 James Darnley :
> > This should help to clarify the API.
>
> OK.
applied
thanks
[...]
--
Michael GnuPG fingerprint: 9FF2128B147EF6730BADF133611EC787040B0FAB
DNS cache poisoning attac
On Thu, Aug 14, 2014 at 08:00:21PM +0200, Clément Bœsch wrote:
> On Thu, Aug 14, 2014 at 05:37:25PM +, Carl Eugen Hoyos wrote:
> > Clément Bœsch pkh.me> writes:
> >
> > > Carl, do you want the fallback to work just when
> > > pkg-config is not available
> >
> > I believe this would still le
On Thu, Aug 14, 2014 at 05:37:25PM +, Carl Eugen Hoyos wrote:
> Clément Bœsch pkh.me> writes:
>
> > Carl, do you want the fallback to work just when
> > pkg-config is not available
>
> I believe this would still leave some use-cases
> unsolved.
>
> > or do you actually want a real fallbac
On Thu, Aug 14, 2014 at 03:13:03PM +0200, Nicolas George wrote:
> Le septidi 27 thermidor, an CCXXII, Clément Bœsch a écrit :
> > contet?
>
> Thanks for spotting it. The series to merge is now:
>
> a3aaaec lavfi/avf_showspectrum: set output frame rate.
> 65b284a lavfi/avf_showspectrum: fix output
Clément Bœsch pkh.me> writes:
> Carl, do you want the fallback to work just when
> pkg-config is not available
I believe this would still leave some use-cases
unsolved.
> or do you actually want a real fallback when it is
> present but didn't succeed?
This is the preferred solution imo.
>
Hi,
2014-08-14 18:08 GMT+02:00 James Darnley :
> Minimum 1, maximum 8. Should I find out what order(s) are actually chosen?
The encoder algorithm might change and make that information useless,
but that could be useful. I fear that only checking which orders it
*tests* is insufficient: if the ds
On 2014-08-14 18:03, Christophe Gisquet wrote:
> Hi,
>
> 2014-08-14 17:15 GMT+02:00 James Darnley :
>> The LPC encoder is not used at compression levels below 3 but the basic
>> acodec
>> test uses -compression_level 2. This adds a test using -compression_level 5,
>> which is the current default
Hi,
2014-08-14 17:15 GMT+02:00 James Darnley :
> The LPC encoder is not used at compression levels below 3 but the basic acodec
> test uses -compression_level 2. This adds a test using -compression_level 5,
> which is the current default.
What order is used in that case? It may be good to in fac
Hi,
2014-08-14 17:14 GMT+02:00 James Darnley :
> This should help to clarify the API.
OK.
--
Christophe
___
ffmpeg-devel mailing list
ffmpeg-devel@ffmpeg.org
http://ffmpeg.org/mailman/listinfo/ffmpeg-devel
Hi,
2014-08-14 17:14 GMT+02:00 James Darnley :
> + * This limit is used:
> + * - when CONFIG_SMALL is 0 to unroll a loop in the C template.
> + * - when SSE4 (or newer) is available on x86 to unroll a copy loop.
Does the implementer of a new dsp version really has a need for this
info
-BEGIN PGP SIGNED MESSAGE-
Hash: SHA512
On 08/14/2014 11:27 AM, Thomas Goirand wrote:
> On 08/13/2014 07:53 AM, Kieran Kunhya wrote:
> On 08/13/2014 06:30 AM, Michael Niedermayer wrote:
>
>> Also ive offered my resignation in the past. I do still offer to
>> resign from the FFmpeg leade
This should help to clarify the API.
---
libavcodec/flacdsp.h |2 +-
1 files changed, 1 insertions(+), 1 deletions(-)
diff --git a/libavcodec/flacdsp.h b/libavcodec/flacdsp.h
index e8487c8..23d1ae4 100644
--- a/libavcodec/flacdsp.h
+++ b/libavcodec/flacdsp.h
@@ -35,7 +35,7 @@ typedef struct F
The LPC encoder is not used at compression levels below 3 but the basic acodec
test uses -compression_level 2. This adds a test using -compression_level 5,
which is the current default.
---
tests/fate/acodec.mak|4 +++-
tests/ref/acodec/flac-comp-5 |4
2 files changed, 7 inse
State that the maximum value of order is 32. This limit is used in both C and
x86 assebly code.
---
libavcodec/flacdsp.h |7 +++
1 files changed, 7 insertions(+), 0 deletions(-)
diff --git a/libavcodec/flacdsp.h b/libavcodec/flacdsp.h
index 272cf2a..e8487c8 100644
--- a/libavcodec/flacds
On Thu, Aug 14, 2014 at 03:08:39PM +0200, Christophe Gisquet wrote:
> Hi,
>
> 2014-08-14 14:32 GMT+02:00 Michael Niedermayer :
> > is this the patch that you wanted to send ?
> > it changes dpx.c instead of version.h
>
> Woops, attached the wrong 0001 one...
>
> --
> Christophe
> version.h |
On Tue, Aug 12, 2014 at 09:08:37AM +0200, Christophe Gisquet wrote:
> Hi,
>
> 2014-08-12 8:10 GMT+02:00 Christophe Gisquet :
> > Yeah, I'll drop it.
>
> Done. Kostya suggested another rounding (the slice bitstreams are
> byte-aligned), integrated in this patch.
>
> --
> Christophe
> proresenc
Le septidi 27 thermidor, an CCXXII, Clément Bœsch a écrit :
> contet?
Thanks for spotting it. The series to merge is now:
a3aaaec lavfi/avf_showspectrum: set output frame rate.
65b284a lavfi/avf_showspectrum: fix output pts computation.
d4de6d4 lavfi/avf_showspectrum: do not push the frame at EOF
On Thu, Aug 14, 2014 at 03:06:14PM +0200, Nicolas George wrote:
> Le nonidi 19 thermidor, an CCXXII, Clément Bœsch a écrit :
> > OK
>
> Thanks for the reviews on this series. I have pushed this to my tree for
> merging at any convenient time:
>
> a3aaaec lavfi/avf_showspectrum: set output frame r
Hi,
2014-08-14 14:32 GMT+02:00 Michael Niedermayer :
> is this the patch that you wanted to send ?
> it changes dpx.c instead of version.h
Woops, attached the wrong 0001 one...
--
Christophe
From a7eb428f6a9e2175885defe22b0e082b320ef7cf Mon Sep 17 00:00:00 2001
From: Christophe Gisquet
Date: T
Le nonidi 19 thermidor, an CCXXII, Clément Bœsch a écrit :
> OK
Thanks for the reviews on this series. I have pushed this to my tree for
merging at any convenient time:
a3aaaec lavfi/avf_showspectrum: set output frame rate.
65b284a lavfi/avf_showspectrum: fix output pts computation.
d4de6d4 lavfi
Le nonidi 19 thermidor, an CCXXII, Clément Bœsch a écrit :
> Having min_samples=max_samples=N doesn't do that already?
No: not for the last frame. That would leave no channel to pass the
information of the exact number of channels.
I will include this in the merge request soon unless you object.
On 08/02/2014 10:40 AM, Stefano Sabatini wrote:
The new option names are more explicit.
---
[..]
Looks OK
Bests,
--
Reynaldo H. Verdejo Pinochet
Sr. Multimedia Engineer, Open Source Group
Samsung Research America - Silicon Valley
___
ffmpeg-devel m
On Thu, Aug 14, 2014 at 02:21:56PM +0200, Christophe Gisquet wrote:
> Hi,
>
> following the discussion in "[PATCH 2/4] dpxenc: enforce alignment
> requirement", here's a bump to offer a chance to other decoders to
> detect and recover from the change.
>
> --
> Christophe
> dpx.c | 42 +++
Hi,
following the discussion in "[PATCH 2/4] dpxenc: enforce alignment
requirement", here's a bump to offer a chance to other decoders to
detect and recover from the change.
--
Christophe
From ea8003852db0e6f64ec50f2ca644c9b51247cb5f Mon Sep 17 00:00:00 2001
From: Christophe Gisquet
Date: Wed,
On Thu, Aug 14, 2014 at 09:24:51AM +0200, Christophe Gisquet wrote:
> Hi,
>
> 2014-08-14 9:00 GMT+02:00 Christophe Gisquet :
>
> > 2014-08-14 4:17 GMT+02:00 Michael Niedermayer :
> >> causes
> >> ./ffmpeg -icheckerboard_1080p_nuke_bigendian_12bit_alpha.dpx -f null -
> >> to segfault
> >>
> >> see
On date Wednesday 2014-08-13 16:27:20 +0200, Attila Kinali encoded:
> On Wed, 13 Aug 2014 00:30:05 +0200
> Michael Niedermayer wrote:
>
> > I never understood why people who once where friends
> > became mutually so hostile
>
> You should know that better than anyone else!
>
> You still claim t
On date Monday 2014-08-11 15:22:59 +0200, Clément Bœsch encoded:
> From: Clément Bœsch
>
> The reasoning behind this addition is that various third party
> applications are interested in getting some motion information out of a
> video "for free" when it is available.
>
> It was considered to ex
On Wed, Aug 13, 2014 at 12:53:41AM +0100, Kieran Kunhya wrote:
> > Also ive offered my resignation in the past.
> > I do still offer to resign from the FFmpeg leader position, if it
> > resolves this split between FFmpeg and Libav and make everyone work
> > together again. I never understood why pe
On date Saturday 2014-08-02 16:40:53 +0200, Stefano Sabatini encoded:
> The new option names are more explicit.
> ---
> doc/ffserver.conf | 4 ++--
> doc/ffserver.texi | 19 ++-
> ffserver.c| 10 +++---
> 3 files changed, 23 insertions(+), 10 deletions(-)
Ping (Reynal
On Thu, Aug 14, 2014 at 09:36:53AM +0200, Christophe Gisquet wrote:
> Hi,
>
> 2014-08-14 4:30 GMT+02:00 Michael Niedermayer :
> >> probably ok
> >
> > applied
>
> Now that I think of it, this change causes older versions of ffmpeg to
> be unable to decode the output of the fixed encoder. So:
> -
On date Monday 2014-08-11 17:39:37 +0800, Steven Liu encoded:
> Hi Guys,
>
> The FFmpeg hls module can make m3u8 and ts, but it dosen't delete the
> old ts segment file.
> If always run this module, the disk will full, so this patch can fix
> the problem.
> When update the segment
Hi,
the attached patch seems to fix ticket #3773, but streamcopy is such a
core feature of ffmpeg that I assume I'm missing a use case or
incorrectly handle the buffers referencing.
Another note is that this patch was developed without a memory tracer
like valgrind: I have just validated that the
On date Wednesday 2014-08-13 08:59:58 -0700, Timothy Gu encoded:
> On Aug 13, 2014 8:58 AM, "Stefano Sabatini" wrote:
> >
> > Fix trac issue #3850.
> > ---
>
> OK.
Pushed (yesterday), thanks.
--
FFmpeg = Fantastic & Frightening Merciless Perfectionist Epic Gymnast
__
Am 14.08.14 01:56, schrieb compn:
> On Wed, 13 Aug 2014 18:48:38 +0200
> Reimar Döffinger wrote:
>
>> On 13.08.2014, at 18:42, Stefano Sabatini wrote:
>>> On date Sunday 2014-07-20 18:16:52 +0200, Jean-Baptiste Kempf
>>> encoded:
My dear friends of the FFmpeg community,
I'd like t
Hi,
2014-08-12 12:44 GMT+02:00 Christophe Gisquet :
> Forgot a parameter to the call to avpriv_request_sample, will be added later.
If there's no further comment on this option for the reallocation,
here's an updated patch.
--
Christophe
From 4ea51dc2cec915ce4c84db92febd8960bb44b650 Mon Sep 17
Hi,
2014-08-14 5:01 GMT+02:00 Michael Niedermayer :
> On Wed, Aug 13, 2014 at 10:21:54AM +, Christophe Gisquet wrote:
>> case 50081:
>> +avctx->pix_fmt = AV_PIX_FMT_BGR24;
>> +break;
>
> this possibly breaks decoding of
> checkerboard_1080p_nuke_bigendian_8bit_noalpha.dpx
Hi,
2014-08-14 4:17 GMT+02:00 Michael Niedermayer :
> see http://samples.ffmpeg.org/image-samples/dpx_samples.zip
Note that the date metadata in that file does not have a valid format either.
--
Christophe
___
ffmpeg-devel mailing list
ffmpeg-devel@ff
Hi,
2014-08-14 4:30 GMT+02:00 Michael Niedermayer :
>> probably ok
>
> applied
Now that I think of it, this change causes older versions of ffmpeg to
be unable to decode the output of the fixed encoder. So:
- Should we bump eg a minor version number to let any decoder have the
possibility to dete
Hi,
2014-08-14 9:00 GMT+02:00 Christophe Gisquet :
> 2014-08-14 4:17 GMT+02:00 Michael Niedermayer :
>> causes
>> ./ffmpeg -icheckerboard_1080p_nuke_bigendian_12bit_alpha.dpx -f null -
>> to segfault
>>
>> see http://samples.ffmpeg.org/image-samples/dpx_samples.zip
Forgot to add this last bit: u
Hi,
2014-08-14 4:17 GMT+02:00 Michael Niedermayer :
> causes
> ./ffmpeg -icheckerboard_1080p_nuke_bigendian_12bit_alpha.dpx -f null -
> to segfault
>
> see http://samples.ffmpeg.org/image-samples/dpx_samples.zip
OK, thanks. Disclaimer, I'm not that knowledgeable in DPX, I'm just
applying the anal
53 matches
Mail list logo