[Octave-bug-tracker] [bug #36372] Improved ranks.m included:

[Lachlan Andrew]

Follow-up Comment #22, bug #36372 (project octave):

Happy New Year again, Dave.

Thanks for the update.  I understand reverse ordinal ranking better now, but
when would it be useful?  We want to keep Octave "lean", and only introduce
options that will actually be used.  The others all seem useful, and the
speedup is worthwhile.

I've modified your latest to comply with Octave's coding standards, and
provided a patch relative the the current development version.

In future, could you try to use Octave's coding standards?  They require:
- space before () in a function call
- no space before () when indexing a matrix
- space after ',', and around '=', '+' etc
- Lines no longer than 80 characters

(They're all trivial whitespace issues, but uniformity makes the code easier
to read.)

(file #35943)
    _______________________________________________________

Additional Item Attachment:

File name: bug_36372_patch.txt            Size:5 KB


    _______________________________________________________

Reply to this item at:

  <http://savannah.gnu.org/bugs/?36372>

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