You all know the CSI episodes where they read a license plate by
"enhancing" some super-grainy security footage. Nonsense, right? Well,
maybe not.. If the car was parked. And it seems what I found doesn't
exist yet. (but perhaps I overlooked it)
If you quickly want to know what I'm on about, t
On 10/27/2015 9:56 PM, Michael Niedermayer wrote:
> On Mon, Oct 26, 2015 at 12:32:58AM +0700, Muhammad Faiz wrote:
>> On Sun, Oct 25, 2015 at 9:51 AM, Michael Niedermayer
>> wrote:
>>> On Sun, Oct 25, 2015 at 08:43:07AM +0700, Muhammad Faiz wrote:
doc/filters.texi | 181 +++--
On 10/28/2015 11:16 PM, Ronald S. Bultje wrote:
> Hi,
>
> On Wed, Oct 28, 2015 at 5:51 PM, wm4 wrote:
>
>> On Wed, 28 Oct 2015 16:05:49 -0400
>> "Ronald S. Bultje" wrote:
>>
>>> Hi,
>>>
>>> On Wed, Oct 28, 2015 at 3:44 PM, wm4 wrote:
>>>
On Wed, 28 Oct 2015 12:21:05 -0400
"Ronald S.
On Wed, Oct 28, 2015 at 07:02:42PM -0400, Ganesh Ajjanagadde wrote:
> On Wed, Oct 28, 2015 at 3:05 PM, Ronald S. Bultje wrote:
> > Hi,
> >
> > On Wed, Oct 28, 2015 at 2:46 PM, Ganesh Ajjanagadde
> > wrote:
> >>
> >> On Wed, Oct 28, 2015 at 2:39 PM, Ronald S. Bultje
> >> wrote:
> >> > Hi,
> >> >
This simplifies and cleans up the code.
Furthermore, it is much faster due to absence of the slow log computation.
Signed-off-by: Ganesh Ajjanagadde
---
libavutil/rational.c | 4 ++--
1 file changed, 2 insertions(+), 2 deletions(-)
diff --git a/libavutil/rational.c b/libavutil/rational.c
index
This is likely more precise and conveys the intent better.
Signed-off-by: Ganesh Ajjanagadde
---
libavfilter/vf_ssim.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/libavfilter/vf_ssim.c b/libavfilter/vf_ssim.c
index ce1e3db..6b2a8d7 100644
--- a/libavfilter/vf_ssim.c
+++ b
This is likely more precise and conveys the intent better.
Signed-off-by: Ganesh Ajjanagadde
---
libavfilter/vf_psnr.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/libavfilter/vf_psnr.c b/libavfilter/vf_psnr.c
index 36bbab2..bce5c51 100644
--- a/libavfilter/vf_psnr.c
+++ b
This is likely more precise and conveys the intent better.
Signed-off-by: Ganesh Ajjanagadde
---
libavfilter/avf_showvolume.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/libavfilter/avf_showvolume.c b/libavfilter/avf_showvolume.c
index 95b5388..395375a 100644
--- a/libavf
This is likely more precise and conveys the intent better.
Signed-off-by: Ganesh Ajjanagadde
---
libavfilter/avf_showspectrum.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/libavfilter/avf_showspectrum.c b/libavfilter/avf_showspectrum.c
index 936db60..7d5c438 100644
--- a/
This is likely more precise and conveys the intent better.
Signed-off-by: Ganesh Ajjanagadde
---
libavfilter/af_volumedetect.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/libavfilter/af_volumedetect.c b/libavfilter/af_volumedetect.c
index 01f24ba..ad5fef3 100644
--- a/lib
This is likely more precise and conveys the intent better.
Signed-off-by: Ganesh Ajjanagadde
---
libavfilter/af_volume.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/libavfilter/af_volume.c b/libavfilter/af_volume.c
index 16e994c..3ba356a 100644
--- a/libavfilter/af_volume
This is likely more precise and conveys the intent better.
Signed-off-by: Ganesh Ajjanagadde
---
libavcodec/zmbvenc.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/libavcodec/zmbvenc.c b/libavcodec/zmbvenc.c
index df06e37..e832bed 100644
--- a/libavcodec/zmbvenc.c
+++ b/lib
This is likely more precise and conveys the intent better.
The expression has also been accordingly simplified.
Signed-off-by: Ganesh Ajjanagadde
---
libavcodec/snowenc.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/libavcodec/snowenc.c b/libavcodec/snowenc.c
index 7e8269c
This is likely more precise and conveys the intent better.
Signed-off-by: Ganesh Ajjanagadde
---
libavcodec/nellymoserenc.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/libavcodec/nellymoserenc.c b/libavcodec/nellymoserenc.c
index d7368d0..d998dba 100644
--- a/libavcodec/n
This is more concise and conveys the intent better.
Furthermore, it is likely more precise as well due to lack of floating
point division.
Signed-off-by: Ganesh Ajjanagadde
---
ffmpeg.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/ffmpeg.c b/ffmpeg.c
index f91fb7b..148c21b
On Fri, Oct 16, 2015 at 5:31 PM, Ganesh Ajjanagadde wrote:
> On Fri, Oct 16, 2015 at 11:46 AM, Michael Niedermayer
> wrote:
>> On Thu, Oct 15, 2015 at 10:22:17PM -0400, Ganesh Ajjanagadde wrote:
>>> This does not trigger any warnings, but adds robustness.
>>> Untested, as my configure does not co
On Fri, Oct 16, 2015 at 8:02 AM, Ganesh Ajjanagadde
wrote:
> On Thu, Oct 15, 2015 at 11:17 PM, highgod0401 wrote:
>>
>> From: Ganesh Ajjanagadde
>> Date: 2015-10-16 08:08
>> To: ffmpeg-devel
>> CC: Ganesh Ajjanagadde
>> Subject: [FFmpeg-devel] [PATCH] avutil/opencl: add av_warn_unused_result
>> T
On Sun, Oct 4, 2015 at 8:44 PM, Ganesh Ajjanagadde
wrote:
> On Sun, Sep 20, 2015 at 9:09 AM, Ganesh Ajjanagadde
> wrote:
>> s->frame is non-const, so this casts them explicitly.
>> This suppresses -Wdiscarded-qualifiers seen in e.g
>> http://fate.ffmpeg.org/log.cgi?time=20150919100330&log=compile
On Wed, Oct 21, 2015 at 11:46 PM, Ganesh Ajjanagadde
wrote:
> On Thu, Oct 15, 2015 at 8:01 PM, Ganesh Ajjanagadde
> wrote:
>> Current code is fine, this just adds robustness.
>>
>> Signed-off-by: Ganesh Ajjanagadde
>> ---
>> libavutil/wchar_filename.h | 1 +
>> 1 file changed, 1 insertion(+)
>>
On Fri, Oct 16, 2015 at 5:16 PM, Ganesh Ajjanagadde wrote:
> On Fri, Oct 16, 2015 at 11:44 AM, Michael Niedermayer
> wrote:
>> On Thu, Oct 15, 2015 at 07:26:04PM -0400, Ganesh Ajjanagadde wrote:
>>> This one should not trigger any warnings, but will be useful for future
>>> robustness.
>>>
>>> Si
On Thu, Oct 15, 2015 at 10:24 PM, Ganesh Ajjanagadde wrote:
> On Thu, Oct 15, 2015 at 10:22 PM, Ganesh Ajjanagadde wrote:
>> On Thu, Oct 15, 2015 at 10:16 PM, James Almer wrote:
>>> On 10/15/2015 10:23 PM, Ganesh Ajjanagadde wrote:
On Thu, Oct 15, 2015 at 8:06 PM, James Almer wrote:
>
On Fri, Oct 16, 2015 at 5:17 PM, Ganesh Ajjanagadde wrote:
> On Fri, Oct 16, 2015 at 11:39 AM, Michael Niedermayer
> wrote:
>> On Thu, Oct 15, 2015 at 07:20:30PM -0400, Ganesh Ajjanagadde wrote:
>>> This will trigger a bunch of warnings (rightfully so). This API has been
>>> abused, see the Doxyg
On Thu, Oct 15, 2015 at 9:46 PM, Ganesh Ajjanagadde
wrote:
> This triggers a few warnings that need to be fixed.
>
> Signed-off-by: Ganesh Ajjanagadde
> ---
> libswscale/swscale_internal.h | 8
> 1 file changed, 8 insertions(+)
>
> diff --git a/libswscale/swscale_internal.h b/libswscale
On Thu, Oct 15, 2015 at 9:31 PM, Ganesh Ajjanagadde
wrote:
> This will trigger a few warnings (e.g in avfilter) that need to be
> fixed.
>
> Signed-off-by: Ganesh Ajjanagadde
> ---
> libswresample/swresample.h | 9 +
> 1 file changed, 9 insertions(+)
>
> diff --git a/libswresample/swresa
On Thu, Oct 15, 2015 at 7:10 PM, Ganesh Ajjanagadde
wrote:
> Signed-off-by: Ganesh Ajjanagadde
> ---
> libavutil/buffer.h | 3 +++
> 1 file changed, 3 insertions(+)
>
> diff --git a/libavutil/buffer.h b/libavutil/buffer.h
> index b4399fd..e0dbc8a 100644
> --- a/libavutil/buffer.h
> +++ b/libavut
On Thu, Oct 15, 2015 at 6:31 PM, Ganesh Ajjanagadde
wrote:
> Signed-off-by: Ganesh Ajjanagadde
> ---
> libavutil/samplefmt.h | 2 ++
> 1 file changed, 2 insertions(+)
>
> diff --git a/libavutil/samplefmt.h b/libavutil/samplefmt.h
> index 6a8a031..246147f 100644
> --- a/libavutil/samplefmt.h
> ++
On Thu, Oct 15, 2015 at 6:24 PM, Ganesh Ajjanagadde
wrote:
> Signed-off-by: Ganesh Ajjanagadde
> ---
> libavutil/opencl_internal.h | 1 +
> 1 file changed, 1 insertion(+)
>
> diff --git a/libavutil/opencl_internal.h b/libavutil/opencl_internal.h
> index dacd930..a4f5885 100644
> --- a/libavutil/
On Thu, Oct 15, 2015 at 7:47 PM, Ganesh Ajjanagadde
wrote:
> This is going to trigger a lot of warnings all over the place that need
> fixing. Main culprits are in avfilter, when one tries to copy a frame
> without checking whether src and dest formats actually are compatible.
>
> Signed-off-by: G
On Thu, Oct 15, 2015 at 6:12 PM, Ganesh Ajjanagadde
wrote:
> Some warnings do get triggered that will need to be fixed.
>
> Signed-off-by: Ganesh Ajjanagadde
> ---
> libavutil/opt.h | 2 ++
> 1 file changed, 2 insertions(+)
>
> diff --git a/libavutil/opt.h b/libavutil/opt.h
> index 753434d..5abe
On Wed, Oct 28, 2015 at 10:09 AM, Ganesh Ajjanagadde
wrote:
> On Tue, Oct 27, 2015 at 8:18 PM, Ganesh Ajjanagadde
> wrote:
>> av_gcd is now always defined regardless of input. This documents this
>> change in the "documented API". Two benefits (closely related):
>> 1. The function is robust, and
This ensures that no undefined behavior is invoked, while retaining
identical return values in all cases and at no loss of performance
(identical asm on clang and gcc).
Essentially, this patch exchanges undefined behavior with implementation
defined behavior, a strict improvement.
Rationale:
1. Th
This fixes an obvious bug for sub 1-hz modulation frequencies.
Wavetable buffer was too small for frequencies under 1 hz, and also
too big for frequencies above 1hz. One cycle is all we need.
___
ffmpeg-devel mailing list
ffmpeg-devel@ffmpeg.org
http://f
Signed-off-by: Kyle Swanson
---
libavfilter/af_tremolo.c | 6 +++---
1 file changed, 3 insertions(+), 3 deletions(-)
diff --git a/libavfilter/af_tremolo.c b/libavfilter/af_tremolo.c
index 50df2e4..572e9e3 100644
--- a/libavfilter/af_tremolo.c
+++ b/libavfilter/af_tremolo.c
@@ -72,7 +72,7 @@ stat
On Tue, Oct 27, 2015 at 07:58:03PM -0400, Ganesh Ajjanagadde wrote:
> In the spirit of commit 8199908fdf9b3797cceaea9d1e2fc09d02ef7309, likely
> typo originally.
>
> Found by enabling -Wparentheses on clang 3.7 and running a manual audit.
> To the best of my knowledge, no such instances remain.
>
Hi,
On Wed, Oct 28, 2015 at 5:51 PM, wm4 wrote:
> On Wed, 28 Oct 2015 16:05:49 -0400
> "Ronald S. Bultje" wrote:
>
> > Hi,
> >
> > On Wed, Oct 28, 2015 at 3:44 PM, wm4 wrote:
> >
> > > On Wed, 28 Oct 2015 12:21:05 -0400
> > > "Ronald S. Bultje" wrote:
> > >
> > > > ---
> > > > libavcodec/vp9
On Wed, Oct 28, 2015 at 10:00 PM, Michael Niedermayer
wrote:
> On Tue, Oct 27, 2015 at 08:09:03PM -0400, Ganesh Ajjanagadde wrote:
>> Untested.
>>
>> Signed-off-by: Ganesh Ajjanagadde
>> ---
>> libavdevice/dshow_enummediatypes.c | 2 ++
>> 1 file changed, 2 insertions(+)
>>
>> diff --git a/libav
On Tue, Oct 27, 2015 at 08:09:03PM -0400, Ganesh Ajjanagadde wrote:
> Untested.
>
> Signed-off-by: Ganesh Ajjanagadde
> ---
> libavdevice/dshow_enummediatypes.c | 2 ++
> 1 file changed, 2 insertions(+)
>
> diff --git a/libavdevice/dshow_enummediatypes.c
> b/libavdevice/dshow_enummediatypes.c
On Wed, Oct 28, 2015 at 10:00 AM, Michael Niedermayer
wrote:
> On Tue, Oct 27, 2015 at 10:51:29PM -0400, Ganesh Ajjanagadde wrote:
>> This code is in a performance critical section. AV_QSORT is
>> substantially faster due to the inlining of the comparison callback.
>> Thus, the increase in perform
On Wed, Oct 28, 2015 at 11:52 AM, James Almer wrote:
> On 10/27/2015 9:00 PM, Ganesh Ajjanagadde wrote:
>> This ensures that the macro remains correct in the sense of allowing
>> expressions for value and bits, by placing the value and bits expressions
>> within
>> parentheses.
>>
>> Signed-off-b
On Sun, 18 Oct 2015, Bodecs Bela wrote:
[...]
It can be anything if it does not change existing behaviour, a complex
specifier can be split to basic specifiers without worrying about the
syntax of the basic specifier and if there is a well defined rule for
escaping special characters. Also i
On Wed, Oct 28, 2015 at 3:05 PM, Ronald S. Bultje wrote:
> Hi,
>
> On Wed, Oct 28, 2015 at 2:46 PM, Ganesh Ajjanagadde
> wrote:
>>
>> On Wed, Oct 28, 2015 at 2:39 PM, Ronald S. Bultje
>> wrote:
>> > Hi,
>> >
>> > On Wed, Oct 28, 2015 at 2:31 PM, Ganesh Ajjanagadde
>> >
>> > wrote:
>> >>
>> >> O
On Wed, 28 Oct 2015 16:05:49 -0400
"Ronald S. Bultje" wrote:
> Hi,
>
> On Wed, Oct 28, 2015 at 3:44 PM, wm4 wrote:
>
> > On Wed, 28 Oct 2015 12:21:05 -0400
> > "Ronald S. Bultje" wrote:
> >
> > > ---
> > > libavcodec/vp9_parser.c | 2 +-
> > > 1 file changed, 1 insertion(+), 1 deletion(-)
Signed-off-by: Paul B Mahol
---
libavformat/Makefile | 1 +
libavformat/allformats.c | 1 +
libavformat/dcstr.c | 79
3 files changed, 81 insertions(+)
create mode 100644 libavformat/dcstr.c
diff --git a/libavformat/Makefile b/libavfor
Signed-off-by: Paul B Mahol
---
libavcodec/Makefile | 1 +
libavcodec/adpcm.c | 13 +
libavcodec/allcodecs.c | 1 +
libavcodec/avcodec.h| 1 +
libavcodec/codec_desc.c | 7 +++
5 files changed, 23 insertions(+)
diff --git a/libavcodec/Makefile b/libavcodec/Makefi
> ---
> libavcodec/nvenc.c | 7 ++-
> 1 file changed, 6 insertions(+), 1 deletion(-)
>
> diff --git a/libavcodec/nvenc.c b/libavcodec/nvenc.c
> index 0e6ef43..083e494 100644
> --- a/libavcodec/nvenc.c
> +++ b/libavcodec/nvenc.c
> @@ -912,6 +912,11 @@ static av_cold int nvenc_encode_init(AVCod
On Wed, Oct 28, 2015 at 11:52:50AM -0700, Philip Langdale wrote:
> On Wed, 28 Oct 2015 14:53:42 +0800
> Agatha Hu wrote:
>
> > ---
> > libavcodec/nvenc.c | 7 ++-
> > 1 file changed, 6 insertions(+), 1 deletion(-)
> >
> > diff --git a/libavcodec/nvenc.c b/libavcodec/nvenc.c
> > index 0e6e
On Sun, Oct 25, 2015 at 4:56 AM, Ronald S. Bultje
wrote:
>
> So this is likely because we init all tables instead of just these that we
> need, right? So how about having one ff_once per table? That should be
> trivial to implement.
Just so we're all on the same page, this is trivial, but will g
Hi,
On Wed, Oct 28, 2015 at 3:44 PM, wm4 wrote:
> On Wed, 28 Oct 2015 12:21:05 -0400
> "Ronald S. Bultje" wrote:
>
> > ---
> > libavcodec/vp9_parser.c | 2 +-
> > 1 file changed, 1 insertion(+), 1 deletion(-)
> >
> > diff --git a/libavcodec/vp9_parser.c b/libavcodec/vp9_parser.c
> > index 0437
On Wed, Oct 28, 2015 at 07:10:24PM +0100, Nicolas George wrote:
> Le septidi 7 brumaire, an CCXXIV, Kyle Swanson a écrit :
> > Signed-off-by: Kyle Swanson
> > ---
> > libavfilter/asrc_sine.c | 4 ++--
> > 1 file changed, 2 insertions(+), 2 deletions(-)
>
> Of course ok, thanks.
applied
thanks
On Wed, 28 Oct 2015 12:21:05 -0400
"Ronald S. Bultje" wrote:
> ---
> libavcodec/vp9_parser.c | 2 +-
> 1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/libavcodec/vp9_parser.c b/libavcodec/vp9_parser.c
> index 0437097..6713850 100644
> --- a/libavcodec/vp9_parser.c
> +++ b/libavco
Hi,
On Wed, Oct 28, 2015 at 2:46 PM, Ganesh Ajjanagadde
wrote:
> On Wed, Oct 28, 2015 at 2:39 PM, Ronald S. Bultje
> wrote:
> > Hi,
> >
> > On Wed, Oct 28, 2015 at 2:31 PM, Ganesh Ajjanagadde <
> gajjanaga...@gmail.com>
> > wrote:
> >>
> >> On Wed, Oct 28, 2015 at 11:38 AM, Ronald S. Bultje
>
On Wed, 28 Oct 2015 14:53:42 +0800
Agatha Hu wrote:
> ---
> libavcodec/nvenc.c | 7 ++-
> 1 file changed, 6 insertions(+), 1 deletion(-)
>
> diff --git a/libavcodec/nvenc.c b/libavcodec/nvenc.c
> index 0e6ef43..083e494 100644
> --- a/libavcodec/nvenc.c
> +++ b/libavcodec/nvenc.c
> @@ -912
On Wed, Oct 28, 2015 at 2:39 PM, Ronald S. Bultje wrote:
> Hi,
>
> On Wed, Oct 28, 2015 at 2:31 PM, Ganesh Ajjanagadde
> wrote:
>>
>> On Wed, Oct 28, 2015 at 11:38 AM, Ronald S. Bultje
>> wrote:
>> > Hi,
>> >
>> > On Wed, Oct 28, 2015 at 11:00 AM, Ganesh Ajjanagadde
>> > wrote:
>> >>
>> >> On W
Hi,
On Wed, Oct 28, 2015 at 2:31 PM, Ganesh Ajjanagadde
wrote:
> On Wed, Oct 28, 2015 at 11:38 AM, Ronald S. Bultje
> wrote:
> > Hi,
> >
> > On Wed, Oct 28, 2015 at 11:00 AM, Ganesh Ajjanagadde
> > wrote:
> >>
> >> On Wed, Oct 28, 2015 at 10:53 AM, Ronald S. Bultje
> >> wrote:
> >> > Hi,
> >>
On Wed, Oct 28, 2015 at 11:33 AM, Hendrik Leppkes wrote:
> On Wed, Oct 28, 2015 at 4:00 PM, Ganesh Ajjanagadde
> wrote:
>> On Wed, Oct 28, 2015 at 10:53 AM, Ronald S. Bultje
>> wrote:
>>> Hi,
>>>
>>> On Wed, Oct 28, 2015 at 10:48 AM, Ganesh Ajjanagadde
>>> wrote:
On Wed, Oct 28, 2015
On Wed, Oct 28, 2015 at 11:44 AM, Clément Bœsch wrote:
> On Wed, Oct 28, 2015 at 10:34:37AM -0400, Ronald S. Bultje wrote:
> [...]
>> You could do the same for the last 4 values of the cinething changes, they
>> are clearly not enums, but flags. Flags are always unsigned, so if enums
>> are unsign
On Wed, Oct 28, 2015 at 11:38 AM, Ronald S. Bultje wrote:
> Hi,
>
> On Wed, Oct 28, 2015 at 11:00 AM, Ganesh Ajjanagadde
> wrote:
>>
>> On Wed, Oct 28, 2015 at 10:53 AM, Ronald S. Bultje
>> wrote:
>> > Hi,
>> >
>> > On Wed, Oct 28, 2015 at 10:48 AM, Ganesh Ajjanagadde
>> > wrote:
>> >>
>> >> On
On Mon, 26 Oct 2015 20:31:00 +0100
Michael Niedermayer wrote:
> On Sun, Oct 25, 2015 at 06:31:02PM +0100, wm4 wrote:
> > Channel layouts are essentially uint64_t, and every value is valid.
> > ---
> > libswresample/options.c | 8
> > 1 file changed, 4 insertions(+), 4 deletions(-)
>
Le septidi 7 brumaire, an CCXXIV, Kyle Swanson a écrit :
> Signed-off-by: Kyle Swanson
> ---
> libavfilter/asrc_sine.c | 4 ++--
> 1 file changed, 2 insertions(+), 2 deletions(-)
Of course ok, thanks.
Regards,
--
Nicolas George
signature.asc
Description: Digital signature
Signed-off-by: Kyle Swanson
---
libavfilter/asrc_sine.c | 4 ++--
1 file changed, 2 insertions(+), 2 deletions(-)
diff --git a/libavfilter/asrc_sine.c b/libavfilter/asrc_sine.c
index f361faa..2a2f3c3 100644
--- a/libavfilter/asrc_sine.c
+++ b/libavfilter/asrc_sine.c
@@ -69,8 +69,8 @@ typedef str
On Wed, 28 Oct 2015 13:30:59 -0400
"Ronald S. Bultje" wrote:
> Hi,
>
> On Sat, Oct 17, 2015 at 4:34 PM, Matthieu Bouron
> wrote:
>
> > From: Matthieu Bouron
> >
> > Avoid decoding twice images such as jpeg and png, once in the
> > avformat_find_stream_info and once when the actual decode is m
On Sat, Oct 17, 2015 at 10:34 PM, Matthieu Bouron
wrote:
> From: Matthieu Bouron
>
> Avoid decoding twice images such as jpeg and png, once in the
> avformat_find_stream_info and once when the actual decode is made.
>
> The decoder must honor the skip_frame option in order to skip
> decoding. For
On Wed, Oct 28, 2015 at 6:07 PM, Derek Buitenhuis
wrote:
> Enjoy my half-assed / useless review.
>
>> +#ifndef SECBUFFER_ALERT
>> +#define SECBUFFER_ALERT17
>> +#endif
>
> Why?
I think it was needed on some configuration, I'll verify.
>
>> +SecPkgContext_StreamSizes Sizes;
>
Hi,
On Sat, Oct 17, 2015 at 4:34 PM, Matthieu Bouron
wrote:
> From: Matthieu Bouron
>
> Avoid decoding twice images such as jpeg and png, once in the
> avformat_find_stream_info and once when the actual decode is made.
>
> The decoder must honor the skip_frame option in order to skip
> decoding
Enjoy my half-assed / useless review.
> +#ifndef SECBUFFER_ALERT
> +#define SECBUFFER_ALERT17
> +#endif
Why?
> +SecPkgContext_StreamSizes Sizes;
Accidental capital?
> +if (c->enc_buf == NULL) {
> +c->enc_buf_offset = 0;
> +c->enc_buf_size = SCHANNEL_INIT
On Sat, Oct 17, 2015 at 10:34:24PM +0200, Matthieu Bouron wrote:
> From: Matthieu Bouron
>
> Avoid decoding twice images such as jpeg and png, once in the
> avformat_find_stream_info and once when the actual decode is made.
>
> The decoder must honor the skip_frame option in order to skip
> deco
On Wed, Oct 28, 2015 at 03:10:00PM +0100, Matthieu Bouron wrote:
> On Wed, Oct 28, 2015 at 12:58:53PM +0100, Michael Niedermayer wrote:
> > On Wed, Oct 28, 2015 at 02:57:14AM +0100, Michael Niedermayer wrote:
> > > On Tue, Oct 27, 2015 at 11:15:29PM +0100, Matthieu Bouron wrote:
> > > > On Sun, Oct
On Wed, Oct 28, 2015 at 03:09:02PM +0100, Matthieu Bouron wrote:
> On Wed, Oct 28, 2015 at 12:59:54PM +0100, Michael Niedermayer wrote:
> > On Mon, Oct 19, 2015 at 10:28:18AM +0200, Matthieu Bouron wrote:
> > > On Sat, Oct 17, 2015 at 10:34:22PM +0200, Matthieu Bouron wrote:
> > > > From: Matthieu
On Sun, Oct 18, 2015 at 02:06:12PM +0200, Bodecs Bela wrote:
> Dear Marton Balint,
>
> see may comments below.
>
> 2015.10.18. 1:10 keltezéssel, Marton Balint írta:
> >
> >On Thu, 15 Oct 2015, Bodecs Bela wrote:
> >
> >[...]
> >
> My enhancement does not alter the current behaviour in any way
---
libavcodec/vp9_parser.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/libavcodec/vp9_parser.c b/libavcodec/vp9_parser.c
index 0437097..6713850 100644
--- a/libavcodec/vp9_parser.c
+++ b/libavcodec/vp9_parser.c
@@ -64,7 +64,7 @@ static int parse_frame(AVCodecParserContext
On 10/27/2015 9:00 PM, Ganesh Ajjanagadde wrote:
> This ensures that the macro remains correct in the sense of allowing
> expressions for value and bits, by placing the value and bits expressions
> within
> parentheses.
>
> Signed-off-by: Ganesh Ajjanagadde
> ---
> libavutil/ripemd.c | 2 +-
>
On Wed, Oct 21, 2015 at 1:52 PM, Hendrik Leppkes wrote:
> This implementation does not support TLS listen sockets and loading
> CA/Certs from files.
>
> The Windows API does not support loading PEM certs, and would either
> require a manual loader or instead be limited to loading Windows PFX
> cer
On Wed, Oct 28, 2015 at 10:34:37AM -0400, Ronald S. Bultje wrote:
[...]
> You could do the same for the last 4 values of the cinething changes, they
> are clearly not enums, but flags. Flags are always unsigned, so if enums
> are unsigned and that causes an issue (as it does here), it makes sense t
Hi,
On Wed, Oct 28, 2015 at 11:00 AM, Ganesh Ajjanagadde wrote:
> On Wed, Oct 28, 2015 at 10:53 AM, Ronald S. Bultje
> wrote:
> > Hi,
> >
> > On Wed, Oct 28, 2015 at 10:48 AM, Ganesh Ajjanagadde
> > wrote:
> >>
> >> On Wed, Oct 28, 2015 at 10:34 AM, Ronald S. Bultje
> >> wrote:
> >> > Hi,
> >
On Wed, Oct 28, 2015 at 4:00 PM, Ganesh Ajjanagadde
wrote:
> On Wed, Oct 28, 2015 at 10:53 AM, Ronald S. Bultje wrote:
>> Hi,
>>
>> On Wed, Oct 28, 2015 at 10:48 AM, Ganesh Ajjanagadde
>> wrote:
>>>
>>> On Wed, Oct 28, 2015 at 10:34 AM, Ronald S. Bultje
>>> wrote:
>>> > Hi,
>>> >
>>> > On Wed,
On Wed, Oct 28, 2015 at 10:53 AM, Ronald S. Bultje wrote:
> Hi,
>
> On Wed, Oct 28, 2015 at 10:48 AM, Ganesh Ajjanagadde
> wrote:
>>
>> On Wed, Oct 28, 2015 at 10:34 AM, Ronald S. Bultje
>> wrote:
>> > Hi,
>> >
>> > On Wed, Oct 28, 2015 at 8:20 AM, Ganesh Ajjanagadde
>> >
>> > wrote:
>> >>
>> >
Hi,
On Wed, Oct 28, 2015 at 10:48 AM, Ganesh Ajjanagadde wrote:
> On Wed, Oct 28, 2015 at 10:34 AM, Ronald S. Bultje
> wrote:
> > Hi,
> >
> > On Wed, Oct 28, 2015 at 8:20 AM, Ganesh Ajjanagadde <
> gajjanaga...@gmail.com>
> > wrote:
> >>
> >> On Wed, Oct 28, 2015 at 7:00 AM, Ronald S. Bultje
>
On Wed, Oct 28, 2015 at 10:34 AM, Ronald S. Bultje wrote:
> Hi,
>
> On Wed, Oct 28, 2015 at 8:20 AM, Ganesh Ajjanagadde
> wrote:
>>
>> On Wed, Oct 28, 2015 at 7:00 AM, Ronald S. Bultje
>> wrote:
>> > Hi,
>> >
>> > On Tue, Oct 27, 2015 at 10:58 PM, Ganesh Ajjanagadde
>> > wrote:
>> >>
>> >> On T
Hi,
On Wed, Oct 28, 2015 at 8:20 AM, Ganesh Ajjanagadde
wrote:
> On Wed, Oct 28, 2015 at 7:00 AM, Ronald S. Bultje
> wrote:
> > Hi,
> >
> > On Tue, Oct 27, 2015 at 10:58 PM, Ganesh Ajjanagadde
> > wrote:
> >>
> >> On Tue, Oct 27, 2015 at 10:41 PM, Ronald S. Bultje
> >> wrote:
> >> > Hi,
> >>
On Wed, Oct 28, 2015 at 05:39:35AM -0500, Rodger Combs wrote:
> ---
> tools/crypto_bench.c | 82
>
> 1 file changed, 82 insertions(+)
>
> diff --git a/tools/crypto_bench.c b/tools/crypto_bench.c
> index b3b24a6..ad20f95 100644
> --- a/tools/cr
On Wed, Oct 28, 2015 at 12:58:53PM +0100, Michael Niedermayer wrote:
> On Wed, Oct 28, 2015 at 02:57:14AM +0100, Michael Niedermayer wrote:
> > On Tue, Oct 27, 2015 at 11:15:29PM +0100, Matthieu Bouron wrote:
> > > On Sun, Oct 18, 2015 at 11:06:50AM +0200, Matthieu Bouron wrote:
> > > [...]
> > > >
On Tue, Oct 27, 2015 at 8:18 PM, Ganesh Ajjanagadde
wrote:
> av_gcd is now always defined regardless of input. This documents this
> change in the "documented API". Two benefits (closely related):
> 1. The function is robust, and there is no need to worry about INT64_MIN, etc.
>
> 2. Clients of av
On Wed, Oct 28, 2015 at 12:59:54PM +0100, Michael Niedermayer wrote:
> On Mon, Oct 19, 2015 at 10:28:18AM +0200, Matthieu Bouron wrote:
> > On Sat, Oct 17, 2015 at 10:34:22PM +0200, Matthieu Bouron wrote:
> > > From: Matthieu Bouron
> > >
> > > ---
> > > libavcodec/pngdec.c | 34
On Wed, Oct 28, 2015 at 05:39:32AM -0500, Rodger Combs wrote:
> ---
> libavutil/aes.c | 19 ++-
> 1 file changed, 14 insertions(+), 5 deletions(-)
LGTM
thx
[...]
--
Michael GnuPG fingerprint: 9FF2128B147EF6730BADF133611EC787040B0FAB
The educated differ from the uneducated
On Tue, Oct 27, 2015 at 10:51:29PM -0400, Ganesh Ajjanagadde wrote:
> This code is in a performance critical section. AV_QSORT is
> substantially faster due to the inlining of the comparison callback.
> Thus, the increase in performance is worth the increase in binary size.
>
> Signed-off-by: Gane
On 5 October 2015 at 20:05, Matt Oliver wrote:
> On 5 October 2015 at 19:29, Matt Oliver wrote:
>
>> On 30 September 2015 at 02:48, wm4 wrote:
>>
>>> On Wed, 23 Sep 2015 12:04:43 -0600
>>> Roger Pack wrote:
>>>
>>> > From 6a972dda58bd5ab31524cd4e5326b4bcdeaeaa8c Mon Sep 17 00:00:00 2001
>>> >
On Wed, Oct 28, 2015 at 02:57:14AM +0100, Michael Niedermayer wrote:
> On Tue, Oct 27, 2015 at 11:15:29PM +0100, Matthieu Bouron wrote:
> > On Sun, Oct 18, 2015 at 11:06:50AM +0200, Matthieu Bouron wrote:
> > [...]
> > >
> > > Patch updated, the markers are now properly skipped (which also fixes a
ping
2015.10.18. 14:06 keltezéssel, Bodecs Bela írta:
Dear Marton Balint,
see may comments below.
2015.10.18. 1:10 keltezéssel, Marton Balint írta:
On Thu, 15 Oct 2015, Bodecs Bela wrote:
[...]
My enhancement does not alter the current behaviour in any way.
Are you sure? What if somebod
On Thu, Oct 22, 2015 at 11:50:35PM +0300, Kirill Gavrilov wrote:
> Use optional sTER chunk defining side-by-side stereo pair
> within "Extensions to the PNG 1.2 Specification", version 1.3.0.
>
> Specification:
> http://www.libpng.org/pub/png/spec/register/pngext-1.4.0-pdg.html
>
> Note that patc
On Wed, Oct 28, 2015 at 5:14 AM, Michael Niedermayer
wrote:
> On Wed, Oct 28, 2015 at 09:55:56AM +0100, wm4 wrote:
>> On Tue, 27 Oct 2015 20:21:11 -0400
>> Ganesh Ajjanagadde wrote:
>>
>> > This fixes warning C4702 "unreachable code" reported in MSVC:
>> > http://fate.ffmpeg.org/log.cgi?time=2015
On Wed, Oct 28, 2015 at 7:00 AM, Ronald S. Bultje wrote:
> Hi,
>
> On Tue, Oct 27, 2015 at 10:58 PM, Ganesh Ajjanagadde
> wrote:
>>
>> On Tue, Oct 27, 2015 at 10:41 PM, Ronald S. Bultje
>> wrote:
>> > Hi,
>> >
>> > On Tue, Oct 27, 2015 at 8:53 PM, Ganesh Ajjanagadde
>> >
>> > wrote:
>> >>
>> >>
On Wed, Oct 28, 2015 at 3:41 AM, Hendrik Leppkes wrote:
> On Wed, Oct 28, 2015 at 4:17 AM, Ganesh Ajjanagadde
> wrote:
>> On Thu, Oct 15, 2015 at 6:05 PM, Ganesh Ajjanagadde
>> wrote:
>>> Signed-off-by: Ganesh Ajjanagadde
>>> ---
>>> libavutil/avstring.h | 2 ++
>>> 1 file changed, 2 insertion
On Mon, Oct 19, 2015 at 10:28:18AM +0200, Matthieu Bouron wrote:
> On Sat, Oct 17, 2015 at 10:34:22PM +0200, Matthieu Bouron wrote:
> > From: Matthieu Bouron
> >
> > ---
> > libavcodec/pngdec.c | 34 ++
> > 1 file changed, 34 insertions(+)
> >
> > diff --git a/li
On Wed, Oct 28, 2015 at 02:57:14AM +0100, Michael Niedermayer wrote:
> On Tue, Oct 27, 2015 at 11:15:29PM +0100, Matthieu Bouron wrote:
> > On Sun, Oct 18, 2015 at 11:06:50AM +0200, Matthieu Bouron wrote:
> > [...]
> > >
> > > Patch updated, the markers are now properly skipped (which also fixes a
On Sun, Oct 25, 2015 at 03:09:24PM +0100, Carl Eugen Hoyos wrote:
> On Sunday 25 October 2015 01:37:15 pm Carl Eugen Hoyos wrote:
> > Hi!
> >
> > Attached patch intends to fix an issue reported by forum user Koracas:
> > For slice_count > 0x1 FFmpeg fails to decode a frame, QT seems to
> > igno
Hi,
On Tue, Oct 27, 2015 at 10:58 PM, Ganesh Ajjanagadde wrote:
> On Tue, Oct 27, 2015 at 10:41 PM, Ronald S. Bultje
> wrote:
> > Hi,
> >
> > On Tue, Oct 27, 2015 at 8:53 PM, Ganesh Ajjanagadde <
> gajjanaga...@gmail.com>
> > wrote:
> >>
> >> ISO C restricts enumerator values to the range of in
---
tests/Makefile | 5 +
tests/fate/avformat.mak | 1 +
tests/lavf-regression.sh | 9 +
tests/ref/lavf/crypto| 3 +++
4 files changed, 18 insertions(+)
create mode 100644 tests/ref/lavf/crypto
diff --git a/tests/Makefile b/tests/Makefile
index 7ee4a46..c501813 100644
---
tools/crypto_bench.c | 50 ++
1 file changed, 42 insertions(+), 8 deletions(-)
diff --git a/tools/crypto_bench.c b/tools/crypto_bench.c
index 8a468ba..b513c55 100644
--- a/tools/crypto_bench.c
+++ b/tools/crypto_bench.c
@@ -32,6 +32,7 @@
#inclu
---
tools/crypto_bench.c | 85 +---
1 file changed, 47 insertions(+), 38 deletions(-)
diff --git a/tools/crypto_bench.c b/tools/crypto_bench.c
index 15bb5f1..8a468ba 100644
--- a/tools/crypto_bench.c
+++ b/tools/crypto_bench.c
@@ -244,6 +244,7 @@ st
---
tools/crypto_bench.c | 140 +--
1 file changed, 137 insertions(+), 3 deletions(-)
diff --git a/tools/crypto_bench.c b/tools/crypto_bench.c
index ad20f95..15bb5f1 100644
--- a/tools/crypto_bench.c
+++ b/tools/crypto_bench.c
@@ -52,6 +52,7 @@ stat
1 - 100 of 112 matches
Mail list logo
