[Octave-bug-tracker] [bug #55525] verLessThan() is unimplemented

[Pantxo Diribarne]

Follow-up Comment #2, bug #55525 (project octave):

We have an Octave specific function "compare_versions". It would probably make
sense to use it for the version string comparison since it handles other
characters like "+".

Other comments:
* Since loops are very slow in Octave we try to vectorize codes as much as
possible. The code could be rewritten to something like

v = ver ();
idx = strcmp ({v.Name}, package);
if (! any (idx))
  error ("verLessThan(): PACKAGE ""%s"" is not installed.", package);
endif

out = compare_versions ({v.Version}{idx}, version, "<");


* Conditions in "if" or "while" statements should be surrounded with
parenthesis
* error/warning strings are prepended with the function name.
* There is a typo, "datenum" should be replaced by "verLessThan" in the doc
string.

I changed the item group, since "Matlab Compatibility" is reserved for already
implemented features that behave differently in Octave and Matlab.

    _______________________________________________________

Reply to this item at:

  <https://savannah.gnu.org/bugs/?55525>

_______________________________________________
  Message posté via Savannah
  https://savannah.gnu.org/