|
From: | Mike Miller |
Subject: | [Octave-patch-tracker] [patch #9148] Added some BIST test for isinteger() in libinterp/corefcn/data.cc |
Date: | Sat, 17 Dec 2016 20:37:17 -0000 |
User-agent: | Mozilla/5.0 (X11; Linux x86_64; rv:49.0) Gecko/20100101 Firefox/49.0 |
Update of patch #9148 (project octave): Status: None => Done Open/Closed: Open => Closed _______________________________________________________ Follow-up Comment #7: Thank you. The code looks great now. I only made a couple minor fixes to the commit message. I added a period at the end of the line, and I added the "F" prefix to "isinteger" because all of the functions that are declared as DEFUNs have a C++ symbol with an "F" prefix. I pushed your patch on the stable branch, it will be part of Octave 4.2 coming in a matter of days. Thank you for your contribution! http://hg.savannah.gnu.org/hgweb/octave/rev/e6bf4f8920d3 _______________________________________________________ Reply to this item at: <http://savannah.gnu.org/patch/?9148> _______________________________________________ Message sent via/by Savannah http://savannah.gnu.org/
[Prev in Thread] | Current Thread | [Next in Thread] |