[FFmpeg-devel] [PATCH 1/2] swresample: fix phase_count calculation

Michael Niedermayer michael at niedermayer.cc
Thu Jun 16 17:03:39 CEST 2016


On Thu, Jun 16, 2016 at 12:31:03AM +0700, Muhammad Faiz wrote:
> support odd phase_count
> stick to low phase_count until set_compensation is called

can you split these in 2 seperate patches ?


[...]
> @@ -382,6 +382,9 @@ static ResampleContext *resample_init(ResampleContext *c, int out_rate, int in_r
>      c->index= -phase_count*((c->filter_length-1)/2);
>      c->frac= 0;
>  
> +    /*av_log(NULL, AV_LOG_ERROR, "phase_count = %d, dst_incr = %d (%d + %d/%d)\n",
> +           c->phase_count, c->dst_incr, c->dst_incr_div, c->dst_incr_mod, c->src_incr);*/
> +
>      swri_resample_dsp_init(c);
>  
>      return c;
[...]
> +    /*av_log(NULL, AV_LOG_ERROR, "phase_count = %d, dst_incr = %d (%d + %d/%d)\n",
> +           c->phase_count, c->dst_incr, c->dst_incr_div, c->dst_incr_mod, c->src_incr);*/

these disabled debug av_log() should probably not be pushed

[...]

-- 
Michael     GnuPG fingerprint: 9FF2128B147EF6730BADF133611EC787040B0FAB

He who knows, does not speak. He who speaks, does not know. -- Lao Tsu
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 181 bytes
Desc: Digital signature
URL: <http://ffmpeg.org/pipermail/ffmpeg-devel/attachments/20160616/0c51a58b/attachment.sig>


More information about the ffmpeg-devel mailing list