|
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/
[Prev in Thread] | Current Thread | [Next in Thread] |