<div dir="ltr"><div>Hi Tristan,</div><div><br></div><div>My first intention was to commit this patch on vlc-3.0 where the sampling rate is still at 352800.I will increase it to 384k. But, should I first create the patch for vlc.git, then doing a backport to vlc-3.0.git? Or just send it for vlc-3.0? In the latter case, how should I specify that a patch is intended for vlc-3.0?</div><div><br></div><div>Thank you in advance for your answers</div><div><br></div><div>Best regards<br></div><div>Farid<br></div></div><br><div class="gmail_quote"><div dir="ltr" class="gmail_attr">On Thu, Jan 2, 2020 at 4:54 AM Tristan Matthews <<a href="mailto:tmatth@videolan.org">tmatth@videolan.org</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">Hi<br>
<br>
On Wed, Jan 1, 2020 at 4:09 PM Farid Hammane <<a href="mailto:farid.hammane@gmail.com" target="_blank">farid.hammane@gmail.com</a>> wrote:<br>
><br>
> fixes a consistency issue between the module<br>
> descriptor and the test that ensures that the<br>
> sample rate is supported.<br>
> ---<br>
>  modules/audio_output/amem.c | 6 ++++--<br>
>  1 file changed, 4 insertions(+), 2 deletions(-)<br>
><br>
> diff --git a/modules/audio_output/amem.c b/modules/audio_output/amem.c<br>
> index b2d3d37f7e..35988a600f 100644<br>
> --- a/modules/audio_output/amem.c<br>
> +++ b/modules/audio_output/amem.c<br>
> @@ -30,6 +30,8 @@<br>
>  static int Open (vlc_object_t *);<br>
>  static void Close (vlc_object_t *);<br>
><br>
> +#define AMEM_SAMPLE_RATE_MAX 352800<br>
> +<br>
>  vlc_module_begin ()<br>
>      set_shortname (N_("Audio memory"))<br>
>      set_description (N_("Audio memory output"))<br>
> @@ -43,7 +45,7 @@ vlc_module_begin ()<br>
>          change_private()<br>
>      add_integer ("amem-rate", 44100,<br>
>                   N_("Sample rate"), N_("Sample rate"), false)<br>
> -        change_integer_range (1, 352800)<br>
> +        change_integer_range (1, AMEM_SAMPLE_RATE_MAX)<br>
>          change_private()<br>
>      add_integer ("amem-channels", 2,<br>
>                   N_("Channels count"), N_("Channels count"), false)<br>
> @@ -186,7 +188,7 @@ static int Start (audio_output_t *aout, audio_sample_format_t *fmt)<br>
>          sys->set_volume(sys->opaque, sys->volume, sys->mute);<br>
><br>
>      /* Ensure that format is supported */<br>
> -    if (fmt->i_rate == 0 || fmt->i_rate > 192000<br>
> +    if (fmt->i_rate == 0 || fmt->i_rate > AMEM_SAMPLE_RATE_MAX<br>
>       || channels == 0 || channels > AOUT_CHAN_MAX<br>
>       || strcmp(format, "S16N") /* TODO: amem-format */)<br>
>      {<br>
> --<br>
> 2.20.1<br>
><br>
> _______________________________________________<br>
> vlc-devel mailing list<br>
> To unsubscribe or modify your subscription options:<br>
> <a href="https://mailman.videolan.org/listinfo/vlc-devel" rel="noreferrer" target="_blank">https://mailman.videolan.org/listinfo/vlc-devel</a><br>
<br>
This doesn't apply on master, see:<br>
<a href="http://git.videolan.org/?p=vlc.git;a=blob;f=modules/audio_output/amem.c;h=9d21412f072b0fb8d1e0c9c1ca7c30b1489d4fc9;hb=bab14fed971d56417cd5faa6a00e36933bbf23fc" rel="noreferrer" target="_blank">http://git.videolan.org/?p=vlc.git;a=blob;f=modules/audio_output/amem.c;h=9d21412f072b0fb8d1e0c9c1ca7c30b1489d4fc9;hb=bab14fed971d56417cd5faa6a00e36933bbf23fc</a><br>
<br>
Note that the upper bound was recently increased to 384000<br>
<br>
Best,<br>
-t<br>
_______________________________________________<br>
vlc-devel mailing list<br>
To unsubscribe or modify your subscription options:<br>
<a href="https://mailman.videolan.org/listinfo/vlc-devel" rel="noreferrer" target="_blank">https://mailman.videolan.org/listinfo/vlc-devel</a></blockquote></div>