MadhavM-117 / pacmanJS

Pacman in JavaScript
MIT License
0 stars 0 forks source link

Levels #14

Closed MadhavM-117 closed 5 years ago

MadhavM-117 commented 5 years ago

Haven't added any tests for this code. How do you think I should do that? Any suggestions?

MadhavM-117 commented 5 years ago

I've made the requested changes, but don't merge it just yet. Working on adding some tests.

Also, for the PACMAN_RADIUS * 2, changes. The getObjectBounds() function was written assuming that position would denote the center of the object, and the size would represent the total dimension of the object along the x and y axes, which for the Pacman object should be the diameter, not the radius, Hence, the modification.

Hope that makes sense.