You are viewing a single comment's thread from:

RE: Seed Game Engine - Multiple Program Support and Text & MapTile Rendering (v1.1.4)

in #utopian-io6 years ago

Thank you for your feedback :) It's great to hear such a positive review.

Thank you for taking the time to break down the PR and suggest improvements.

I purposely use == when comparing null as it also returns true when comparing against undefined as well, which I always handle both the same. Referenced from this Stack Overflow question.
As for the for loops, I haven't seen that practice before for simple incremental loops. I'll take a look at changing them over in the future; but I do enjoy using ++ when appropriate.

I'll watch for cleaning up my code and comment more before finalizing the PR in the future.

The rendering is still very early in development as well, I'd like to later implement 3D, keeping the camera targeted on your selected player. It'll most likely still be short of acceptable for a console gamer however ;)