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
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,
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
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 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
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 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 +
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
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
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
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.
---
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
---
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
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
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
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 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
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
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
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
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
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
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
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
---
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 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
> -
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 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 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
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
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
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
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 +--
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 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 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
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 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
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;
> +
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
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
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
>
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
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 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
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
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 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
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
>
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
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
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
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
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
---
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
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
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
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
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, 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 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
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 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 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 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 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 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 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
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 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
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
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 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
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
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 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
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 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
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 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
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
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
83 matches
Mail list logo
