[vsipl++] [patch] SSAR opts

Jules Bergmann jules at codesourcery.com
Thu Sep 18 00:55:10 UTC 2008


>> -  s_filt_ = ft_fftm_(fftshift(s_raw, s_filt_));
>> +  { Scope<user> scope("fftshift-1", n_ * mc_ * sizeof(complex<float>));
>> +  fftshift(s_raw, s_filt_);
>> +  }
>> +  ft_fftm_(s_filt_);
> 
> Please add a newline between '{' and the Scope declaration, for 
> readability (also at some other places).

I understand the feedback.

However, the Scope decls get in the way of readability since they're 
unrelated to algorithm's function.  Ideally, we wouldn't have them at 
all.  Adding vertical space will reduce the readability of the algorithm.

I have an evil macro idea, let me see if that works :)

				-- Jules

-- 
Jules Bergmann
CodeSourcery
jules at codesourcery.com
(650) 331-3385 x705



More information about the vsipl++ mailing list