[Octave-patch-tracker] [patch #10440] Allow unique to process cellstr arrays with the 'rows' option

[Nicholas Jankowski]

Update of patch #10440 (group octave):

                  Status:                    None => In Progress            

    _______________________________________________________

Follow-up Comment #1:

here's a modified patch which will apply cleanly to the current default
branch.  

this seems like a worthwhile extension to unique.  I wasn't able to break it
or cause any other problems after the change. 

There will be a potential compatibility issue, as currently for a cellstring
array A, unique(A,'rows') returns the same as unique(A). Matlab is also pretty
explicit in it's help for unique that 'rows' does not work for cell arrays.
Does anyone know if there are particular problems supporting cellstrings would
cause? 

running the test suite now to see if it triggers any other issues, but don't
expect any.

(file #55857)

    _______________________________________________________

Additional Item Attachment:

File name: unique_cellstr_rows_patch10440.patch Size:2 KB
   
<https://file.savannah.gnu.org/file/unique_cellstr_rows_patch10440.patch?file_id=55857>


    AGPL NOTICE

These attachments are served by Savane. You can download the corresponding
source code of Savane at
https://git.savannah.nongnu.org/cgit/administration/savane.git/snapshot/savane-cf05868ce827c9612cb6083f86556077b7ab06b3.tar.gz


    _______________________________________________________

Reply to this item at:

  <https://savannah.gnu.org/patch/?10440>

_______________________________________________
Message sent via Savannah
https://savannah.gnu.org/