From: Sebastian Dröge
Signed-off-by: Sebastian Dröge
---
libavformat/mpegtsenc.c | 43 ---
1 file changed, 40 insertions(+), 3 deletions(-)
diff --git a/libavformat/mpegtsenc.c b/libavformat/mpegtsenc.c
index 0674064..11ce130 100644
--- a/libavformat/mpe
Fixes an issue where an int64_t ffurl_seek return-value was being stored
in a generic int "r" variable, leading to integer overflow when seeking
into a large file (>2GB), and ultimately a "Failed to perform internal
seek" error mesage. The "r" variable was used both for storing the
result of any se
On Sun, Nov 1, 2015 at 4:45 PM Michael Niedermayer
wrote:
> On Sun, Nov 01, 2015 at 10:40:03AM -0800, Timothy Gu wrote:
> > ---
> > libavfilter/vf_boxblur.c | 110
> +++
> > 1 file changed, 44 insertions(+), 66 deletions(-)
>
> LGTM
> thx
>
Pushed, th
Also rearranged the relevant check to reduce code duplication
---
ffmpeg.c | 15 +++
1 file changed, 7 insertions(+), 8 deletions(-)
diff --git a/ffmpeg.c b/ffmpeg.c
index dbb2520..526f094 100644
--- a/ffmpeg.c
+++ b/ffmpeg.c
@@ -1760,7 +1760,6 @@ static void do_streamcopy(InputStream
On Sun, Nov 1, 2015 at 8:23 PM, highgod0401 wrote:
>
> From: Ganesh Ajjanagadde
> Date: 2015-10-29 10:53
> To: FFmpeg development discussions and patches
> CC: Ganesh Ajjanagadde
> Subject: Re: [FFmpeg-devel] [PATCH] avutil/opencl_internal: add
> av_warn_unused_result
> On Thu, Oct 15, 2015 at 6:2
On Sun, Nov 1, 2015 at 8:23 PM, Michael Niedermayer
wrote:
> On Sun, Nov 01, 2015 at 11:57:54AM -0500, Ganesh Ajjanagadde wrote:
>> This converts the usage of strtod to av_strtod in order to be more free
>> in accepting double parameters.
>
> "more free" is a bit unclear and confusing
> the change
On Sun, Nov 01, 2015 at 07:59:08PM -0500, Ronald S. Bultje wrote:
> Hi,
>
> On Sun, Nov 1, 2015 at 7:21 PM, Ganesh Ajjanagadde wrote:
>
> > On Sun, Nov 1, 2015 at 7:15 PM, Michael Niedermayer
> > wrote:
> > > On Sun, Nov 01, 2015 at 06:51:19PM -0500, Ganesh Ajjanagadde wrote:
> > >> On Sun, Nov
From: Ganesh Ajjanagadde
Date: 2015-10-29 10:53
To: FFmpeg development discussions and patches
CC: Ganesh Ajjanagadde
Subject: Re: [FFmpeg-devel] [PATCH] avutil/opencl_internal: add
av_warn_unused_result
On Thu, Oct 15, 2015 at 6:24 PM, Ganesh Ajjanagadde
wrote:
> Signed-off-by: Ganesh Ajjanagad
On Sun, Nov 01, 2015 at 11:57:54AM -0500, Ganesh Ajjanagadde wrote:
> This converts the usage of strtod to av_strtod in order to be more free
> in accepting double parameters.
"more free" is a bit unclear and confusing
the change itself should be ok
thanks
[...]
--
Michael GnuPG fingerprin
On Sun, Nov 1, 2015 at 7:59 PM, Ronald S. Bultje wrote:
> Hi,
>
> On Sun, Nov 1, 2015 at 7:21 PM, Ganesh Ajjanagadde wrote:
>
>> On Sun, Nov 1, 2015 at 7:15 PM, Michael Niedermayer
>> wrote:
>> > On Sun, Nov 01, 2015 at 06:51:19PM -0500, Ganesh Ajjanagadde wrote:
>> >> On Sun, Nov 1, 2015 at 6:2
Hi,
On Sun, Nov 1, 2015 at 7:21 PM, Ganesh Ajjanagadde wrote:
> On Sun, Nov 1, 2015 at 7:15 PM, Michael Niedermayer
> wrote:
> > On Sun, Nov 01, 2015 at 06:51:19PM -0500, Ganesh Ajjanagadde wrote:
> >> On Sun, Nov 1, 2015 at 6:25 PM, Nicolas George wrote:
> >> > Le primidi 11 brumaire, an CCXX
On Sun, Nov 01, 2015 at 10:40:03AM -0800, Timothy Gu wrote:
> ---
> libavfilter/vf_boxblur.c | 110
> +++
> 1 file changed, 44 insertions(+), 66 deletions(-)
LGTM
thx
[...]
--
Michael GnuPG fingerprint: 9FF2128B147EF6730BADF133611EC787040B0FAB
T
On Sun, Nov 1, 2015 at 5:10 PM, Michael Niedermayer
wrote:
> On Sun, Nov 01, 2015 at 11:57:53AM -0500, Ganesh Ajjanagadde wrote:
>> Signed-off-by: Ganesh Ajjanagadde
>> ---
>> libavutil/eval.h | 2 +-
>> 1 file changed, 1 insertion(+), 1 deletion(-)
>>
>> diff --git a/libavutil/eval.h b/libavuti
FFDIFFSIGN was created explicitly for this purpose, since the common
return a - b idiom is unsafe regarding overflow on signed integers. It
optimizes to branchless code on common compilers.
FFDIFFSIGN also has the subjective benefit of being easier to read due
to lack of ternary operators.
Tested
On Sun, Nov 1, 2015 at 6:26 PM, Michael Niedermayer
wrote:
> On Sun, Nov 01, 2015 at 12:19:48PM -0500, Ganesh Ajjanagadde wrote:
>> FFDIFFSIGN was created explicitly for this purpose, since the common
>> return a - b idiom is unsafe regarding overflow on signed integers. It
>> optimizes to branchl
On Sun, Nov 1, 2015 at 7:15 PM, Michael Niedermayer
wrote:
> On Sun, Nov 01, 2015 at 06:51:19PM -0500, Ganesh Ajjanagadde wrote:
>> On Sun, Nov 1, 2015 at 6:25 PM, Nicolas George wrote:
>> > Le primidi 11 brumaire, an CCXXIV, Ronald S. Bultje a écrit :
>> >> So, is this a bug in llrint, or is thi
On Sun, Nov 01, 2015 at 06:51:19PM -0500, Ganesh Ajjanagadde wrote:
> On Sun, Nov 1, 2015 at 6:25 PM, Nicolas George wrote:
> > Le primidi 11 brumaire, an CCXXIV, Ronald S. Bultje a écrit :
> >> So, is this a bug in llrint, or is this a failure to use llrint, or is this
> >> different from llrint?
On Sun, Nov 01, 2015 at 05:34:38PM +, Derek Buitenhuis wrote:
> This saves a lot of muxing overhead, especially on lower bitrate
> segments.
>
> Signed-off-by: Derek Buitenhuis
> ---
> libavformat/hlsenc.c | 10 +-
> 1 file changed, 9 insertions(+), 1 deletion(-)
>
> diff --git a/li
On Sun, Nov 1, 2015 at 6:25 PM, Nicolas George wrote:
> Le primidi 11 brumaire, an CCXXIV, Ronald S. Bultje a écrit :
>> So, is this a bug in llrint, or is this a failure to use llrint, or is this
>> different from llrint? It sounds to me that llrint should be used, not our
>> own alternative.
>
>
On Sun, Nov 1, 2015 at 6:20 PM, Nicolas George wrote:
> Le primidi 11 brumaire, an CCXXIV, Ganesh Ajjanagadde a écrit :
>> Can't users give arbitrarily long arithmetic expressions?
>
> They can, and the current code deals gracefully with them.
ok, all patches except 2/4 (which maybe a valid typo
On Sat, Oct 31, 2015 at 10:47:55AM -0400, Ganesh Ajjanagadde wrote:
> There seems to be some typos in the log messages that are fixed by this.
probably ok
[...]
--
Michael GnuPG fingerprint: 9FF2128B147EF6730BADF133611EC787040B0FAB
He who knows, does not speak. He who speaks, does not know.
On Mon, 2 Nov 2015, Nicolas George wrote:
Le primidi 11 brumaire, an CCXXIV, Marton Balint a écrit :
Because I add it to the metadata store here in the next patch, which
metadata will be set for every file packet, so calculating it at the end of
file is not an option.
I see. But do you reall
On Sun, Nov 01, 2015 at 12:19:48PM -0500, Ganesh Ajjanagadde wrote:
> FFDIFFSIGN was created explicitly for this purpose, since the common
> return a - b idiom is unsafe regarding overflow on signed integers. It
> optimizes to branchless code on common compilers.
>
> FFDIFFSIGN also has the subjec
Le primidi 11 brumaire, an CCXXIV, Ronald S. Bultje a écrit :
> So, is this a bug in llrint, or is this a failure to use llrint, or is this
> different from llrint? It sounds to me that llrint should be used, not our
> own alternative.
Is there a sized version of the function? int64rint? Otherwise
On Sun, Nov 1, 2015 at 6:18 PM, Ronald S. Bultje wrote:
> Hi,
>
> On Sun, Nov 1, 2015 at 6:15 PM, Ganesh Ajjanagadde
> wrote:
>>
>> On Sun, Nov 1, 2015 at 6:12 PM, Ronald S. Bultje
>> wrote:
>> > Hi,
>> >
>> > On Sun, Nov 1, 2015 at 1:03 PM, Ganesh Ajjanagadde
>> >
>> > wrote:
>> >>
>> >> Float
Le primidi 11 brumaire, an CCXXIV, Ganesh Ajjanagadde a écrit :
> Can't users give arbitrarily long arithmetic expressions?
They can, and the current code deals gracefully with them.
Regards,
--
Nicolas George
signature.asc
Description: Digital signature
Hi,
On Sun, Nov 1, 2015 at 6:15 PM, Ganesh Ajjanagadde
wrote:
> On Sun, Nov 1, 2015 at 6:12 PM, Ronald S. Bultje
> wrote:
> > Hi,
> >
> > On Sun, Nov 1, 2015 at 1:03 PM, Ganesh Ajjanagadde <
> gajjanaga...@gmail.com>
> > wrote:
> >>
> >> Floating point to integer conversion is well defined when
This option can be used to select useful frames from an ffconcat file which is
using inpoints and outpoints but where the source files are not intra frame
only.
Signed-off-by: Marton Balint
---
doc/filters.texi | 16
libavfilter/f_select.c | 20
2 file
Signed-off-by: Marton Balint
---
doc/demuxers.texi | 8
libavformat/concatdec.c | 12
2 files changed, 20 insertions(+)
diff --git a/doc/demuxers.texi b/doc/demuxers.texi
index 34bfc9b..41d3722 100644
--- a/doc/demuxers.texi
+++ b/doc/demuxers.texi
@@ -204,6 +204,14
Signed-off-by: Marton Balint
---
libavformat/concatdec.c | 19 ---
1 file changed, 12 insertions(+), 7 deletions(-)
diff --git a/libavformat/concatdec.c b/libavformat/concatdec.c
index 0180a7e..15094bf 100644
--- a/libavformat/concatdec.c
+++ b/libavformat/concatdec.c
@@ -285,6 +
Signed-off-by: Marton Balint
---
tests/Makefile |1 +
tests/extended.ffconcat| 114 +
tests/fate-run.sh | 20 +
tests/fate/concatdec.mak | 21 +
tests/ref/fate/conca
On Sun, Nov 1, 2015 at 6:12 PM, Ronald S. Bultje wrote:
> Hi,
>
> On Sun, Nov 1, 2015 at 1:03 PM, Ganesh Ajjanagadde
> wrote:
>>
>> Floating point to integer conversion is well defined when the float lies
>> within
>> the representation bounds of the integer after discarding the fractional
>> par
On Sun, Nov 1, 2015 at 6:09 PM, Ronald S. Bultje wrote:
> Hi,
>
> On Sun, Nov 1, 2015 at 1:03 PM, Ganesh Ajjanagadde
> wrote:
>>
>> The rationale for this function is reflected in the documentation for
>> it, and is copied here:
>>
>> Clip a double value into the long long amin-amax range.
>> Thi
Hi,
On Sun, Nov 1, 2015 at 1:03 PM, Ganesh Ajjanagadde
wrote:
> Floating point to integer conversion is well defined when the float lies
> within
> the representation bounds of the integer after discarding the fractional
> part.
> However, in other cases, unfortunately the standard leaves it und
Hi,
On Sun, Nov 1, 2015 at 1:03 PM, Ganesh Ajjanagadde
wrote:
> The rationale for this function is reflected in the documentation for
> it, and is copied here:
>
> Clip a double value into the long long amin-amax range.
> This function is needed because conversion of floating point to integers
>
On Sun, Nov 1, 2015 at 5:46 PM, Nicolas George wrote:
> Le primidi 11 brumaire, an CCXXIV, Michael Niedermayer a écrit :
>> less fragile could be ret < 0 && ret != the error code generated by NaN
>> i dont know what to do about the clutter or if this approuch makes
>> sense or if just droping the
Le primidi 11 brumaire, an CCXXIV, Marton Balint a écrit :
> Because I add it to the metadata store here in the next patch, which
> metadata will be set for every file packet, so calculating it at the end of
> file is not an option.
I see. But do you really need the duration metadata when it is no
On Fri, 30 Oct 2015, Nicolas George wrote:
Le quartidi 4 brumaire, an CCXXIV, Marton Balint a écrit :
Hmm. I need this computed here for the next patch. Maybe we could calcualate
the duration here and then update it in open_next_file as well?
That is probably possible somehow but a bit tricky
On Sun, Nov 01, 2015 at 12:19:47PM -0500, Ganesh Ajjanagadde wrote:
> This is of use for defining comparator callbacks. Common approaches like
> return x-y are not safe due to the risks of overflow.
> Furthermore, the (x > y) - (x < y) trick is optimized to branchless
> code.
> This also documents
Le primidi 11 brumaire, an CCXXIV, Michael Niedermayer a écrit :
> less fragile could be ret < 0 && ret != the error code generated by NaN
> i dont know what to do about the clutter or if this approuch makes
> sense or if just droping the return warning for av_expr_parse_and_eval
> would be better
On Sun, Nov 01, 2015 at 04:16:18PM -0500, Ganesh Ajjanagadde wrote:
> On Sun, Nov 1, 2015 at 3:45 PM, Nicolas George wrote:
> > Le primidi 11 brumaire, an CCXXIV, Ganesh Ajjanagadde a écrit :
> >> This function can return ENOMEM that needs to be propagated.
> >>
> >> Signed-off-by: Ganesh Ajjanaga
On Sun, Nov 01, 2015 at 11:57:53AM -0500, Ganesh Ajjanagadde wrote:
> Signed-off-by: Ganesh Ajjanagadde
> ---
> libavutil/eval.h | 2 +-
> 1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/libavutil/eval.h b/libavutil/eval.h
> index 6159b0f..dacd22b 100644
> --- a/libavutil/eval.h
>
From: Sebastian Dröge
Signed-off-by: Sebastian Dröge
---
libavformat/mpegtsenc.c | 47 ---
1 file changed, 44 insertions(+), 3 deletions(-)
diff --git a/libavformat/mpegtsenc.c b/libavformat/mpegtsenc.c
index 0674064..51d1a0b 100644
--- a/libavformat
From: Sebastian Dröge
Signed-off-by: Sebastian Dröge
---
libavformat/mpegtsenc.c | 180 +++-
1 file changed, 179 insertions(+), 1 deletion(-)
diff --git a/libavformat/mpegtsenc.c b/libavformat/mpegtsenc.c
index 4d74252..0674064 100644
--- a/libavform
On So, 2015-11-01 at 22:20 +0200, Sebastian Dröge wrote:
> +data = av_malloc(pkt->size + 2 + pkt->size / 255 + 1);
> [...]
> +n = pkt->size;
> +i = 2;
> +do {
> +data[i] = FFMIN(n, 255);
> +n -= 255;
> +
On Sun, Nov 1, 2015 at 3:45 PM, Nicolas George wrote:
> Le primidi 11 brumaire, an CCXXIV, Ganesh Ajjanagadde a écrit :
>> This function can return ENOMEM that needs to be propagated.
>>
>> Signed-off-by: Ganesh Ajjanagadde
>> ---
>> libavfilter/vf_pad.c| 12 +++-
>> libavfilter/vf_r
Le primidi 11 brumaire, an CCXXIV, Ganesh Ajjanagadde a écrit :
> This function can return ENOMEM that needs to be propagated.
>
> Signed-off-by: Ganesh Ajjanagadde
> ---
> libavfilter/vf_pad.c| 12 +++-
> libavfilter/vf_rotate.c | 5 +++--
> libavfilter/vf_scale.c | 5 +++--
> li
On Sun, Nov 1, 2015 at 2:14 PM, Michael Niedermayer
wrote:
> On Sat, Oct 31, 2015 at 10:47:54AM -0400, Ganesh Ajjanagadde wrote:
>> This function can return ENOMEM that needs to be propagated.
>>
>> Signed-off-by: Ganesh Ajjanagadde
>> ---
>> libavfilter/vf_pad.c| 10 ++
>> libavfilt
This function can return ENOMEM that needs to be propagated.
Signed-off-by: Ganesh Ajjanagadde
---
libavfilter/vf_pad.c| 12 +++-
libavfilter/vf_rotate.c | 5 +++--
libavfilter/vf_scale.c | 5 +++--
libavfilter/vf_zscale.c | 5 +++--
4 files changed, 16 insertions(+), 11 deletion
On Sun, Nov 1, 2015 at 1:33 PM, Ganesh Ajjanagadde wrote:
> On Sun, Nov 1, 2015 at 12:52 PM, Michael Niedermayer
> wrote:
>> On Sat, Oct 31, 2015 at 10:47:56AM -0400, Ganesh Ajjanagadde wrote:
>>> This returns the error code from main in the test, in this case ENOMEM.
>>> This should not matter,
This returns the error code from main in the test in the case of ENOMEM.
This should not matter, and will ensure that no warnings are triggered
when av_warn_unused_result is added to avutil/eval.h.
Tested with FATE.
Signed-off-by: Ganesh Ajjanagadde
---
libavutil/eval.c | 22 +--
From: Sebastian Dröge
Signed-off-by: Sebastian Dröge
---
libavformat/mpegtsenc.c | 47 ---
1 file changed, 44 insertions(+), 3 deletions(-)
diff --git a/libavformat/mpegtsenc.c b/libavformat/mpegtsenc.c
index a7e78ac..023ca02 100644
--- a/libavformat
From: Sebastian Dröge
Signed-off-by: Sebastian Dröge
---
libavformat/mpegtsenc.c | 180 +++-
1 file changed, 179 insertions(+), 1 deletion(-)
diff --git a/libavformat/mpegtsenc.c b/libavformat/mpegtsenc.c
index 4d74252..a7e78ac 100644
--- a/libavform
Hi everybody,
the following patches are adding support for muxing Opus into MPEG-TS. ffmpeg
could already demux such files since a while, but I guess it would also be
nice to be able to create them.
Please review carefully, thanks!
Sebastian
___
ffmpe
On Sun, Nov 1, 2015 at 2:14 PM, Michael Niedermayer
wrote:
> On Sat, Oct 31, 2015 at 10:47:54AM -0400, Ganesh Ajjanagadde wrote:
>> This function can return ENOMEM that needs to be propagated.
>>
>> Signed-off-by: Ganesh Ajjanagadde
>> ---
>> libavfilter/vf_pad.c| 10 ++
>> libavfilt
On Sun, Nov 1, 2015 at 11:10 AM Ganesh Ajjanagadde wrote:
> Have not tested, but just a general comment. Personally, I follow the
> twice repition is ok, thrice means it is good to factor out. I would
> have been happier if the diff-stat was better than 44+/66- in terms of
> deletions.
The most
On Sat, Oct 31, 2015 at 10:47:54AM -0400, Ganesh Ajjanagadde wrote:
> This function can return ENOMEM that needs to be propagated.
>
> Signed-off-by: Ganesh Ajjanagadde
> ---
> libavfilter/vf_pad.c| 10 ++
> libavfilter/vf_rotate.c | 3 +--
> libavfilter/vf_scale.c | 5 +++--
> li
On Sun, Nov 1, 2015 at 1:40 PM, Timothy Gu wrote:
> ---
> libavfilter/vf_boxblur.c | 110
> +++
> 1 file changed, 44 insertions(+), 66 deletions(-)
>
> diff --git a/libavfilter/vf_boxblur.c b/libavfilter/vf_boxblur.c
> index ef01cf9..6934076 100644
> -
---
libavfilter/vf_boxblur.c | 110 +++
1 file changed, 44 insertions(+), 66 deletions(-)
diff --git a/libavfilter/vf_boxblur.c b/libavfilter/vf_boxblur.c
index ef01cf9..6934076 100644
--- a/libavfilter/vf_boxblur.c
+++ b/libavfilter/vf_boxblur.c
@@ -20
On Sun, Nov 1, 2015 at 12:52 PM, Michael Niedermayer
wrote:
> On Sat, Oct 31, 2015 at 10:47:56AM -0400, Ganesh Ajjanagadde wrote:
>> This returns the error code from main in the test, in this case ENOMEM.
>> This should not matter, and will ensure that no warnings are triggered
>> when av_warn_unu
This uses av_strtod for added flexibility, and av_clipd64 for ensuring that
no undefined behavior gets invoked.
Signed-off-by: Ganesh Ajjanagadde
---
ffserver_config.c | 21 +
1 file changed, 5 insertions(+), 16 deletions(-)
diff --git a/ffserver_config.c b/ffserver_config.c
The rationale for this function is reflected in the documentation for
it, and is copied here:
Clip a double value into the long long amin-amax range.
This function is needed because conversion of floating point to integers when
it does not fit in the integer's representation does not necessarily s
Floating point to integer conversion is well defined when the float lies within
the representation bounds of the integer after discarding the fractional part.
However, in other cases, unfortunately the standard leaves it undefined.
In particular, assuming that it saturates in a sane way is a danger
On Sat, Oct 31, 2015 at 10:47:56AM -0400, Ganesh Ajjanagadde wrote:
> This returns the error code from main in the test, in this case ENOMEM.
> This should not matter, and will ensure that no warnings are triggered
> when av_warn_unused_result is added to avutil/eval.h.
>
> Signed-off-by: Ganesh A
From: Michael Niedermayer
Fixes: ==13287== Invalid read of size 4
==13287==at 0x45161A: flush_buffer (aviobuf.c:143)
==13287==by 0x451971: avio_flush (aviobuf.c:200)
==13287==by 0x512CCF: av_write_trailer (mux.c:1016)
==13287==by 0x41A5E0: close_connection (ffserver.c:853)
==13287
From: Michael Niedermayer
Signed-off-by: Michael Niedermayer
---
ffserver.c |8 +---
1 file changed, 5 insertions(+), 3 deletions(-)
diff --git a/ffserver.c b/ffserver.c
index 526cbfc..8ddc210 100644
--- a/ffserver.c
+++ b/ffserver.c
@@ -2272,7 +2272,7 @@ static int http_prepare_data(H
This saves a lot of muxing overhead, especially on lower bitrate
segments.
Signed-off-by: Derek Buitenhuis
---
libavformat/hlsenc.c | 10 +-
1 file changed, 9 insertions(+), 1 deletion(-)
diff --git a/libavformat/hlsenc.c b/libavformat/hlsenc.c
index 8daf53f..d7884e5 100644
--- a/libavf
FFDIFFSIGN was created explicitly for this purpose, since the common
return a - b idiom is unsafe regarding overflow on signed integers. It
optimizes to branchless code on common compilers.
FFDIFFSIGN also has the subjective benefit of being easier to read due
to lack of ternary operators.
Tested
This is of use for defining comparator callbacks. Common approaches like
return x-y are not safe due to the risks of overflow.
Furthermore, the (x > y) - (x < y) trick is optimized to branchless
code.
This also documents this macro accordingly.
Signed-off-by: Ganesh Ajjanagadde
---
libavutil/com
This patch series draws upon remarks of Mark and Ronald regarding the lack
of safety of the common idiom return a - b for qsort comparators.
I made an observation that the interesting (x > y) - (x < y) idiom works well:
it not only avoids branches commonly not optimized by compilers when the terna
---
tests/checkasm/Makefile | 1 +
tests/checkasm/checkasm.c| 3 ++
tests/checkasm/checkasm.h| 1 +
tests/checkasm/pixblockdsp.c | 107 +++
4 files changed, 112 insertions(+)
create mode 100644 tests/checkasm/pixblockdsp.c
diff --git a/t
---
libavcodec/x86/pixblockdsp.asm | 66 --
1 file changed, 31 insertions(+), 35 deletions(-)
diff --git a/libavcodec/x86/pixblockdsp.asm b/libavcodec/x86/pixblockdsp.asm
index 7c5377b..a7d9816 100644
--- a/libavcodec/x86/pixblockdsp.asm
+++ b/libavcodec/x8
All filters now use av_strtod for accepting floating point parameters.
There was an isolated remark on this, but the point applies generally
now.
This moves the comment and suitably elaborates on it for additional
clarity. A link to the code for av_strtod is also provided for the
user's benefit.
This converts the usage of strtod to av_strtod in order to be more free
in accepting double parameters.
Signed-off-by: Ganesh Ajjanagadde
---
libavfilter/vf_frei0r.c | 3 ++-
1 file changed, 2 insertions(+), 1 deletion(-)
diff --git a/libavfilter/vf_frei0r.c b/libavfilter/vf_frei0r.c
index 0a98
Signed-off-by: Ganesh Ajjanagadde
---
libavutil/eval.h | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/libavutil/eval.h b/libavutil/eval.h
index 6159b0f..dacd22b 100644
--- a/libavutil/eval.h
+++ b/libavutil/eval.h
@@ -102,7 +102,7 @@ void av_expr_free(AVExpr *e);
* @param n
Patch 1/3 was just a minor typo I noticed while working on av_strtod.
Patch 2/3 offers improved flexibility in setting parameters, and it may be noted
that with this, all of libavfilter uses av_strtod as opposed to strtod.
Patch 3/3 accordingly documents it. Since all of libavfilter now uses av_s
On Sun, 1 Nov 2015 17:07:07 +0100
Paul B Mahol wrote:
> The size of decoding map can differ from one calculated
> internally, producing artifacts while decoding video.
>
> Signed-off-by: Paul B Mahol
> ---
> libavcodec/interplayvideo.c | 14 +-
> libavformat/ipmovie.c | 7 +
The size of decoding map can differ from one calculated
internally, producing artifacts while decoding video.
Signed-off-by: Paul B Mahol
---
libavcodec/interplayvideo.c | 14 +-
libavformat/ipmovie.c | 7 ---
2 files changed, 13 insertions(+), 8 deletions(-)
diff --git a
On Sun, 1 Nov 2015 13:03:50 +0100
Hendrik Leppkes wrote:
> On Sun, Nov 1, 2015 at 12:57 PM, wm4 wrote:
> > On Sun, 1 Nov 2015 11:21:26 +0100
> > Hendrik Leppkes wrote:
> >
> >> This fixes a memory leak when side-data is present.
> >> ---
> >> libavformat/utils.c | 9 -
> >> 1 file c
On Sun, Nov 1, 2015 at 12:57 PM, wm4 wrote:
> On Sun, 1 Nov 2015 11:21:26 +0100
> Hendrik Leppkes wrote:
>
>> This fixes a memory leak when side-data is present.
>> ---
>> libavformat/utils.c | 9 -
>> 1 file changed, 4 insertions(+), 5 deletions(-)
>>
>> diff --git a/libavformat/utils.
On Sun, 1 Nov 2015 11:21:26 +0100
Hendrik Leppkes wrote:
> This fixes a memory leak when side-data is present.
> ---
> libavformat/utils.c | 9 -
> 1 file changed, 4 insertions(+), 5 deletions(-)
>
> diff --git a/libavformat/utils.c b/libavformat/utils.c
> index 7e4f54f..3f82659 100644
Fixes a memory leak when using genpts
---
libavformat/utils.c | 1 +
1 file changed, 1 insertion(+)
diff --git a/libavformat/utils.c b/libavformat/utils.c
index 3f82659..5c4d452 100644
--- a/libavformat/utils.c
+++ b/libavformat/utils.c
@@ -1559,6 +1559,7 @@ int av_read_frame(AVFormatContext *s,
This fixes a memory leak when side-data is present.
---
libavformat/utils.c | 9 -
1 file changed, 4 insertions(+), 5 deletions(-)
diff --git a/libavformat/utils.c b/libavformat/utils.c
index 7e4f54f..3f82659 100644
--- a/libavformat/utils.c
+++ b/libavformat/utils.c
@@ -1285,12 +1285,11
83 matches
Mail list logo
