> -#define YUVRGB_TABLE_HEADROOM 256
> +#define YUVRGB_TABLE_HEADROOM 512
[...]
> -const int yoffs = fullRange ? 384 : 326;
> +const int yoffs = fullRange ? 896 : 838;
I think it's time to use that macro everywhere it is actually used without
showing up.
Best regards,
Christophe
_
On Mon, Jan 11, 2016 at 7:23 PM, Ganesh Ajjanagadde
wrote:
> This is quite an accurate approximation; testing shows ~ 2ulp error in
> the floating point result. Tested with FATE.
>
> Alternatively, if one wants "full accuracy", one can use powf, or sqrt
> instead of sqrtf. With powf, one gets 1 ul
From: Michael Niedermayer
This makes SWS more robust
Fixes:
07650a772d98aa63b0fed6370dc89037/asan_heap-oob_27ddeaf_2657_2c81ff264dee5d9712cb3251fb9c3bbb.264
Fixes: out of array read
Found-by: Mateusz "j00ru" Jurczyk and Gynvael Coldwind
Signed-off-by: Michael Niedermayer
---
libswscale/swscal
Hi,
On Wed, Jan 13, 2016 at 8:09 PM, Ganesh Ajjanagadde
wrote:
> On Wed, Jan 13, 2016 at 8:07 PM, Ronald S. Bultje
> wrote:
> > Hi,
> >
> > On Wed, Jan 13, 2016 at 7:48 PM, Ganesh Ajjanagadde <
> gajjanaga...@gmail.com>
> > wrote:
> >>
> >> On Wed, Jan 13, 2016 at 7:39 PM, Ronald S. Bultje
> >
---
libavcodec/avcodec.h| 2 ++
libavcodec/codec_desc.c | 1 +
libavcodec/mlp_parser.c | 5 +
libavcodec/mlpdec.c | 2 ++
libavcodec/profiles.c | 6 ++
libavcodec/profiles.h | 1 +
6 files changed, 17 insertions(+)
diff --git a/libavcodec/avcodec.h b/libavcodec/avcodec.h
index
On Wed, Jan 13, 2016 at 8:11 PM, James Almer wrote:
> On 1/13/2016 9:48 PM, Ganesh Ajjanagadde wrote:
>> On Wed, Jan 13, 2016 at 7:39 PM, Ronald S. Bultje wrote:
>>> Hi,
>>>
>>> On Wed, Jan 13, 2016 at 7:10 PM, Ganesh Ajjanagadde
>>> wrote:
Signed-off-by: Ganesh Ajjanagadde
---
>
On 1/13/2016 9:48 PM, Ganesh Ajjanagadde wrote:
> On Wed, Jan 13, 2016 at 7:39 PM, Ronald S. Bultje wrote:
>> Hi,
>>
>> On Wed, Jan 13, 2016 at 7:10 PM, Ganesh Ajjanagadde
>> wrote:
>>>
>>> Signed-off-by: Ganesh Ajjanagadde
>>> ---
>>> configure| 1 +
>>> libavutil/libm.h | 4
>>>
On Wed, Jan 13, 2016 at 8:07 PM, Ronald S. Bultje wrote:
> Hi,
>
> On Wed, Jan 13, 2016 at 7:48 PM, Ganesh Ajjanagadde
> wrote:
>>
>> On Wed, Jan 13, 2016 at 7:39 PM, Ronald S. Bultje
>> wrote:
>> > Hi,
>> >
>> > On Wed, Jan 13, 2016 at 7:10 PM, Ganesh Ajjanagadde
>> >
>> > wrote:
>> >>
>> >> S
On 9 January 2016 at 01:37, Derek Buitenhuis wrote:
> On 1/8/2016 2:19 PM, Jean-Yves Avenard wrote:
>> I will come on Monday if that's okay... could setup a time that works
>> for you. I'm based in Melbourne Australia
>
> Most of the channel is usually around during daytime GMT/CET hours.
>
Happy
Hi,
On Wed, Jan 13, 2016 at 7:48 PM, Ganesh Ajjanagadde
wrote:
> On Wed, Jan 13, 2016 at 7:39 PM, Ronald S. Bultje
> wrote:
> > Hi,
> >
> > On Wed, Jan 13, 2016 at 7:10 PM, Ganesh Ajjanagadde <
> gajjanaga...@gmail.com>
> > wrote:
> >>
> >> Signed-off-by: Ganesh Ajjanagadde
> >> ---
> >> conf
On Wed, Jan 13, 2016 at 7:39 PM, Ronald S. Bultje wrote:
> Hi,
>
> On Wed, Jan 13, 2016 at 7:10 PM, Ganesh Ajjanagadde
> wrote:
>>
>> Signed-off-by: Ganesh Ajjanagadde
>> ---
>> configure| 1 +
>> libavutil/libm.h | 4
>> 2 files changed, 5 insertions(+)
>>
>> diff --git a/configur
Hi,
On Wed, Jan 13, 2016 at 7:10 PM, Ganesh Ajjanagadde
wrote:
> Signed-off-by: Ganesh Ajjanagadde
> ---
> configure| 1 +
> libavutil/libm.h | 4
> 2 files changed, 5 insertions(+)
>
> diff --git a/configure b/configure
> index 415d939..a3aaa25 100755
> --- a/configure
> +++ b/co
On 13.01.2016 20:07, Michael Niedermayer wrote:
> On Wed, Jan 13, 2016 at 12:56:39AM +0100, Andreas Cadhalpun wrote:
>> src and dst are only 8-bit-aligned, so accessing them as uint16_t causes
>> SIGBUS crashes on architectures like sparc.
>>
>> This fixes ubsan runtime error: load of misaligned ad
On 13.01.2016 20:06, Michael Niedermayer wrote:
> On Wed, Jan 13, 2016 at 12:52:21AM +0100, Andreas Cadhalpun wrote:
>> ---
>> libavutil/common.h | 2 +-
>> 1 file changed, 1 insertion(+), 1 deletion(-)
>>
>> diff --git a/libavutil/common.h b/libavutil/common.h
>> index f3276a2..5ae2847 100644
>>
Signed-off-by: Ganesh Ajjanagadde
---
configure| 1 +
libavutil/libm.h | 4
2 files changed, 5 insertions(+)
diff --git a/configure b/configure
index 415d939..a3aaa25 100755
--- a/configure
+++ b/configure
@@ -1821,6 +1821,7 @@ MATH_FUNCS="
exp2f
expf
hypot
+isfin
On Wed, Jan 13, 2016 at 6:28 PM, Hendrik Leppkes wrote:
> On Thu, Jan 14, 2016 at 12:24 AM, Ganesh Ajjanagadde wrote:
>> On Wed, Jan 13, 2016 at 6:16 PM, Michael Niedermayer
>> wrote:
>>> On Wed, Jan 13, 2016 at 06:03:32PM -0500, Ganesh Ajjanagadde wrote:
On Wed, Jan 13, 2016 at 5:53 PM, Mi
On Wed, Jan 13, 2016 at 6:59 PM, Ganesh Ajjanagadde
wrote:
> This improves accuracy (very slightly) and speed for processors having
> fma3.
>
> Sample benchmark (fate flac-16-lpc-cholesky, Haswell):
> old:
> 5993610 decicycles in ff_lpc_calc_coefs, 64 runs, 0 skips
> 5951528 decicycles i
This improves accuracy (very slightly) and speed for processors having
fma3.
Sample benchmark (fate flac-16-lpc-cholesky, Haswell):
old:
5993610 decicycles in ff_lpc_calc_coefs, 64 runs, 0 skips
5951528 decicycles in ff_lpc_calc_coefs, 128 runs, 0 skips
new:
5252410 decicycles
On Thu, Jan 14, 2016 at 12:24 AM, Ganesh Ajjanagadde wrote:
> On Wed, Jan 13, 2016 at 6:16 PM, Michael Niedermayer
> wrote:
>> On Wed, Jan 13, 2016 at 06:03:32PM -0500, Ganesh Ajjanagadde wrote:
>>> On Wed, Jan 13, 2016 at 5:53 PM, Michael Niedermayer
>>> wrote:
>>> > ffmpeg | branch: master |
On 13.01.2016 19:18, foo86 wrote:
> On Tue, Jan 12, 2016 at 11:58:43PM +0100, Andreas Cadhalpun wrote:
>> It's not completely fixed yet, because the check is only done if
>> static_fields_present is 1.
>
> You are right, placing the check there doesn't account for case when
> static_fields_present
On Wed, Jan 13, 2016 at 6:16 PM, Michael Niedermayer
wrote:
> On Wed, Jan 13, 2016 at 06:03:32PM -0500, Ganesh Ajjanagadde wrote:
>> On Wed, Jan 13, 2016 at 5:53 PM, Michael Niedermayer
>> wrote:
>> > ffmpeg | branch: master | Michael Niedermayer |
>> > Wed Jan 13 22:33:59 2016 +0100| [92465a2
On 13.01.2016 08:01, Luca Barbato wrote:
> On 13/01/16 00:56, Andreas Cadhalpun wrote:
>> src and dst are only 8-bit-aligned, so accessing them as uint16_t causes
>> SIGBUS crashes on architectures like sparc.
>>
>> This fixes ubsan runtime error: load of misaligned address for type
>> 'const uint1
On Wed, Jan 13, 2016 at 06:03:32PM -0500, Ganesh Ajjanagadde wrote:
> On Wed, Jan 13, 2016 at 5:53 PM, Michael Niedermayer
> wrote:
> > ffmpeg | branch: master | Michael Niedermayer |
> > Wed Jan 13 22:33:59 2016 +0100| [92465a2347d959cbd9864b017a39b2a4ab9313ff]
> > | committer: Michael Nieder
On Thu, Jan 14, 2016 at 01:41:51AM +0300, Andrey Turkin wrote:
> 2016-01-13 22:32 GMT+03:00 Michael Niedermayer :
>
> > to calculate the phase
> >
> > the area that has alternating 0/1 values can be correlated with a
> > sin() and a cos(), if you consider the 2 resulting values as x and y
> > coor
Slicer is a part of the library and as such is under LGPL; in fact FFmpeg
already uses the library to decode DVB teletext.
2016-01-14 1:27 GMT+03:00 Carl Eugen Hoyos :
> Andrey Turkin gmail.com> writes:
>
> > Why not use libzvbi's slicer? It should be pretty
> > robust with less-than-ideal sig
On Wed, Jan 13, 2016 at 5:53 PM, Michael Niedermayer wrote:
> ffmpeg | branch: master | Michael Niedermayer | Wed
> Jan 13 22:33:59 2016 +0100| [92465a2347d959cbd9864b017a39b2a4ab9313ff] |
> committer: Michael Niedermayer
>
> avcodec/aacenc: Check for +-Inf too
>
> Fixes out of array read
> Fix
From: Michael Niedermayer
It seems the previous tables where calculated with 32bit integers ignoring
overflows.
Also check for the max qindex, the value is choosen so that the qfactor/offset
fit in int32.
Fixes:
070b7914fd5dfe8f93248bea71363410/asan_static-oob_c8d034_2764_258e20f4a3c79158aecddb
2016-01-13 22:32 GMT+03:00 Michael Niedermayer :
> to calculate the phase
>
> the area that has alternating 0/1 values can be correlated with a
> sin() and a cos(), if you consider the 2 resulting values as x and y
> coordinates the angle they form from the origin is the phase.
> (this can be word
Andrey Turkin gmail.com> writes:
> Why not use libzvbi's slicer? It should be pretty
> robust with less-than-ideal signal.
It is not entirely available under LGPL afaict.
Carl Eugen
___
ffmpeg-devel mailing list
ffmpeg-devel@ffmpeg.org
http://ffmpeg
Hi Will,
I think we addressed all the review comments in this patch. Which are as
follows:
1. Remove inline asm for color conversion and replace with swscale, which I
did.
2. Replace static variables with #defines, which I did.
3. Remove unnecessary stuff from configure script, which I did.
4. Us
On Wed, Jan 13, 2016 at 05:01:01AM +0300, Andrey Turkin wrote:
> Why not use libzvbi's slicer? It should be pretty robust with
> less-than-ideal signal.
>
> Regarding your code - is there a need to calculate a frequency? I did
> something similar a while back with VBI from TV signal and in my expe
On Wed, Jan 13, 2016 at 12:56:39AM +0100, Andreas Cadhalpun wrote:
> src and dst are only 8-bit-aligned, so accessing them as uint16_t causes
> SIGBUS crashes on architectures like sparc.
>
> This fixes ubsan runtime error: load of misaligned address for type
> 'const uint16_t', which requires 2 b
On Wed, Jan 13, 2016 at 12:52:21AM +0100, Andreas Cadhalpun wrote:
> ---
> libavutil/common.h | 2 +-
> 1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/libavutil/common.h b/libavutil/common.h
> index f3276a2..5ae2847 100644
> --- a/libavutil/common.h
> +++ b/libavutil/common.h
> @@
On Tue, Jan 12, 2016 at 11:58:43PM +0100, Andreas Cadhalpun wrote:
> It's not completely fixed yet, because the check is only done if
> static_fields_present is 1.
You are right, placing the check there doesn't account for case when
static_fields_present becomes 0 later. I moved the check closer t
On Wed, Jan 13, 2016 at 1:14 PM, wm4 wrote:
> On Wed, 13 Jan 2016 10:48:01 -0500
> Ganesh Ajjanagadde wrote:
>
>> On Wed, Jan 13, 2016 at 4:05 AM, wm4 wrote:
>> > On Tue, 12 Jan 2016 10:07:08 -0500
>> > Ganesh Ajjanagadde wrote:
>> >
>> >> On Tue, Jan 12, 2016 at 9:43 AM, Ronald S. Bultje
>>
On Wed, 13 Jan 2016 10:48:01 -0500
Ganesh Ajjanagadde wrote:
> On Wed, Jan 13, 2016 at 4:05 AM, wm4 wrote:
> > On Tue, 12 Jan 2016 10:07:08 -0500
> > Ganesh Ajjanagadde wrote:
> >
> >> On Tue, Jan 12, 2016 at 9:43 AM, Ronald S. Bultje
> >> wrote:
> >> > Hi,
> >> >
> >> > On Tue, Jan 12, 2
On Wed, Jan 13, 2016 at 04:12:17PM +, Rostislav Pehlivanov wrote:
> On Tue, 2016-01-12 at 23:32 +0100, Michael Niedermayer wrote:
> > From: Michael Niedermayer
> >
> > encoding changes from 17 to 20 fps
> >
> > Signed-off-by: Michael Niedermayer
> > ---
> > libavcodec/diracdec.c | 18 +++
On Wed, Jan 13, 2016 at 04:12:54PM +, Rostislav Pehlivanov wrote:
> On Tue, 2016-01-12 at 23:32 +0100, Michael Niedermayer wrote:
> > From: Michael Niedermayer
> >
> > Signed-off-by: Michael Niedermayer
> > ---
> > libavcodec/diracdec.c | 10 +-
> > 1 file changed, 5 insertions(+)
On Wed, Jan 13, 2016 at 04:11:40PM +, Rostislav Pehlivanov wrote:
> On Tue, 2016-01-12 at 23:32 +0100, Michael Niedermayer wrote:
> > From: Michael Niedermayer
> >
> > This avoids closing and opening the bit reader
> >
> > Signed-off-by: Michael Niedermayer
> > ---
> > libavcodec/diracdec.
On Wed, Dec 30, 2015 at 10:44 PM, Claudio Freire wrote:
> On Tue, Dec 29, 2015 at 1:12 PM, Claudio Freire
> wrote:
>> On Tue, Dec 29, 2015 at 6:18 AM, Rostislav Pehlivanov
>> wrote:
>>> Wouldn't it be simpler to just check if the maximum codebook was 0 after
>>> calculating cost1 and skipping t
On Tue, 2016-01-12 at 23:32 +0100, Michael Niedermayer wrote:
> From: Michael Niedermayer
>
> Signed-off-by: Michael Niedermayer
> ---
> libavcodec/diracdec.c | 10 +-
> 1 file changed, 5 insertions(+), 5 deletions(-)
>
LGTM,
Thanks
___
f
On Tue, 2016-01-12 at 23:32 +0100, Michael Niedermayer wrote:
> From: Michael Niedermayer
>
> encoding changes from 17 to 20 fps
>
> Signed-off-by: Michael Niedermayer
> ---
> libavcodec/diracdec.c | 18 --
> 1 file changed, 12 insertions(+), 6 deletions(-)
>
LGTM,
Thanks
On Tue, 2016-01-12 at 23:32 +0100, Michael Niedermayer wrote:
> From: Michael Niedermayer
>
> This avoids closing and opening the bit reader
>
> Signed-off-by: Michael Niedermayer
> ---
> libavcodec/diracdec.c | 37 ++---
> 1 file changed, 34 insertions(+), 3
On Tue, 2016-01-12 at 21:15 +0100, Michael Niedermayer wrote:
> On Tue, Jan 12, 2016 at 01:01:24AM +, Kieran Kunhya wrote:
> >
>
> > diracdec.c | 38 ++
> > 1 file changed, 30 insertions(+), 8 deletions(-)
> > 73ce8405d52477db0faa2a2b76daca698a49bff2 00
Around 35% faster than the avx version.
---
libavcodec/v210enc.c | 5 ++--
libavcodec/v210enc.h | 1 +
libavcodec/x86/v210enc.asm| 53 +++
libavcodec/x86/v210enc_init.c | 7 ++
4 files changed, 49 insertions(+), 17 deletions(-)
On Wed, Jan 13, 2016 at 4:05 AM, wm4 wrote:
> On Tue, 12 Jan 2016 10:07:08 -0500
> Ganesh Ajjanagadde wrote:
>
>> On Tue, Jan 12, 2016 at 9:43 AM, Ronald S. Bultje wrote:
>> > Hi,
>> >
>> > On Tue, Jan 12, 2016 at 7:52 AM, Ganesh Ajjanagadde
>> > wrote:
>> >
>> >> On Tue, Jan 12, 2016 at 4:38 A
Match the use of AV_PIX_FMT_PAL8 for 1-bit QuickTime Animation in
lavc/qtrle. To reiterate, 1-bit video is not necessary black & white in
QuickTime, merely bi-level. The two colors can be any color. The palette,
either included in the sample description, or the default Macintosh
palette (black & w
On Wed, 13 Jan 2016 at 02:16 Andrey Turkin wrote:
> Why not use libzvbi's slicer? It should be pretty robust with
> less-than-ideal signal.
>
(now with a proper response).
IMO FFmpeg shouldn't be doing analogue signal processing. This should be
left to libzvbi.
Kieran
__
On Wed, 13 Jan 2016 at 02:16 Andrey Turkin wrote:
> Why not use libzvbi's slicer? It should be pretty robust with
> less-than-ideal signal.
>
___
ffmpeg-devel mailing list
ffmpeg-devel@ffmpeg.org
http://ffmpeg.org/mailman/listinfo/ffmpeg-devel
On 01/13/2016 09:06 AM, Mats Peterson wrote:
slightly misleading, since it says that the 'raw ' fourcc stands for
"Uncompressed RGB". However, below are two files with the 'raw ' fourcc
I suppose what they mean by "uncompressed RGB" here is just a way to
differentiate it from uncompressed YUV4
On Wed, 13 Jan 2016 08:57:30 +0100
Clément Bœsch wrote:
> On Tue, Jan 12, 2016 at 05:42:59PM -0800, Aman Gupta wrote:
> > From: Aman Gupta
> >
> > Positioning math is based on the guidelines in
> > https://dvcs.w3.org/hg/text-tracks/raw-file/default/608toVTT/608toVTT.html#positioning-in-cea-608
On Tue, 12 Jan 2016 20:51:46 -0600
Rodger Combs wrote:
> ---
> libavcodec/avcodec.h| 2 ++
> libavcodec/mlp_parser.c | 5 +
> libavcodec/mlpdec.c | 2 ++
> libavcodec/profiles.c | 6 ++
> libavcodec/profiles.h | 1 +
> 5 files changed, 16 insertions(+)
>
> diff --git a/libav
On Tue, 12 Jan 2016 10:07:08 -0500
Ganesh Ajjanagadde wrote:
> On Tue, Jan 12, 2016 at 9:43 AM, Ronald S. Bultje wrote:
> > Hi,
> >
> > On Tue, Jan 12, 2016 at 7:52 AM, Ganesh Ajjanagadde
> > wrote:
> >
> >> On Tue, Jan 12, 2016 at 4:38 AM, wm4 wrote:
> >> > This makes no sense. Even if fc
On 13 January 2016 7:12:55 AM IST, Aman Gupta wrote:
>From: Aman Gupta
>
>This ensures that captions are written towards the bottom of the screen
>when tuning into mid-stream. The row will be reset on the receipt of
>the
>next PAC command. Row 10 was chosen as it corresponds to the value of
>"0
On 13 January 2016 7:12:54 AM IST, Aman Gupta wrote:
>From: Aman Gupta
>
>---
> tests/fate/subtitles.mak | 3 +++
>tests/ref/fate/sub-cc-realtime | 42
>++
> 2 files changed, 45 insertions(+)
> create mode 100644 tests/ref/fate/sub-cc-realtime
>
>dif
On 13 January 2016 7:12:53 AM IST, Aman Gupta wrote:
>From: Aman Gupta
>
>This new mode is useful for realtime decoding of closed captions so
>they
>can be display along with mpeg2 frames.
>
>Closed caption streams contain two major types of captions:
>
>- POPON captions, which are buffered off
On 13 January 2016 7:12:52 AM IST, Aman Gupta wrote:
>From: Aman Gupta
>
>---
> libavcodec/ccaption_dec.c | 21 +
> 1 file changed, 21 insertions(+)
>
>diff --git a/libavcodec/ccaption_dec.c b/libavcodec/ccaption_dec.c
>index ca497e5..a9dfc94 100644
>--- a/libavcodec/ccaption
Michael Niedermayer niedermayer.cc> writes:
> > New patch attached that fixes ticket #5151.
>
> LGTM
Patch applied.
Thank you, Carl Eugen
___
ffmpeg-devel mailing list
ffmpeg-devel@ffmpeg.org
http://ffmpeg.org/mailman/listinfo/ffmpeg-devel
MPlayer seems to naively assume that a QuickTime file (or Matroska file
with V_QUICKTIME, for that matter) with the video fourcc 'raw ' contains
24-bit RGB video data. The QuickTime File Format Specification is
slightly misleading, since it says that the 'raw ' fourcc stands for
"Uncompressed R
Michael Niedermayer niedermayer.cc> writes:
> > mpjpegdec.c |3 +++
> > 1 file changed, 3 insertions(+)
> > acd4f042b52c6400a6aaf613b89ecad9ca3405ac patchmpjpeglog.diff
>
> should be ok
Patch applied.
Thank you, Carl Eugen
___
ffmpeg-devel mai
60 matches
Mail list logo
