Skip to content

Super inside inner class crashes#7144

Merged
sandersn merged 3 commits into
masterfrom
super-inside-inner-class-crashes
Feb 19, 2016
Merged

Super inside inner class crashes#7144
sandersn merged 3 commits into
masterfrom
super-inside-inner-class-crashes

Conversation

@sandersn

Copy link
Copy Markdown
Member

Fixes issue: #6859

super's containing class doesn't require base node

In a class nested inside a constructor, super refers to the outer
class' super, but when resolving a super call its containing class is
identified as the immediately containing class. Previously, the compiler
crashed, preventing the error from being reported correctly. Now it
handles this disparity and correctly reports the error.

In a class nested inside a constructor, `super` refers to the outer
class' `super`, but when resolving a super call its containing class is
identified as the immediately containing class. Previously, the compiler
crashed, preventing the error from being reported correctly. Now it
handles this disparity and correctly reports the error.
@mhegazy

mhegazy commented Feb 19, 2016

Copy link
Copy Markdown
Contributor

👍

1 similar comment
@yuit

yuit commented Feb 19, 2016

Copy link
Copy Markdown
Contributor

👍

@RyanCavanaugh

Copy link
Copy Markdown
Member

Any idea why GitHub thinks the testcase is binary?

@sandersn

Copy link
Copy Markdown
Member Author

Emacs says it's UTF-16. I think the normal encoding is UTF-8. I'll reencode as UTF8 and see what happens.

The reason it's UTF16 is that I was trying to make an equivalent of touch in powershell, which (I forgot) communicates in UTF16.

@sandersn

Copy link
Copy Markdown
Member Author

Yep. I guess git doesn't like UTF16.

sandersn added a commit that referenced this pull request Feb 19, 2016
@sandersn sandersn merged commit e46b7b0 into master Feb 19, 2016
@sandersn sandersn deleted the super-inside-inner-class-crashes branch February 19, 2016 16:45
@microsoft microsoft locked and limited conversation to collaborators Jun 19, 2018
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants