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

Muhammad Faiz mfcc64 at gmail.com
Fri Jun 17 00:48:55 CEST 2016


On Thu, Jun 16, 2016 at 10:03 PM, Michael Niedermayer
<michael at niedermayer.cc> wrote:
> 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
>

OK, patches attached

Thank's
-------------- next part --------------
A non-text attachment was scrubbed...
Name: 0001-swresample-resample-add-support-for-odd-phase_count.patch
Type: text/x-patch
Size: 4582 bytes
Desc: not available
URL: <http://ffmpeg.org/pipermail/ffmpeg-devel/attachments/20160617/16503963/attachment.bin>
-------------- next part --------------
A non-text attachment was scrubbed...
Name: 0002-swresample-resample-do-not-increase-phase_count-on-e.patch
Type: text/x-patch
Size: 4988 bytes
Desc: not available
URL: <http://ffmpeg.org/pipermail/ffmpeg-devel/attachments/20160617/16503963/attachment-0001.bin>


More information about the ffmpeg-devel mailing list