Skip to content

Update string compare return values#1412

Merged
ondrejmirtes merged 1 commit intophpstan:1.7.xfrom
bytestream:strcasecmp
Jun 14, 2022
Merged

Update string compare return values#1412
ondrejmirtes merged 1 commit intophpstan:1.7.xfrom
bytestream:strcasecmp

Conversation

@bytestream
Copy link
Copy Markdown
Contributor

@staabm
Copy link
Copy Markdown
Contributor

staabm commented Jun 10, 2022

same for

https://github.com/phpstan/phpstan-src/blob/d1e0b1bf15516a020c1aff77f9778a1ddf32ac79/resources/functionMap.php#L11780-L11783

?

maybe also substr_compare

additionally a test-case regarding your reported error would be great.

@bytestream
Copy link
Copy Markdown
Contributor Author

@ondrejmirtes this is 🟢 now 🎉

@ondrejmirtes
Copy link
Copy Markdown
Member

No merge commits please, but clean linear history on top of 1.7.x.

@bytestream
Copy link
Copy Markdown
Contributor Author

I've rebased and squashed 👍

@ondrejmirtes ondrejmirtes merged commit 1c9f823 into phpstan:1.7.x Jun 14, 2022
@ondrejmirtes
Copy link
Copy Markdown
Member

Thank you!

@bytestream bytestream deleted the strcasecmp branch June 14, 2022 10:24
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.

3 participants