bugGNU Octave - Bugs: bug #51276, private function in classdef file

 
 

bug #51276: private function in classdef file

Submitted by:  Guillaume <gyom>
Submitted on:  Tue 20 Jun 2017 12:25:03 PM UTC  
 
Category: InterpreterSeverity: 3 - Normal
Priority: 5 - NormalItem Group: Unexpected Error
Status: DuplicateAssigned to: None
Originator Name: GuillaumeOpen/Closed: Closed
Release: devOperating System: Any

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 Jump to the original submission)

Sat 24 Jun 2017 01:19:14 PM UTC, comment #8:

Guillaume: I extended your test case. See my comment here: https://savannah.gnu.org/bugs/?45444#comment5

Amro <amro_octave>
Wed 21 Jun 2017 04:09:24 PM UTC, comment #7:

This is a good test case. I was going to look into adding a similar case to yours for +namespace directories, but that seems to be broken in other ways at the moment.

Mike Miller <mtmiller>
Project Administrator
Wed 21 Jun 2017 03:44:12 PM UTC, comment #6:

Thanks, it works now. I attach a test.

(file #41013)

Guillaume <gyom>
Wed 21 Jun 2017 03:34:30 PM UTC, comment #5:

OK, I backed out my previous changeset and replaced it with this one:

http://hg.savannah.gnu.org/hgweb/octave/rev/2d4a7ae1f6cd

John W. Eaton <jwe>
Project Administrator
Wed 21 Jun 2017 08:33:29 AM UTC, comment #4:

Thanks John!

I started writing a test for this and noticed that a local function cannot call a private function, ie having a local function in myclass.m calling myprivatefcn() fails.
Checking with Matlab, this works, while it is not possible to call a local function from a private function.

Guillaume <gyom>
Tue 20 Jun 2017 09:49:23 PM UTC, comment #3:

I checked in the following changeset:

http://hg.savannah.gnu.org/hgweb/octave/rev/97c3223c17a6

On the other bug report, I also noted that we could use at test for this feature. Would someone like to add one?

John W. Eaton <jwe>
Project Administrator
Tue 20 Jun 2017 04:25:34 PM UTC, comment #2:

Sorry about this - a search with keywords "classdef" and "private" did not return the report you mention.

Guillaume <gyom>
Tue 20 Jun 2017 03:58:18 PM UTC, comment #1:

This has been reported previously, see bug #45444. Closing as a duplicate.

Mike Miller <mtmiller>
Project Administrator
Tue 20 Jun 2017 12:25:03 PM UTC, original submission:

I was looking at the compatibility of gramm with Octave:
https://github.com/piermorel/gramm/
and noticed a problem with functions defined in a "private" folder:

The error message is:

As for gramm, it will require matlab.mixin.Copyable...

Guillaume <gyom>

 

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

Attach File(s):
   
   
Comment:
   

Attached Files
file #41013:  bug-51276.tar.gz added by gyom (2KiB - application/gzip)

 

Digest:
   bug dependencies.

Items that depend on this one: None found

 

Carbon-Copy List
  • -unavailable- added by amro_octave (Posted a comment)
  • -unavailable- added by jwe (Posted a comment)
  • -unavailable- added by mtmiller (Posted a comment)
  • -unavailable- added by gyom (Submitted the item)
  •  

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

    Only project members can vote.

     

    Please enter the title of George Orwell's famous dystopian book (it's a date):

     

     

    Follow 4 latest changes.

    Date Changed By Updated Field Previous Value => Replaced By
    Wed 21 Jun 2017 03:44:12 PM UTCgyomAttached File-=>Added bug-51276.tar.gz, #41013
    Tue 20 Jun 2017 03:58:18 PM UTCmtmillerStatusNone=>Duplicate
      Open/ClosedOpen=>Closed
      Dependencies-=>Depends on bugs #45444

    Back to the top


    Powered by Savane 3.1-cleanup1