[Changeset] isa function with float and numeric arguments
David Bateman
David.Bateman at motorola.com
Tue May 13 05:26:53 CDT 2008
John W. Eaton wrote:
> On 12-May-2008, Bill Denney wrote:
>
> | David Bateman wrote:
> | > David Bateman wrote:
> | >
> | >> The isa function should allow the "float" and "numeric" arguments. The
> | >> attached patch adds this.
> | >>
> | >> D.
> | > Please use this version instead.
> | >
> | > D.
> | >
> | Wouldn't this be easier to read (and marginally faster) if written as:
> |
> | if (strcmp (cname, "float"))
> | retval = any (strcmp (class (x), {"double" "single"}));
> | elseif (strcmp (cname, "fnumeric"))
> | classes = {"double" "single" "uint8" "uint16" "uint32" "uint64" "int8" "int16" "int32" "int64"};
> | retval = any (strcmp (class (x), classes));
> | else
> | retval = strcmp (class (x), cname);
> | endif
>
> I made this change, and used persistent variables for the cell arrays
> to avoid the need for reinitializing them on each call.
>
The version in the repository seems to be mine.. You don't seem to have
committed the changes that make the classes variable persisent..
D.
--
David Bateman David.Bateman at motorola.com
Motorola Labs - Paris +33 1 69 35 48 04 (Ph)
Parc Les Algorithmes, Commune de St Aubin +33 6 72 01 06 33 (Mob)
91193 Gif-Sur-Yvette FRANCE +33 1 69 35 77 01 (Fax)
The information contained in this communication has been classified as:
[x] General Business Information
[ ] Motorola Internal Use Only
[ ] Motorola Confidential Proprietary
More information about the Octave-maintainers
mailing list