Skip to content

Conversation

@yuit
Copy link
Contributor

@yuit yuit commented Nov 30, 2015

Issue #4211. This is to replaced very old PR #4814

Yui T added 7 commits November 30, 2015 09:39
Conflicts:
	src/compiler/diagnosticInformationMap.generated.ts
	src/compiler/diagnosticMessages.json
	src/compiler/types.ts
Conflicts:
	src/compiler/diagnosticInformationMap.generated.ts
	src/compiler/diagnosticMessages.json
	src/compiler/types.ts
@yuit
Copy link
Contributor Author

yuit commented Nov 30, 2015

@vladima @rbuckton I closed other PR in flavor of this. let me know if you have more comment.

@yuit
Copy link
Contributor Author

yuit commented Dec 8, 2015

@mhegazy if you could take a look too :)

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

getNodeLinks(container).flags |= NodeCheckFlags.HasSeenThisCall;

@yuit
Copy link
Contributor Author

yuit commented Jan 10, 2016

@mhegazy any more comments?

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

do we still need this? should not your check already catch this one?

@mhegazy
Copy link
Contributor

mhegazy commented Jan 28, 2016

Thanks!

@mhegazy
Copy link
Contributor

mhegazy commented Jan 28, 2016

👍

yuit added a commit that referenced this pull request Jan 28, 2016
Error when accessing this before super
@yuit yuit merged commit e954929 into master Jan 28, 2016
@yuit yuit deleted the fix4211 branch January 28, 2016 23:55
@yuit yuit mentioned this pull request Jan 29, 2016
@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.

4 participants