thanks, i see the karaoke filter should work at least there're two channels.<br><br>and if you have some opinion on the sweep filter<br><br><div class="gmail_quote">2011/4/14 Ilkka Ollakka <span dir="ltr"><<a href="mailto:ileoo@videolan.org">ileoo@videolan.org</a>></span><br>
<blockquote class="gmail_quote" style="margin: 0pt 0pt 0pt 0.8ex; border-left: 1px solid rgb(204, 204, 204); padding-left: 1ex;"><div class="im">On Thu, Apr 14, 2011 at 11:00:31PM +0800, ¹ùÓîºê wrote:<br>
> Yes, i check the problem of the filter.<br>
<br>
</div>> And i made a new patch of the filter *karaoke and sweep.*<br>
<br>
Hi,<br>
<br>
> From 6e85c56990a90f84049485f03942bd8c224d2852 Mon Sep 17 00:00:00 2001<br>
> From: yuhong <<a href="mailto:yuhong.guo@gmail.com">yuhong.guo@gmail.com</a>><br>
> Date: Thu, 14 Apr 2011 22:56:08 +0800<br>
> Subject: [PATCH] karaoke and sweep audio filters<br>
<br>
> + * DoWork : Do karaoke filter<br>
> + *****************************************************************************/<br>
> +static block_t *DoWork( filter_t *p_filter, block_t *p_in_buf )<br>
> +{<br>
> +   int i, i_chan;<br>
> +   int i_samples = p_in_buf->i_nb_samples/4;<br>
> +   int i_channels = aout_FormatNbChannels( &p_filter->fmt_in.audio );<br>
> +   float *p_in =  (float*)p_in_buf->p_buffer;<br>
> +<br>
> +   /* filter calculation */<br>
> +   for( i = 0; i < i_samples; i+=i_channels) {<br>
> +          p_in[i] = (p_in[i] - p_in[i+1]) * 0.7;<br>
> +          p_in[i+1] = p_in[i];<br>
> +   }<br>
<br>
What if theres only mono channel? your code seems to assume that theres<br>
allways atleast two channels.<br>
<font color="#888888"><br>
--<br>
Ilkka Ollakka<br>
A penny saved is ridiculous.<br>
</font><br>_______________________________________________<br>
vlc-devel mailing list<br>
To unsubscribe or modify your subscription options:<br>
<a href="http://mailman.videolan.org/listinfo/vlc-devel" target="_blank">http://mailman.videolan.org/listinfo/vlc-devel</a><br>
<br></blockquote></div><br>