[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
[Octave-patch-tracker] [patch #9148] Added some BIST test for isinteger(
From: |
Mike Miller |
Subject: |
[Octave-patch-tracker] [patch #9148] Added some BIST test for isinteger() in libinterp/corefcn/data.cc |
Date: |
Mon, 31 Oct 2016 18:00:38 +0000 (UTC) |
User-agent: |
Mozilla/5.0 (X11; Linux x86_64; rv:49.0) Gecko/20100101 Firefox/49.0 |
Follow-up Comment #3, patch #9148 (project octave):
Sorry for the delay. Can you please squash your commits into one patch instead
of making a new commit that changes the previous? I would like to see one
patch at the end of this review process that can apply cleanly to the stable
branch of Octave.
The commit message is not quite right, please take a look at the suggestions,
and search the commit log for similar changes that add BIST tests, for
example
http://hg.savannah.gnu.org/hgweb/octave/search/?rev=add%20BIST&revcount=80
http://hg.savannah.gnu.org/hgweb/octave/rev/dc9039163760
_______________________________________________________
Reply to this item at:
<http://savannah.gnu.org/patch/?9148>
_______________________________________________
Message sent via/by Savannah
http://savannah.gnu.org/