[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