man, 22 02 2010 kl. 18:33 +0100, skrev Benjamin Lindner:
> Søren Hauberg wrote:
> > man, 22 02 2010 kl. 03:52 -0500, skrev Carnë Draug:
> >> Hi
> >>
> >> the function bwmorph in the image package requires the function swap
> >> from the control package. This is not listed as a dependencie for the
> >> package. Here's line 550 of file bwmorph.m where the swap function is
> >> used
> >>
> >>     [BW,BW2]=swap(BW,BW2);
> >>
> >> Just found this and thought I should point it out.
> > 
> > Thanks. This has been fixed in SVN for a while. I guess I should make a
> > release soon...
> 
> swap(x,y) is a trivial two-liner. Wouldn't it be better to replace the 
> call to swap in bwmorph by simply two assignments rather than 
> introducing unnecessary dependencies?

Yeah, that's what I've done :-)

Søren

P.S. 'swap' used to be a core Octave function (before the control tools
where moved to a package), which is why it was being used.


------------------------------------------------------------------------------
Download Intel® Parallel Studio Eval
Try the new software tools for yourself. Speed compiling, find bugs
proactively, and fine-tune applications for parallel performance.
See why Intel Parallel Studio got high marks during beta.
http://p.sf.net/sfu/intel-sw-dev
_______________________________________________
Octave-dev mailing list
Octave-dev@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/octave-dev

Reply via email to