Re: possible issue in octave-forge package image (fftconv2)

[Ben Abbott]

On Jul 3, 2013, at 8:08 PM, Alireza Nejati wrote:

> Hi all,
> 
> In fftconv2.m in the image package, this is what line 47 currently looks like 
> (in the latest version downloaded from sourceforge):
> 
>      if ((nargin > 2) && ismatrix(varargin{3}))
> 
> Since octave returns 1 for, say, ismatrix("same"), it doesn't recognize 
> fftconv2(a,b,shape) properly. It thus returns errors for me. However, if I 
> also add a isnumeric(varargin{3}) check, it works fine. This is what the 
> corrected line looks like:
> 
>     if ((nargin > 2) && ismatrix(varargin{3}) && isnumeric(varargin{3}))
> 
> Thanks for your attention.
> 
> Regards,

Thanks for the detailed report.  May we ask you to file a bug report so that we 
don't forget to fix this?

        https://savannah.gnu.org/bugs/?group=octave

Ben