On Mon, Feb 22, 2010 at 6:49 PM, Søren Hauberg <so...@hauberg.org> wrote: > 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. >
I should also note that it can be done through [A, B] = deal (B, A); which generalizes also to more than 2 variables. -- RNDr. Jaroslav Hajek, PhD computing expert & GNU Octave developer Aeronautical Research and Test Institute (VZLU) Prague, Czech Republic url: www.highegg.matfyz.cz ------------------------------------------------------------------------------ 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
