Skip to content

Simplify handling of the 'any' type inside the checker.#3372

Merged
CyrusNajmabadi merged 5 commits into
masterfrom
taggedAny
Jun 4, 2015
Merged

Simplify handling of the 'any' type inside the checker.#3372
CyrusNajmabadi merged 5 commits into
masterfrom
taggedAny

Conversation

@CyrusNajmabadi

Copy link
Copy Markdown
Contributor
  1. Never reference 'anyType' directly, unless it is the result of some operation. i.e. do not use it when checking if some type is the 'any' type or not.
  2. Always use the TypeFlags.Any flag to see if a type is the 'any' type.
  3. When possible, propagate the 'unknown' type along to help suppress cascading errors.

I recommend looking at this commit-wise, and appending "?w=1" to make it easier to read.

@JsonFreeman

Copy link
Copy Markdown
Contributor

👍

CyrusNajmabadi added a commit that referenced this pull request Jun 4, 2015
Simplify handling of the 'any' type inside the checker.
@CyrusNajmabadi CyrusNajmabadi merged commit 343a923 into master Jun 4, 2015
@CyrusNajmabadi CyrusNajmabadi deleted the taggedAny branch June 4, 2015 00:25
@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.

3 participants