Skip to content

Simplify some places in the compiler where we have esoteric declarations.#2162

Merged
CyrusNajmabadi merged 7 commits into
masterfrom
declarations
Feb 27, 2015
Merged

Simplify some places in the compiler where we have esoteric declarations.#2162
CyrusNajmabadi merged 7 commits into
masterfrom
declarations

Conversation

@CyrusNajmabadi
Copy link
Copy Markdown
Contributor

No description provided.

Comment thread src/compiler/emitter.ts Outdated
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Remove

Comment thread src/compiler/checker.ts
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Add test

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

Done.

@JsonFreeman
Copy link
Copy Markdown
Contributor

👍

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

i do not think this is correct. you can not put a breakpoint on "e".

CyrusNajmabadi added a commit that referenced this pull request Feb 27, 2015
Simplify some places in the compiler where we have esoteric declarations.
@CyrusNajmabadi CyrusNajmabadi merged commit f4158ee into master Feb 27, 2015
@CyrusNajmabadi CyrusNajmabadi deleted the declarations branch February 27, 2015 07:21
@microsoft microsoft locked and limited conversation to collaborators Jun 18, 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