[Octave-bug-tracker] [bug #31392] fix for case insensitivity of optimset options

[Olaf Till]

Follow-up Comment #7, bug #31392 (project octave):

There is still a small mistake that prevents it from working: The lookup is
done in _uppercase_, but the stored sort-order is in _lowercase_:

in __all_opts__.m:

    [lnames, idx] = unique (tolower (names));
...
      names = names(idx);


It must be both uppercase or both lowercase, otherwise, e.g.:


octave:1> munlock ("__all_opts__") 
octave:2> clear __all_opts__ # for reproducibility 
octave:3> function ret = test_options (arg) 
> ret = optimset ("a_b", 1, "ab", 2, "c", 3); 
> endfunction 
octave:4> __all_opts__ ("test_options") 
ans =

{
  [1,1] = a_b
  [1,2] = ab
  [1,3] = c
}

octave:5> optimset ("ab", 1);
warning: unrecognized option: ab
octave:6> 


And a typo: lookup is still called with "i" in optimset.m


    _______________________________________________________

Reply to this item at:

  <http://savannah.gnu.org/bugs/?31392>

_______________________________________________
  Message sent via/by Savannah
  http://savannah.gnu.org/