Boo

Error msg when accessing non-static outer class member

Details

  • Type: Bug Bug
  • Status: Resolved Resolved
  • Priority: Major Major
  • Resolution: Fixed
  • Affects Version/s: 0.5.6
  • Fix Version/s: 0.7
  • Component/s: Compiler
  • Labels:
    None
  • Number of attachments :
    1

Description

See: http://groups-beta.google.com/group/boolang/browse_frm/thread/5668b195c6dec549/0939ef078aed3db3#0939ef078aed3db3

Boo should throw an error if accessing a non-static member of outer class. (Actually, C# just says the reference cannot be found I believe)

One fix is around line 2029 in the processmethodbodies step, add:

//check if found entity can't possibly be a member of self:
if (member.DeclaringType != CurrentType &&
!(CurrentType.IsSubclassOf(member.DeclaringType)))

{ Error(CompilerErrorFactory.InstanceRequired(node, member.DeclaringType.ToString(), member.Name)); }

That 'fix' doesn't change name resolution though, to exclude non-static members in outer classes.

Issue Links

is duplicated by

Bug - A problem which impairs or prevents the functions of the product. BOO-460 Accessing member of outer class

  • Minor - Minor loss of function, or other problem where easy workaround is present.
  • Resolved - A resolution has been taken, and it is awaiting verification by reporter. From here issues are either reopened, or are closed.
is related to

Improvement - An improvement or enhancement to an existing feature or task. BOO-624 Java-style inner classes

  • Major - Major loss of function.
  • Open - The issue is open and ready for the assignee to start work on it.

Activity

Ascending order - Click to sort in descending order
Doug H made changes -
Field Original Value New Value
Resolution Won't Fix [ 2 ]
Status Open [ 1 ] Closed [ 6 ]
Rodrigo B. de Oliveira made changes -
Assignee Peter Johanson [ latexer ]
Status Closed [ 6 ] Reopened [ 4 ]
Resolution Won't Fix [ 2 ]
Rodrigo B. de Oliveira made changes -
Assignee Peter Johanson [ latexer ] Rodrigo B. de Oliveira [ bamboo ]
Rodrigo B. de Oliveira made changes -
Link This issue is duplicated by BOO-460 [ BOO-460 ]
Doug H made changes -
Attachment boo-450.patch [ 16857 ]
Rodrigo B. de Oliveira made changes -
Assignee Rodrigo B. de Oliveira [ bamboo ] Doug Holton [ dh ]
Fix Version/s 0.7 [ 11022 ]
Resolution Fixed [ 1 ]
Status Reopened [ 4 ] Resolved [ 5 ]
Doug H made changes -
Link This issue is related to BOO-624 [ BOO-624 ]

People

  • Assignee:
    Doug H
    Reporter:
    Doug H
Vote (0)
Watch (0)

Dates

  • Created:
    Updated:
    Resolved: