[Octave-bug-tracker] [bug #43098] "if (array)" should raise a warning, and give the same result for dense and sparse

[Lachlan Andrew]

Update of bug #43098 (project octave):

                Severity:                1 - Wish => 4 - Important          
                  Status:               Confirmed => Patch Submitted        
             Assigned to:                    None => lachlan                
                 Summary: "if (array)" should raise a warning => "if (array)"
should raise a warning, and give the same result for dense and sparse

    _______________________________________________________

Follow-up Comment #3:

Here is a patch that seem to work.

Rik, rather than making this disabled by default, is it sufficient that it
just prints the warning once per invocation, as it currently does?

I plan to write a patch soon that will make each warning default to outputting
the command to disable it, like Matlab does.  (It will use the
currently-unused "verbose" mode of warnings.)  That will allow those who don't
want the warning to turn it off easily.  Otherwise, people who do want the
warning will never learn of its existence, unless they read "warning_ids"
every time they upgrade.

I've added the sparseness issue to the subject and increased the severity to
"important".  Let me know if I should file a separate bug report instead.  The
patch fixes both issues.

(file #37454)
    _______________________________________________________

Additional Item Attachment:

File name: bug_43098_warn_if_array.cset   Size:10 KB


    _______________________________________________________

Reply to this item at:

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

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