-
Notifications
You must be signed in to change notification settings - Fork 125
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Explictly test for players car in lane #477
Conversation
Signed-off-by: yzamir <kobi.zamir@gmail.com>
Not clear to me why the current code is implicit and the new code is explicit. I think both |
4a9fcad
to
7b3872f
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks the tests!
e9dadd8
to
4da7aab
Compare
28a6bf3
to
fbecad3
Compare
66efbe6
to
c0ae8d1
Compare
c0ae8d1
to
005bb2e
Compare
Signed-off-by: yzamir <kobi.zamir@gmail.com>
005bb2e
to
ac4613b
Compare
Issue:
When testing if a car is in lane, we do an implicit test, calculating the start and end x value of the player lane, it's better to use explicit calculation to get the lane directly from car position.