octave-bug-tracker
[Top][All Lists]
Advanced

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

[Octave-bug-tracker] [bug #43098] "if (array)" should raise a warning, a


From: Colin Macdonald
Subject: [Octave-bug-tracker] [bug #43098] "if (array)" should raise a warning, and give the same result for dense and sparse
Date: Fri, 24 Jun 2016 18:22:06 +0000 (UTC)
User-agent: Mozilla/5.0 (X11; Fedora; Linux x86_64; rv:47.0) Gecko/20100101 Firefox/47.0

Follow-up Comment #12, bug #43098 (project octave):

>  I still think it is useful to have a "once only" mode for warnings. 

I personally also like this idea.  But I also think people prefer patches that
do one thing right.  You can always prepare a separate patch for you "once
only" idea.

These tests look reasonable.

 * I don't think "%! error (meh)" usually has those brackets.

 * There are some extraneous spaces in "[ 2i 4i ]".

 * I don't really know the internals: I guess you're saying that "array || 0"
tests the same functionality as "if (array)"...  I would've explicit had:

%!warning <boolean value implies all>
%! if ([1 1 0])
%!   assert (false)
%! end

(but maybe its not a big deal).


    _______________________________________________________

Reply to this item at:

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

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




reply via email to

[Prev in Thread] Current Thread [Next in Thread]