[vsipl++] [patch] SAL convolution

[Jules Bergmann]

Don, looks good!  Please check it in.  Thanks -- Jules

Don McCoy wrote:
> The attached patch integrates SAL's version of convolution.  Only the 
> 1-D case is implemented at this time.  The decimation parameter must 
> also be one.
> 
> A new set of tests was added for testing support of data strides other 
> than one.  However, because the filter is held in a default Vector<T> as 
> part of the convolution object, it is always unit-stride.  The input and 
> output strides do not have to be the same, but they are at this time.
> 
> The SAL functions used for this (*convx) perform time-domain 
> convolution.  According to Mercury's documentation, this is efficient 
> for cases where the filter (or kernel) length is up to 36 (17 for 
> complex values).  This patch reverts to the generic method for 
> convolution for longer lengths.  For the future we may want to use SAL's 
> fast (FFT-based) convolution functions instead.
> 
> Regards,



> Index: src/vsip/impl/sal.hpp
> ===================================================================
> RCS file: /home/cvs/Repository/vpp/src/vsip/impl/sal.hpp,v
> retrieving revision 1.2
> diff -c -p -r1.2 sal.hpp
> *** src/vsip/impl/sal.hpp	15 Nov 2005 21:01:27 -0000	1.2
> --- src/vsip/impl/sal.hpp	28 Nov 2005 07:59:21 -0000
> ***************
> *** 14,19 ****
> --- 14,20 ----
>     Included Files
>   ***********************************************************************/
>   
> + #include <iostream>

Is it necessary to include iostream here?