[vsipl++] fftw3
Jules Bergmann
jules at codesourcery.com
Wed May 9 14:27:18 UTC 2007
Assem Salama wrote:
> Everyone,
> Sorry about last patch, forgot to change something on line 169. Here is
> new one.
Assem,
This looks good. There is one comment you missed for Create_plan:
> > + static rt_complex_type const type = cmplx_inter_fmt;
>
> [4] Please use a name other than 'type' for this member variable.
> Perhaps 'format'?
>
> In general, 'type' should be reserved for member type names create by
> typedefs.
Once you address that, please check it in.
thanks,
-- Jules
--
Jules Bergmann
CodeSourcery
jules at codesourcery.com
(650) 331-3385 x705
More information about the vsipl++
mailing list