Re: [FFmpeg-devel] [PATCH v2 1/3] avfilter/vf_elbg: Fix for the seed type

2019-12-06 Thread Michael Niedermayer 
On Thu, Dec 05, 2019 at 09:54:44PM +0800, Limin Wang wrote:
> On Wed, Dec 04, 2019 at 08:31:15PM +0100, Michael Niedermayer wrote:
> > On Fri, Nov 22, 2019 at 09:50:36AM +0800, lance.lmw...@gmail.com wrote:
> > > From: Limin Wang 
> > > 
> > > Signed-off-by: Limin Wang 
> > > ---
> > >  libavfilter/vf_elbg.c | 6 +++---
> > >  1 file changed, 3 insertions(+), 3 deletions(-)
> > 
> > will apply
> 
> Thanks, please help to review the PATCH v2 2/3 and 3/3 also, 
> they're fix the same typo.

will apply them too

thanks

[...]
-- 
Michael GnuPG fingerprint: 9FF2128B147EF6730BADF133611EC787040B0FAB

Why not whip the teacher when the pupil misbehaves? -- Diogenes of Sinope


signature.asc
Description: PGP signature
___
ffmpeg-devel mailing list
ffmpeg-devel@ffmpeg.org
https://ffmpeg.org/mailman/listinfo/ffmpeg-devel

To unsubscribe, visit link above, or email
ffmpeg-devel-requ...@ffmpeg.org with subject "unsubscribe".

Re: [FFmpeg-devel] [PATCH v2 1/3] avfilter/vf_elbg: Fix for the seed type

2019-12-05 Thread Limin Wang 
On Wed, Dec 04, 2019 at 08:31:15PM +0100, Michael Niedermayer wrote:
> On Fri, Nov 22, 2019 at 09:50:36AM +0800, lance.lmw...@gmail.com wrote:
> > From: Limin Wang 
> > 
> > Signed-off-by: Limin Wang 
> > ---
> >  libavfilter/vf_elbg.c | 6 +++---
> >  1 file changed, 3 insertions(+), 3 deletions(-)
> 
> will apply

Thanks, please help to review the PATCH v2 2/3 and 3/3 also, 
they're fix the same typo.

> 
> thx
> 
> [...]
> -- 
> Michael GnuPG fingerprint: 9FF2128B147EF6730BADF133611EC787040B0FAB
> 
> I have never wished to cater to the crowd; for what I know they do not
> approve, and what they approve I do not know. -- Epicurus



> ___
> ffmpeg-devel mailing list
> ffmpeg-devel@ffmpeg.org
> https://ffmpeg.org/mailman/listinfo/ffmpeg-devel
> 
> To unsubscribe, visit link above, or email
> ffmpeg-devel-requ...@ffmpeg.org with subject "unsubscribe".

___
ffmpeg-devel mailing list
ffmpeg-devel@ffmpeg.org
https://ffmpeg.org/mailman/listinfo/ffmpeg-devel

To unsubscribe, visit link above, or email
ffmpeg-devel-requ...@ffmpeg.org with subject "unsubscribe".

Re: [FFmpeg-devel] [PATCH v2 1/3] avfilter/vf_elbg: Fix for the seed type

2019-12-04 Thread Michael Niedermayer 
On Fri, Nov 22, 2019 at 09:50:36AM +0800, lance.lmw...@gmail.com wrote:
> From: Limin Wang 
> 
> Signed-off-by: Limin Wang 
> ---
>  libavfilter/vf_elbg.c | 6 +++---
>  1 file changed, 3 insertions(+), 3 deletions(-)

will apply

thx

[...]
-- 
Michael GnuPG fingerprint: 9FF2128B147EF6730BADF133611EC787040B0FAB

I have never wished to cater to the crowd; for what I know they do not
approve, and what they approve I do not know. -- Epicurus


signature.asc
Description: PGP signature
___
ffmpeg-devel mailing list
ffmpeg-devel@ffmpeg.org
https://ffmpeg.org/mailman/listinfo/ffmpeg-devel

To unsubscribe, visit link above, or email
ffmpeg-devel-requ...@ffmpeg.org with subject "unsubscribe".

Re: [FFmpeg-devel] [PATCH v2 1/3] avfilter/vf_elbg: Fix for the seed type

2019-12-03 Thread Limin Wang 

ping the patchset, thx.

On Fri, Nov 22, 2019 at 09:50:36AM +0800, lance.lmw...@gmail.com wrote:
> From: Limin Wang 
> 
> Signed-off-by: Limin Wang 
> ---
>  libavfilter/vf_elbg.c | 6 +++---
>  1 file changed, 3 insertions(+), 3 deletions(-)
> 
> diff --git a/libavfilter/vf_elbg.c b/libavfilter/vf_elbg.c
> index b59da9ec85..5bccb5f58d 100644
> --- a/libavfilter/vf_elbg.c
> +++ b/libavfilter/vf_elbg.c
> @@ -36,7 +36,7 @@
>  typedef struct ELBGContext {
>  const AVClass *class;
>  AVLFG lfg;
> -unsigned int lfg_seed;
> +int64_t lfg_seed;
>  int max_steps_nb;
>  int *codeword;
>  int codeword_length;
> @@ -56,8 +56,8 @@ static const AVOption elbg_options[] = {
>  { "l",   "set codebook length", OFFSET(codebook_length), 
> AV_OPT_TYPE_INT, { .i64 = 256 }, 1, INT_MAX, FLAGS },
>  { "nb_steps", "set max number of steps used to compute the mapping", 
> OFFSET(max_steps_nb), AV_OPT_TYPE_INT, { .i64 = 1 }, 1, INT_MAX, FLAGS },
>  { "n","set max number of steps used to compute the mapping", 
> OFFSET(max_steps_nb), AV_OPT_TYPE_INT, { .i64 = 1 }, 1, INT_MAX, FLAGS },
> -{ "seed", "set the random seed", OFFSET(lfg_seed), AV_OPT_TYPE_INT, 
> {.i64 = -1}, -1, UINT32_MAX, FLAGS },
> -{ "s","set the random seed", OFFSET(lfg_seed), AV_OPT_TYPE_INT, { 
> .i64 = -1 }, -1, UINT32_MAX, FLAGS },
> +{ "seed", "set the random seed", OFFSET(lfg_seed), AV_OPT_TYPE_INT64, 
> {.i64 = -1}, -1, UINT32_MAX, FLAGS },
> +{ "s","set the random seed", OFFSET(lfg_seed), AV_OPT_TYPE_INT64, { 
> .i64 = -1 }, -1, UINT32_MAX, FLAGS },
>  { "pal8", "set the pal8 output", OFFSET(pal8), AV_OPT_TYPE_BOOL, { .i64 
> = 0 }, 0, 1, FLAGS },
>  { NULL }
>  };
> -- 
> 2.21.0
> 
___
ffmpeg-devel mailing list
ffmpeg-devel@ffmpeg.org
https://ffmpeg.org/mailman/listinfo/ffmpeg-devel

To unsubscribe, visit link above, or email
ffmpeg-devel-requ...@ffmpeg.org with subject "unsubscribe".

[FFmpeg-devel] [PATCH v2 1/3] avfilter/vf_elbg: Fix for the seed type

2019-11-21 Thread lance . lmwang 
From: Limin Wang 

Signed-off-by: Limin Wang 
---
 libavfilter/vf_elbg.c | 6 +++---
 1 file changed, 3 insertions(+), 3 deletions(-)

diff --git a/libavfilter/vf_elbg.c b/libavfilter/vf_elbg.c
index b59da9ec85..5bccb5f58d 100644
--- a/libavfilter/vf_elbg.c
+++ b/libavfilter/vf_elbg.c
@@ -36,7 +36,7 @@
 typedef struct ELBGContext {
 const AVClass *class;
 AVLFG lfg;
-unsigned int lfg_seed;
+int64_t lfg_seed;
 int max_steps_nb;
 int *codeword;
 int codeword_length;
@@ -56,8 +56,8 @@ static const AVOption elbg_options[] = {
 { "l",   "set codebook length", OFFSET(codebook_length), 
AV_OPT_TYPE_INT, { .i64 = 256 }, 1, INT_MAX, FLAGS },
 { "nb_steps", "set max number of steps used to compute the mapping", 
OFFSET(max_steps_nb), AV_OPT_TYPE_INT, { .i64 = 1 }, 1, INT_MAX, FLAGS },
 { "n","set max number of steps used to compute the mapping", 
OFFSET(max_steps_nb), AV_OPT_TYPE_INT, { .i64 = 1 }, 1, INT_MAX, FLAGS },
-{ "seed", "set the random seed", OFFSET(lfg_seed), AV_OPT_TYPE_INT, {.i64 
= -1}, -1, UINT32_MAX, FLAGS },
-{ "s","set the random seed", OFFSET(lfg_seed), AV_OPT_TYPE_INT, { .i64 
= -1 }, -1, UINT32_MAX, FLAGS },
+{ "seed", "set the random seed", OFFSET(lfg_seed), AV_OPT_TYPE_INT64, 
{.i64 = -1}, -1, UINT32_MAX, FLAGS },
+{ "s","set the random seed", OFFSET(lfg_seed), AV_OPT_TYPE_INT64, { 
.i64 = -1 }, -1, UINT32_MAX, FLAGS },
 { "pal8", "set the pal8 output", OFFSET(pal8), AV_OPT_TYPE_BOOL, { .i64 = 
0 }, 0, 1, FLAGS },
 { NULL }
 };
-- 
2.21.0

___
ffmpeg-devel mailing list
ffmpeg-devel@ffmpeg.org
https://ffmpeg.org/mailman/listinfo/ffmpeg-devel

To unsubscribe, visit link above, or email
ffmpeg-devel-requ...@ffmpeg.org with subject "unsubscribe".

[FFmpeg-devel] [PATCH v2 1/3] avfilter/vf_elbg: Fix for the seed type

2019-11-21 Thread lance . lmwang 
From: Limin Wang 

Signed-off-by: Limin Wang 
---
 libavfilter/vf_elbg.c | 6 +++---
 1 file changed, 3 insertions(+), 3 deletions(-)

diff --git a/libavfilter/vf_elbg.c b/libavfilter/vf_elbg.c
index b59da9ec85..5bccb5f58d 100644
--- a/libavfilter/vf_elbg.c
+++ b/libavfilter/vf_elbg.c
@@ -36,7 +36,7 @@
 typedef struct ELBGContext {
 const AVClass *class;
 AVLFG lfg;
-unsigned int lfg_seed;
+int64_t lfg_seed;
 int max_steps_nb;
 int *codeword;
 int codeword_length;
@@ -56,8 +56,8 @@ static const AVOption elbg_options[] = {
 { "l",   "set codebook length", OFFSET(codebook_length), 
AV_OPT_TYPE_INT, { .i64 = 256 }, 1, INT_MAX, FLAGS },
 { "nb_steps", "set max number of steps used to compute the mapping", 
OFFSET(max_steps_nb), AV_OPT_TYPE_INT, { .i64 = 1 }, 1, INT_MAX, FLAGS },
 { "n","set max number of steps used to compute the mapping", 
OFFSET(max_steps_nb), AV_OPT_TYPE_INT, { .i64 = 1 }, 1, INT_MAX, FLAGS },
-{ "seed", "set the random seed", OFFSET(lfg_seed), AV_OPT_TYPE_INT, {.i64 
= -1}, -1, UINT32_MAX, FLAGS },
-{ "s","set the random seed", OFFSET(lfg_seed), AV_OPT_TYPE_INT, { .i64 
= -1 }, -1, UINT32_MAX, FLAGS },
+{ "seed", "set the random seed", OFFSET(lfg_seed), AV_OPT_TYPE_INT64, 
{.i64 = -1}, -1, UINT32_MAX, FLAGS },
+{ "s","set the random seed", OFFSET(lfg_seed), AV_OPT_TYPE_INT64, { 
.i64 = -1 }, -1, UINT32_MAX, FLAGS },
 { "pal8", "set the pal8 output", OFFSET(pal8), AV_OPT_TYPE_BOOL, { .i64 = 
0 }, 0, 1, FLAGS },
 { NULL }
 };
-- 
2.21.0

___
ffmpeg-devel mailing list
ffmpeg-devel@ffmpeg.org
https://ffmpeg.org/mailman/listinfo/ffmpeg-devel

To unsubscribe, visit link above, or email
ffmpeg-devel-requ...@ffmpeg.org with subject "unsubscribe".