bugGNU Octave - Bugs: bug #36476, classdef: parsing of superclass...

 
 

bug #36476: classdef: parsing of superclass calls fails

Submitted by:  None
Submitted on:  Tue 15 May 2012 11:31:42 PM UTC  
 
Category: InterpreterSeverity: 3 - Normal
Priority: 5 - NormalItem Group: Incorrect Result
Status: FixedAssigned to: None
Originator Name: James LairdOriginator Email: -unavailable-
Open/Closed: ClosedRelease: dev
Operating 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)

Tue 12 Apr 2016 05:58:24 PM UTC, comment #7:

It seems to be because I use this.method@super
instead of method@super(this)

Don't know if this intentional, the former syntax works in Matlab. Made a bug report here:
https://savannah.gnu.org/bugs/index.php?47680

Ivo <icouckuy>
Tue 12 Apr 2016 05:29:16 PM UTC, comment #6:

The original test case works for me with 4.0.1 and the current development sources. If there is still a problem, open a new bug report with a complete test case that shows the problem.

John W. Eaton <jwe>
Project Administrator
Tue 12 Apr 2016 04:32:04 PM UTC, comment #5:

I recently started using octave 4.0.1 and porting some of our matlab code to octave. I might have stumbled on the same error:

>> myInputParser

parse error near line 519 of file /home/ilm/projecten/src/matlab/tools/myInputParser.m

syntax error

>>> this.parse@inputParser( requiredArgs{:}, optionalArgs{:} );


Is it possible this bug is back ? We do use fairly advanced Matlab functionality (such as deriving from the inputParser) so the problem could be something else. If noone else has the problem I should simplify the code and see if I can make a test case

Ivo <icouckuy>
Tue 11 Feb 2014 07:57:56 AM UTC, comment #4:

Great,
thank you.

Anonymous
Mon 10 Feb 2014 03:05:36 PM UTC, comment #3:

This has already been re-fixed with this changeset:

http://hg.savannah.gnu.org/hgweb/octave/rev/b60a8a1cda54

Michael Goffioul <goffioul>
Project Member
Mon 10 Feb 2014 03:00:32 PM UTC, comment #2:

As of Octave 3.8.0,
this issue returned. The original test case still fails.

Anonymous
Wed 16 May 2012 08:22:52 PM UTC, comment #1:

I checked in the following change:

http://hg.savannah.gnu.org/hgweb/octave/rev/5b0872b5b234

John W. Eaton <jwe>
Project Administrator
Tue 15 May 2012 11:31:42 PM UTC, original submission:

Any superclass call of the form:
func@superclass(ident)
with no whitespace between superclass and ident, falls foul of:
{IDENT}@{IDENT}.{IDENT}{S}*
in the lexer; '.' should be '\.'

Patch and crude test case attached.

Anonymous

 

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

Attach File(s):
   
   
Comment:
   

Attached Files
file #25866:  classdef_supers.patch added by None (468B - text/x-patch)
file #25867:  classdef_test_super.m added by None (150B - text/x-objcsrc)

 

Depends on the following items: None found

Items that depend on this one: None found

 

Carbon-Copy List
  • -unavailable- added by icouckuy (Posted a comment)
  • -unavailable- added by goffioul (Posted a comment)
  • -unavailable- added by jwe (Posted a comment)
  • -unavailable- added by None (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 16 May 2012 08:22:52 PM UTCjweStatusNone=>Fixed
      Open/ClosedOpen=>Closed
    Tue 15 May 2012 11:31:42 PM UTCNoneAttached File-=>Added classdef_supers.patch, #25866
      Attached File-=>Added classdef_test_super.m, #25867

    Back to the top


    Powered by Savane 3.1-cleanup1