[Octave-patch-tracker] [patch #9148] Added some BIST test for isinteger() in libinterp/corefcn/data.cc

[Mike Miller]

Follow-up Comment #5, patch #9148 (project octave):

Yes, it doesn't look like you changed the commit message from the last one
(file #38781). Please compare yours with this commit message for example:
http://hg.savannah.gnu.org/hgweb/octave/rev/dc9039163760

You could use the single-line style as you have done, but you are missing the
asterisk character at the start of the line in that case.

You also still have some tests that are


%!assert (expression, false)


These should instead be written as


%!assert (! expression)


(Like I said, numerous examples of these kinds of tests throughout the Octave
code tree for you to refer to)

    _______________________________________________________

Reply to this item at:

  <http://savannah.gnu.org/patch/?9148>

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