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?
Or - if not - is swap() correctly placed in the control package?
I guess it would fit better in a miscellaneous or general package.

benjamin

------------------------------------------------------------------------------
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