Re: 3.2 status report

[John W. Eaton]

On  6-Aug-2008, Jaroslav Hajek wrote:

| The reason for this change is that Matlab's choice makes the structure
| visually useless: if you dump it on the screen, the relevant options
| are hindered by a flood of empty fields which you're completely not
| interested in (and sometimes they don't even make sense). I really
| don't understand why the Matlab people made this choice.
| Thus, there is no simple way to inspect what options a structure
| actually contains.

Other than the clutter, I'm not sure I understand what real problem
this change is trying to solve.  Can you give an example?

| -  elseif (rem (nargs, 2) && isstruct (varargin{1}))
| -    ## Set values in old from name/value pairs.
| -    retval = optimset (varargin{1}, struct (varargin{2:end}));
| -  elseif (rem (nargs, 2) == 0)
| -    ## Create struct.  Default values are replaced by those specified by
| -    ## name/value pairs.
| -    retval = optimset (optimset (), struct (varargin{:}));
|    else
| -    print_usage ();
| +    if (rem (nargs, 2) && isstruct (varargin{1}))
| +      ## Set values in old from name/value pairs.
| +      s1 = varargin{1}; op = varargin(2:end);
| +    elseif (rem (nargs, 2) == 0)
| +      ## Create struct.  Default values are replaced by those specified by
| +      ## name/value pairs.
| +      s1 = struct (); op = varargin(:);
| +    else
| +      print_usage ();
| +    endif
|    endif

Why change this from

  elseif
  elseif
  else
  endif

to

  else
    if
    elseif
    else
    endif
  endif

?

jwe