bugGNU Octave - Bugs: bug #44562, strmatch should not be marked as...

 
 

bug #44562: strmatch should not be marked as deprecated

Submitter:  Sébastien Villemot <svillemot>
Submitted:  Tue 17 Mar 2015 05:17:50 PM UTC
   
 
Category:  Octave Function Severity:  3 - Normal
Priority:  5 - Normal Item Group:  Matlab Compatibility
Status:  Fixed Assigned to:  None
Originator Name:  Open/Closed:  * Closed
Release:  * 4.0.0 Operating System:  * Any
Fixed Release:  None Planned Release:  None
* Mandatory Fields

Add a New Comment Rich Markup
   

Jump to the original submission

Thu 09 Aug 2018 05:35:57 PM UTC, comment #7: 

strmatch has been made a "legacy function", i.e., it should not be used in new code, but there is no fixed schedule for its removal (unlike deprecated functions which do have a set version after which they will be removed).

Rik <rik5>
Group administrator
Tue 02 Jun 2015 04:33:10 PM UTC, comment #6: 

Retagging release from 4.0.0-rc1 to 4.0.0.

John W. Eaton <jwe>
Group administrator
Wed 18 Mar 2015 05:37:05 PM UTC, comment #5: 

Thanks very much!

Philip Nienhuis <philipnienhuis>
Group Member
Wed 18 Mar 2015 03:33:25 PM UTC, comment #4: 
John W. Eaton <jwe>
Group administrator
Wed 18 Mar 2015 02:51:46 PM UTC, comment #3: 

Thanks John.

Before closing this bug, I think you'll also want to remove strmatch from the list of deprecated functions in the NEWS file.

Sébastien Villemot <svillemot>
Wed 18 Mar 2015 02:22:26 PM UTC, comment #2: 

I backed out those two changesets:

http://hg.savannah.gnu.org/hgweb/octave/rev/55c2d33e6773
http://hg.savannah.gnu.org/hgweb/octave/rev/e78c0514523d

Sorry, I forgot to note the bug report in the commit message.

John W. Eaton <jwe>
Group administrator
Tue 17 Mar 2015 06:03:31 PM UTC, comment #1: 

+1
At my work the Matlab-addicts have even made strmatchi (case-insensitive strmatch); both forms are used intensively in the code base. So I wasn't glad to see strmatch being deprecated - that'll surely negatively affect Matlab compatibility within a few years.

Although I've just converted the io package functions, I see little reason to deprecate strmatch other than to cut down on some maintenance effort.

Philip Nienhuis <philipnienhuis>
Group Member
Tue 17 Mar 2015 05:17:50 PM UTC, original submission:  

In changesets edf5d63c82e1 and f9959972949a, the strmatch function was marked as deprecated.

At some point, the function was indeed marked to be soon deprecated in ML, but it seems that this decision has been reverted and that the function will be kept in the future. Its usage is only discouraged now. See http://www.mathworks.com/help/matlab/ref/strmatch.html

As a consequence, the function will not be removed from Octave in order to keep ML compatibility.

So I think that the aforementioned commits (or at least the first one) should be reverted.

Sébastien Villemot <svillemot>

 

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

Attach Files:
   
   
Comment:
   

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 jwe (Posted a comment)
  • -email is unavailable- added by svillemot (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 group members can vote.

     

    Follow 4 latest changes.

    Date Changed by Updated Field Previous Value => Replaced by
    2015-06-02 jwe Release4.0.0-rc1 4.0.0
    2015-03-18 jwe StatusNone Fixed
        Open/ClosedOpen Closed
    2015-03-17 philipnienhuis Operating SystemGNU/Linux Any

    Back to the top

    Powered by Savane 3.14-1244.
    Corresponding source code