Skip to content

Conversation

@sean-purcell
Copy link
Contributor

@sean-purcell sean-purcell commented Mar 29, 2017

Didn't cause correctness issues, but technically was too large for its type.

@Cyan4973
Copy link
Contributor

Good point @iburinoc !
Silent truncation.
I'm surprised it did not generated any warning so far...

@Cyan4973 Cyan4973 merged commit dae3be8 into facebook:dev Mar 29, 2017
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants