bugGNU Octave - Bugs: bug #56087, issorted docstring needs update...


bug #56087: issorted docstring needs update (option of "ascend" or "descend")

Submitter:  None
Submitted:  Sat 06 Apr 2019 12:30:10 PM UTC
Category:  Documentation Severity:  3 - Normal
Priority:  5 - Normal Item Group:  Documentation
Status:  Fixed Assigned to:  None
Originator Name:  Originator Email:  -email is unavailable-
Open/Closed:  * Closed Release:  * 5.1.0
Operating System:  * Any
* Mandatory Fields

Add a New Comment (Rich Markup)

Comment Type & Canned Response:


Mon 08 Apr 2019 03:23:18 PM UTC, comment #1: 

I fixed the problem here (https://hg.savannah.gnu.org/hgweb/octave/rev/df32e0cf0992) and also re-wrapped the documentation to fit in 80 columns.  Thanks for the patch.

Rik <rik5>
Project Administrator
Sat 06 Apr 2019 12:30:10 PM UTC, original submission:  

The issorted function now uses a direction option of "ascend" or "descend"
instead of "ascending" and "descending".
( bug #54147
changeset http://hg.savannah.gnu.org/hgweb/octave/rev/b7db401e1a99 )

But the docstring is still old.  Here is a patch:

--- a/libinterp/corefcn/data.cc
+++ b/libinterp/corefcn/data.cc
@@ -6863,10 +6863,9 @@
 @deftypefnx {} {} issorted (@var{a}, @var{mode})
 @deftypefnx {} {} issorted (@var{a}, "rows", @var{mode})
 Return true if the array is sorted according to @var{mode}, which
-may be either @qcode{"ascending"}, @qcode{"descending"}, or
+may be either @qcode{"ascend"}, @qcode{"descend"}, or @qcode{"either"}.

-By default,  @var{mode} is @qcode{"ascending"}.  NaNs are treated in the
+By default,  @var{mode} is @qcode{"ascend"}.  NaNs are treated in the
 same manner as @code{sort}.

 If the optional argument @qcode{"rows"} is supplied, check whether



(Note: upload size limit is set to 16384 kB, after insertion of the required escape characters.)

Attach Files:

No files currently attached


Depends on the following items: None found

Items that depend on this one: None found


Carbon-Copy List
  • -email is unavailable- added by rik5 (Posted a comment)
  • -email is unavailable- added by None (Submitted the item)

    There are 0 votes so far. Votes easily highlight which items people would like to see resolved in priority, independently of the priority of the item set by tracker managers.

    Only project members can vote.





    Follow 2 latest changes.

    Date Changed by Updated Field Previous Value => Replaced by
    2019-04-08 rik5 StatusNone Fixed
        Open/ClosedOpen Closed

    Back to the top

    Powered by Savane 3.9