[pulseaudio-discuss] [PATCH 1/3] resampler: Split the resampler implementations into separate files

Alexander E. Patrakov patrakov at gmail.com
Mon Aug 4 09:28:48 PDT 2014


04.08.2014 18:40, Peter Meerwald wrote:
>   static bool speex_is_fixed_point(void) {

I am of the mixed opinion on the fact that this function is not moved to 
speex.c. The patch subjectively feels incomplete because of that - but 
this function is special due to the fact that it also has to exist (as a 
stub) even without speex.  Maybe it indeed should not be moved, maybe it 
should be moved and a stub has to be created then in resampler.h.

Other than that, the patch looks good.

-- 
Alexander E. Patrakov


More information about the pulseaudio-discuss mailing list