Skip to content

Increment/decrement operators should change type of union of IntegerRangeTypes#140

Merged
ondrejmirtes merged 1 commit intophpstan:masterfrom
cs278:issue-2954
Feb 12, 2020
Merged

Increment/decrement operators should change type of union of IntegerRangeTypes#140
ondrejmirtes merged 1 commit intophpstan:masterfrom
cs278:issue-2954

Conversation

@cs278
Copy link
Copy Markdown
Contributor

@cs278 cs278 commented Feb 12, 2020

Fixes phpstan/phpstan#2954

I did come up with another fix for the initially but personally I think this is the better choice, the other one is at: 7d5ad3a

@cs278 cs278 requested a review from ondrejmirtes February 12, 2020 00:32
@ondrejmirtes
Copy link
Copy Markdown
Member

I also like this version more, it already takes advantage of what’s in Scope::getType. Thank you!

@ondrejmirtes ondrejmirtes merged commit cf05c26 into phpstan:master Feb 12, 2020
@cs278 cs278 deleted the issue-2954 branch February 12, 2020 09:42
@cs278
Copy link
Copy Markdown
Contributor Author

cs278 commented Feb 12, 2020

I also like this version more, it already takes advantage of what’s in Scope::getType. Thank you!

Exactly my thinking 😄

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Increment/decement should reset int range

2 participants