Viewing a response to: @jaegar/seed-game-engine-multiple-program-support-and-text-and-maptile-rendering-v1-1-4
Hi, so honestly I don't do WebGL so I'll review on what I know :) Let's go for the review: - First congrats on this project. I'm curious to see what comes next. Graphic programming is usually hard and involves a lot of maths. At least that's what I remember when I did this at school :D - +1 for the quality of the commits and use of the PR - https://github.com/JaegarSarauer/SeedGameEngine/pull/9/files#diff-27574bc0ca0795c0d8a7d9ae3ca11c57L16 => [Best practice](https://eslint.org/docs/rules/eqeqeq#require--and--eqeqeq) is to use `===` - https://github.com/JaegarSarauer/SeedGameEngine/pull/9/files#diff-eecf63f3d684d4bd35bcb69b57a281dfR52 => little comment overkill but overall your comments are awesome - https://github.com/JaegarSarauer/SeedGameEngine/pull/9/files#diff-eecf63f3d684d4bd35bcb69b57a281dfR77 => [Best practice](https://eslint.org/docs/rules/no-plusplus) is to use `x += 1`, you got a few of them - https://github.com/JaegarSarauer/SeedGameEngine/pull/9/files#diff-206715a518d6bcbb953b26a3767ad14aR162 => avoid dead code - https://github.com/JaegarSarauer/SeedGameEngine/blob/f655eb977213100e5eeb166f3cb4332165fff387/src/manager/TextureManager.js#L148 => I'm more used to see `resolve, reject` as `res` is used for response for expressjs but that's because I mostly do web development. In overall your code is of great quality. Hats off. The rendering though I'm not a fan ha ha because I'm a console gamer and loving AAA games. Good luck on your roadmap because you got some heavy features to develop. Your contribution has been evaluated according to [Utopian policies and guidelines](https://join.utopian.io/guidelines), as well as a predefined set of questions pertaining to the category. To view those questions and the relevant answers related to your post, [click here](https://review.utopian.io/result/3/1211111). ---- Need help? Write a ticket on https://support.utopian.io/. Chat with us on [Discord](https://discord.gg/uTyJkNm). [[utopian-moderator]](https://join.utopian.io/)
| author | gregory.latinier |
|---|---|
| permlink | re-jaegar-seed-game-engine-multiple-program-support-and-text-and-maptile-rendering-v1-1-4-20180728t224731316z |
| category | utopian-io |
| json_metadata | {"tags":["utopian-io"],"links":["https://github.com/JaegarSarauer/SeedGameEngine/pull/9/files#diff-27574bc0ca0795c0d8a7d9ae3ca11c57L16","https://eslint.org/docs/rules/eqeqeq#require--and--eqeqeq","https://github.com/JaegarSarauer/SeedGameEngine/pull/9/files#diff-eecf63f3d684d4bd35bcb69b57a281dfR52","https://github.com/JaegarSarauer/SeedGameEngine/pull/9/files#diff-eecf63f3d684d4bd35bcb69b57a281dfR77","https://eslint.org/docs/rules/no-plusplus","https://github.com/JaegarSarauer/SeedGameEngine/pull/9/files#diff-206715a518d6bcbb953b26a3767ad14aR162","https://github.com/JaegarSarauer/SeedGameEngine/blob/f655eb977213100e5eeb166f3cb4332165fff387/src/manager/TextureManager.js#L148","https://join.utopian.io/guidelines","https://review.utopian.io/result/3/1211111","https://support.utopian.io/","https://discord.gg/uTyJkNm","https://join.utopian.io/"],"app":"steemit/0.1"} |
| created | 2018-07-28 22:47:30 |
| last_update | 2018-07-28 22:47:30 |
| depth | 1 |
| children | 2 |
| last_payout | 2018-08-04 22:47:30 |
| cashout_time | 1969-12-31 23:59:59 |
| total_payout_value | 2.441 HBD |
| curator_payout_value | 0.364 HBD |
| pending_payout_value | 0.000 HBD |
| promoted | 0.000 HBD |
| body_length | 2,025 |
| author_reputation | 34,278,323,818,021 |
| root_title | "Seed Game Engine - Multiple Program Support and Text & MapTile Rendering (v1.1.4)" |
| beneficiaries | [] |
| max_accepted_payout | 1,000,000.000 HBD |
| percent_hbd | 10,000 |
| post_id | 66,325,456 |
| net_rshares | 1,594,929,848,873 |
| author_curate_reward | "" |
| voter | weight | wgt% | rshares | pct | time |
|---|---|---|---|---|---|
| espoem | 0 | 17,629,160,050 | 15% | ||
| utopian-io | 0 | 1,526,467,636,896 | 1% | ||
| amosbastian | 0 | 50,496,349,746 | 100% | ||
| mops2e | 0 | 336,702,181 | 10% |
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](https://stackoverflow.com/questions/20687983/whats-the-difference-between-a-null-and-a-null). 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 ;)
| author | jaegar |
|---|---|
| permlink | re-gregorylatinier-re-jaegar-seed-game-engine-multiple-program-support-and-text-and-maptile-rendering-v1-1-4-20180731t012416684z |
| category | utopian-io |
| json_metadata | {"tags":["utopian-io"],"links":["https://stackoverflow.com/questions/20687983/whats-the-difference-between-a-null-and-a-null"],"app":"steemit/0.1"} |
| created | 2018-07-31 01:24:15 |
| last_update | 2018-07-31 01:24:15 |
| depth | 2 |
| children | 0 |
| last_payout | 2018-08-07 01:24:15 |
| cashout_time | 1969-12-31 23:59:59 |
| total_payout_value | 0.000 HBD |
| curator_payout_value | 0.000 HBD |
| pending_payout_value | 0.000 HBD |
| promoted | 0.000 HBD |
| body_length | 941 |
| author_reputation | 4,270,430,860,513 |
| root_title | "Seed Game Engine - Multiple Program Support and Text & MapTile Rendering (v1.1.4)" |
| beneficiaries | [] |
| max_accepted_payout | 1,000,000.000 HBD |
| percent_hbd | 10,000 |
| post_id | 66,578,797 |
| net_rshares | 0 |
Hey @gregory.latinier Here's a tip for your valuable feedback! @Utopian-io loves and incentivises informative comments. **Contributing on Utopian** Learn how to contribute on <a href="https://join.utopian.io">our website</a>. **Want to chat? Join us on Discord https://discord.gg/h52nFrV.** <a href="https://v2.steemconnect.com/sign/account-witness-vote?witness=utopian-io&approve=1">Vote for Utopian Witness!</a>
| author | utopian-io |
|---|---|
| permlink | 20180728t230309179z |
| category | utopian-io |
| json_metadata | {"tags":["utopian.tip"],"app":"utopian-io"} |
| created | 2018-07-28 23:03:09 |
| last_update | 2018-07-28 23:03:09 |
| depth | 2 |
| children | 0 |
| last_payout | 2018-08-04 23:03:09 |
| cashout_time | 1969-12-31 23:59:59 |
| total_payout_value | 0.000 HBD |
| curator_payout_value | 0.000 HBD |
| pending_payout_value | 0.000 HBD |
| promoted | 0.000 HBD |
| body_length | 416 |
| author_reputation | 152,955,367,999,756 |
| root_title | "Seed Game Engine - Multiple Program Support and Text & MapTile Rendering (v1.1.4)" |
| beneficiaries | [] |
| max_accepted_payout | 1,000,000.000 HBD |
| percent_hbd | 10,000 |
| post_id | 66,326,475 |
| net_rshares | 0 |
hiveblocks