bugGNU Octave - Bugs: bug #53565, Implement clearvars function for...

 
 

bug #53565: Implement clearvars function for Matlab compatibility

Submitted by:  Ceral Paquet <octavebugs>
Submitted on:  Thu 05 Apr 2018 04:24:16 PM UTC  
 
Category:  Octave Function Severity:  3 - Normal
Priority:  5 - Normal Item Group:  Feature Request
Status:  Fixed Assigned to:  None
Originator Name:  Open/Closed:  Closed
Release:  4.2.1 Operating System:  GNU/Linux

Add a New Comment(Rich Markup)
   

You are not logged in

Please log in, so followups can be emailed to you.

 

( Jump to the original submission)

Thu 06 Dec 2018 04:13:44 PM UTC, comment #7:

I'm not sure what the record is for the oldest bug that has now been cleared. The original clearvars request goes back to 2013. I just happened on it because I was looking for bugs with submitted patches ahead of the 5.0 release. I was hoping that I might find some easy additions to Octave. This one turned out to be quite a bit of work, however.

Rik <rik5>
Project Administrator
Thu 06 Dec 2018 05:02:26 AM UTC, comment #6:

OK great I guess. Any reason to reimplement this?

if !Rik
pause(intmax);
return(complaints);
else
accept();
endif

;)

Trying to be funny... but srlsy?

Ceral Paquet <octavebugs>
Wed 05 Dec 2018 06:32:57 PM UTC, comment #5:

I implemented the clearvars function in an m-file in this changeset (https://hg.savannah.gnu.org/hgweb/octave/rev/96bc9ee8e77f).

Marking as fixed and closing report.

Rik <rik5>
Project Administrator
Wed 05 Dec 2018 05:20:55 AM UTC, comment #4:

I improved the input validation of clear to reject multiple options. I also changed the documentation to state that all options must appear before any name patterns. The code

is incorrect. Once option processing has stopped all remaining items on the line are treated as patterns. The code above would try to remove a variable named "-x".

See cset https://hg.savannah.gnu.org/hgweb/octave/rev/096b38cac97f.

Rik <rik5>
Project Administrator
Wed 05 Dec 2018 05:05:35 AM UTC, comment #3:

I closed bug #40095 as a duplicate of this one.

Rik <rik5>
Project Administrator
Fri 06 Apr 2018 06:28:05 PM UTC, comment #2:

So should we close bug #40095 as a duplicate of this now?

Mike Miller <mtmiller>
Project Administrator
Fri 06 Apr 2018 07:14:24 AM UTC, comment #1:

Oops, didn't post the "clear -x" issue.

If I understand correctly, the exclusive option (-x) should protect a1 so it isn't behaving correctly (or giving an error) when other arguments are supplied.

Ceral Paquet <octavebugs>
Thu 05 Apr 2018 04:24:16 PM UTC, original submission:

This is a reboot of bug #40095 which is not active. Please find attached a working version of clearvars and test cases.

Ceral Paquet <octavebugs>

 

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

Attach Files:
   
   
Comment:
   

Attached Files
file #43810:  clearvars.m added by octavebugs (3KiB - text/x-objcsrc)
file #43811:  test_clearvars.m added by octavebugs (1KiB - text/x-objcsrc)

 

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 mtmiller (Posted a comment)
  • -email is unavailable- added by octavebugs (Submitted the item)
  •  

    Do you think this task is very important?
    If so, you can add your encouragement to it.
    This task has 0 encouragements so far.

    Only project members can vote.

     

     

     

    Follow 8 latest changes.

    Date Changed by Updated Field Previous Value => Replaced by
    2018-12-05 rik5 StatusPatch Submitted => Fixed
        Open/ClosedOpen => Closed
    2018-12-05 rik5 Summaryclear -x and clearvars revisit => Implement clearvars function for Matlab compatibility
    2018-04-06 mtmiller CategoryNone => Octave Function
        Item GroupNone => Feature Request
        StatusNone => Patch Submitted
    2018-04-05 octavebugs Attached File- => Added clearvars.m, #43810
        Attached File- => Added test_clearvars.m, #43811

    Back to the top


    Powered by Savane 3.3