Page MenuHomeWildfire Games

Krinkle (Timo Tijhof)
Engineering

Projects

User Details

User Since
Jun 15 2019, 3:22 AM (22 w, 3 d)

Blog: https://timotijhof.net
Twitter: @TimoTijhof
GitHub: Krinkle
Wildfire Games Forum: Krinkle
Interests: JavaScript macOS Continuous Integration Localisation NL (Dutch)


Recent Activity

Oct 18 2019

Krinkle updated Krinkle.
Oct 18 2019, 9:56 PM

Oct 6 2019

Krinkle added inline comments to D2261: lobby,gamesetup: Fix remaining ESLint warnings.
Oct 6 2019, 1:45 AM
Krinkle added inline comments to D2261: lobby,gamesetup: Fix remaining ESLint warnings.
Oct 6 2019, 1:45 AM

Sep 29 2019

Krinkle updated the diff for D2279: UnitAI: Fix ESLint coding style warnings.

Updated IsAnimal to use stricter !== undefined check instead of bool cast.

Sep 29 2019, 1:35 AM
Krinkle added inline comments to D2279: UnitAI: Fix ESLint coding style warnings.
Sep 29 2019, 1:34 AM

Sep 25 2019

Krinkle resigned from rP22881: ptolemy houses, Enrique's pines.
Sep 25 2019, 9:04 PM
Krinkle added inline comments to D284: lobby: Display observer count in gameslist.
Sep 25 2019, 9:01 PM
Krinkle added inline comments to D2279: UnitAI: Fix ESLint coding style warnings.
Sep 25 2019, 8:56 PM

Sep 22 2019

Krinkle added inline comments to D2261: lobby,gamesetup: Fix remaining ESLint warnings.
Sep 22 2019, 10:08 PM
Krinkle added inline comments to D2279: UnitAI: Fix ESLint coding style warnings.
Sep 22 2019, 10:04 PM
Krinkle raised a concern with rP22881: ptolemy houses, Enrique's pines.

Found during today's test match on r22960.

Sep 22 2019, 12:31 AM

Sep 15 2019

Krinkle awarded D966: Don't show silhouettes through fields and berries a Orange Medal token.
Sep 15 2019, 1:41 AM

Sep 14 2019

Krinkle added inline comments to D284: lobby: Display observer count in gameslist.
Sep 14 2019, 8:13 PM
Krinkle updated the diff for D284: lobby: Display observer count in gameslist.

(Updated translation info to say "human" instead of "non-bot" player.)

Sep 14 2019, 8:07 PM
Krinkle updated the diff for D2279: UnitAI: Fix ESLint coding style warnings.
Sep 14 2019, 7:57 PM
Krinkle added inline comments to D2279: UnitAI: Fix ESLint coding style warnings.
Sep 14 2019, 7:57 PM
Krinkle added inline comments to D2279: UnitAI: Fix ESLint coding style warnings.
Sep 14 2019, 7:02 PM
Krinkle updated the diff for D2279: UnitAI: Fix ESLint coding style warnings.

Added line breaks in a multi-line conditional.

Sep 14 2019, 6:58 AM
Krinkle added inline comments to D2279: UnitAI: Fix ESLint coding style warnings.
Sep 14 2019, 6:57 AM
Krinkle updated the diff for D2279: UnitAI: Fix ESLint coding style warnings.

Removed redundant parenthesis.

Sep 14 2019, 6:49 AM
Krinkle updated the diff for D2005: Standardize "Territory Influence" in tooltips.

Rebased to re-trigger CI which had a false negative it seems.

Sep 14 2019, 5:28 AM

Sep 13 2019

Krinkle updated the diff for D284: lobby: Display observer count in gameslist.
Sep 13 2019, 1:52 AM
Krinkle added inline comments to D284: lobby: Display observer count in gameslist.
Sep 13 2019, 1:50 AM
Krinkle updated the diff for D284: lobby: Display observer count in gameslist.

Addressed concerns raised on previous diff.

Sep 13 2019, 1:50 AM
Krinkle planned changes to D2279: UnitAI: Fix ESLint coding style warnings.
Sep 13 2019, 12:08 AM
Krinkle added inline comments to D2279: UnitAI: Fix ESLint coding style warnings.
Sep 13 2019, 12:08 AM

Sep 10 2019

Krinkle updated the summary of D2279: UnitAI: Fix ESLint coding style warnings.
Sep 10 2019, 3:38 AM
Krinkle created D2279: UnitAI: Fix ESLint coding style warnings.
Sep 10 2019, 3:38 AM

Sep 8 2019

Krinkle added a comment to D2247: no decimals for armor tooltip.

Perhaps a "temporary" compromise could be to keep 1 digit only if the number is less than 1? I think as long as it is above 1, players may be more forgiving of rounding errors

Sep 8 2019, 4:42 PM

Sep 7 2019

Krinkle updated the diff for D2261: lobby,gamesetup: Fix remaining ESLint warnings.
Sep 7 2019, 10:38 PM

Sep 6 2019

Krinkle added inline comments to D2240: Move Mainmenu from XML to JS and use object oriented JS.
Sep 6 2019, 2:45 AM

Sep 5 2019

Krinkle updated the test plan for D2261: lobby,gamesetup: Fix remaining ESLint warnings.
Sep 5 2019, 1:17 AM
Krinkle created D2261: lobby,gamesetup: Fix remaining ESLint warnings.
Sep 5 2019, 1:14 AM

Sep 3 2019

Krinkle added inline comments to rP20697: Add helper function to apply arbitrary GUI tags (such as font and color)..
Sep 3 2019, 7:07 PM
Krinkle added inline comments to rP20697: Add helper function to apply arbitrary GUI tags (such as font and color)..
Sep 3 2019, 7:06 PM

Sep 2 2019

Krinkle updated the summary of D284: lobby: Display observer count in gameslist.
Sep 2 2019, 3:18 AM
Krinkle updated the diff for D284: lobby: Display observer count in gameslist.
Sep 2 2019, 2:41 AM
Krinkle added inline comments to D284: lobby: Display observer count in gameslist.
Sep 2 2019, 2:40 AM
Krinkle updated the diff for D284: lobby: Display observer count in gameslist.

Per CR:

  • Factored out color to a global variable.
  • Make use of the newer setStringTags function.
  • Keep translate() call inline (this means it isn't cached, but I guess that's fine?).
Sep 2 2019, 2:40 AM
Krinkle awarded D2079: Adds snapping to edges for buildings a Orange Medal token.
Sep 2 2019, 2:09 AM
Krinkle added a comment to rP21984: Corinthian Helmet Update:.

Created a thread at https://wildfiregames.com/forum/index.php?/topic/26940-hovering-shrubbery-bushes/.

Sep 2 2019, 12:10 AM

Sep 1 2019

Krinkle updated the diff for D2005: Standardize "Territory Influence" in tooltips.

Rebased on latest trunk.

Sep 1 2019, 7:16 PM
Krinkle added inline comments to D2240: Move Mainmenu from XML to JS and use object oriented JS.
Sep 1 2019, 4:15 PM
Krinkle added inline comments to D2213: Delete CSimulation2 m_MapSettings and redundant getters.
Sep 1 2019, 1:43 AM
Krinkle added inline comments to D2240: Move Mainmenu from XML to JS and use object oriented JS.
Sep 1 2019, 1:35 AM
Krinkle added inline comments to D2240: Move Mainmenu from XML to JS and use object oriented JS.
Sep 1 2019, 1:34 AM

Aug 31 2019

Krinkle created D2238: simulation: Use Object.create(parent) to create inheriting objects.
Aug 31 2019, 6:12 PM
Krinkle updated the summary of D2237: Linting: Enable no-caller and no-irregular-whitespace in ESLint.
Aug 31 2019, 5:26 PM
Krinkle created D2237: Linting: Enable no-caller and no-irregular-whitespace in ESLint.
Aug 31 2019, 5:25 PM
Krinkle updated the test plan for D1993: petra: Fix all ESLint and JSHint warnings.
Aug 31 2019, 5:08 PM
Krinkle updated the diff for D1993: petra: Fix all ESLint and JSHint warnings.
Aug 31 2019, 4:58 PM
Krinkle updated the summary of D1993: petra: Fix all ESLint and JSHint warnings.
Aug 31 2019, 4:57 PM

Aug 29 2019

Krinkle updated the diff for D2103: petra: Remove redundant 'PETRA' variable indirection.
Aug 29 2019, 2:53 PM

Aug 26 2019

elexis awarded D2103: petra: Remove redundant 'PETRA' variable indirection a Like token.
Aug 26 2019, 12:46 AM

Aug 25 2019

Krinkle updated the diff for D2103: petra: Remove redundant 'PETRA' variable indirection.
Aug 25 2019, 11:56 PM

Aug 23 2019

Krinkle reclaimed D2090: glooxwrapper: Remove unused private 'm_Owned' in Jingle::Session::Jingle.
Aug 23 2019, 1:56 PM

Aug 18 2019

Krinkle added inline comments to D1767: Rewrite Profiler2's GUI using D3 / CanvasJS.
Aug 18 2019, 2:46 PM

Aug 6 2019

Krinkle updated the diff for D2147: gui/gamesetup: Fix no-unmodified-loop-condition warning on 'g_IsNetworked'.

(Rebased)

Aug 6 2019, 10:21 PM
Krinkle updated the diff for D2146: gui/common: Fix ESLint code quality violations.

(Rebased)

Aug 6 2019, 10:20 PM

Aug 4 2019

Krinkle updated the diff for D2147: gui/gamesetup: Fix no-unmodified-loop-condition warning on 'g_IsNetworked'.
Aug 4 2019, 8:58 PM
Krinkle updated the Trac tickets for D2147: gui/gamesetup: Fix no-unmodified-loop-condition warning on 'g_IsNetworked'.
Aug 4 2019, 8:21 PM
Krinkle created D2147: gui/gamesetup: Fix no-unmodified-loop-condition warning on 'g_IsNetworked'.
Aug 4 2019, 8:21 PM
Krinkle created D2146: gui/common: Fix ESLint code quality violations.
Aug 4 2019, 7:59 PM
Krinkle updated the test plan for D2103: petra: Remove redundant 'PETRA' variable indirection.
Aug 4 2019, 1:19 AM

Jul 28 2019

Krinkle updated the diff for D2005: Standardize "Territory Influence" in tooltips.

Appearance:

  • Change phrasing away from the status quo of "Buildings in the territory do not decay", to be like @elexis suggestion: "This structure provides territory rooting".
  • Make the Territory tooltip included when hovering icons in the Tech Tree, this should make it relatively easy for players to quickly find which buildings can provide territory rooting. This change was done by moving the function from viewer#g_InfoFunctions to draw#g_StatsFunctions *thus more like getPopulationBonusTooltip, and less like getAurasTooltip). This two side-effects:
    • In the "More information" dialog, it is now crammed in the top part, instead of in the more spaced out body area.
    • When selecting an existing structure on the map that has rooting and hovering the central panel on the screen, this tooltip is not included.
Jul 28 2019, 5:12 PM
Krinkle updated the diff for D284: lobby: Display observer count in gameslist.
  • Use the coloredText() function instead of an inline [color] construct (the utility was added in Dec 2017, a few months after the patch started).
  • Switched from postfix to prefix increment.
  • Switch away from (N) format, back to +N, and allow the observer count formatting to be translated.
  • Change observer count colour from yellow to grey, so that the player count remains more prominent.
Jul 28 2019, 4:27 PM
Krinkle awarded D1781: Fix lobby chat performance issue. a Orange Medal token.
Jul 28 2019, 3:51 PM

Jul 27 2019

Krinkle updated the summary of D2103: petra: Remove redundant 'PETRA' variable indirection.
Jul 27 2019, 5:21 PM
Krinkle updated the summary of D2103: petra: Remove redundant 'PETRA' variable indirection.
Jul 27 2019, 5:18 PM
Krinkle updated the diff for D2103: petra: Remove redundant 'PETRA' variable indirection.

(Rebased. Hopefully Jenkins is working again properly now.)

Jul 27 2019, 5:15 PM

Jul 24 2019

Krinkle added inline comments to D284: lobby: Display observer count in gameslist.
Jul 24 2019, 2:30 AM

Jul 21 2019

Krinkle accepted D2080: Replace Eval calls with new ScriptInterface CreateObject and CreateArray functions.

LGTM! Statically reviewers and everything looks functionally identical, with one or two naming improvements.

Jul 21 2019, 11:44 PM
Krinkle added inline comments to D2080: Replace Eval calls with new ScriptInterface CreateObject and CreateArray functions.
Jul 21 2019, 10:51 PM
Krinkle added inline comments to D1684: Pass callback function to PushGuiPage and unify PopGuiPage/PopGuiPageCB.
Jul 21 2019, 3:04 AM
Krinkle updated the diff for D284: lobby: Display observer count in gameslist.
Jul 21 2019, 1:22 AM
Krinkle updated the diff for D284: lobby: Display observer count in gameslist.
  • Rebased on latest trunk.
  • Re-used earlier result of parsing the game.players string.
  • Implemented the suggestion of displaying (N) instead of +N.
  • Increase colour and brightness for consistency with other text.
Jul 21 2019, 1:21 AM
Krinkle added inline comments to D284: lobby: Display observer count in gameslist.
Jul 21 2019, 1:14 AM
Krinkle added inline comments to D284: lobby: Display observer count in gameslist.
Jul 21 2019, 1:14 AM
Krinkle commandeered D284: lobby: Display observer count in gameslist.
Jul 21 2019, 12:48 AM

Jul 20 2019

Krinkle updated the diff for D1993: petra: Fix all ESLint and JSHint warnings.
Jul 20 2019, 10:20 PM
Krinkle added inline comments to D2080: Replace Eval calls with new ScriptInterface CreateObject and CreateArray functions.
Jul 20 2019, 10:09 PM
Krinkle updated the summary of D2103: petra: Remove redundant 'PETRA' variable indirection.
Jul 20 2019, 10:05 PM
Krinkle updated the summary of D2103: petra: Remove redundant 'PETRA' variable indirection.
Jul 20 2019, 10:04 PM
Krinkle added a comment to D2103: petra: Remove redundant 'PETRA' variable indirection.

This has been split out of D2103 as being the only semantic change (due to fixing a quality issue rather than a style issue). This way D2103, can focus solely on auto-fixed style changes.

Jul 20 2019, 10:01 PM
Krinkle updated the summary of D2103: petra: Remove redundant 'PETRA' variable indirection.
Jul 20 2019, 9:59 PM
Krinkle created D2103: petra: Remove redundant 'PETRA' variable indirection.
Jul 20 2019, 9:59 PM
Krinkle added a comment to rP22518: Fix incorrect use of setlocale() in cppformat CStr tests following rP22378.

From Jenkins:

Jul 20 2019, 9:39 PM
Krinkle updated the diff for D2070: Linting: Remove controversial 'curly' eslint rules.

Reduced to only curly. Keeping no-multi-spaces enabled, which per @elexis is desirable to enforce, but just something we need to fix in a few existing files.

Jul 20 2019, 9:36 PM
Krinkle updated the summary of D2070: Linting: Remove controversial 'curly' eslint rules.
Jul 20 2019, 9:35 PM
Krinkle updated the Trac tickets for D2004: Linter: Fix ESLint semicolon-related warnings.
Jul 20 2019, 9:33 PM
Krinkle updated the Trac tickets for D2003: Linting: Enable 'no-floating-decimal' rule in ESLint.
Jul 20 2019, 9:33 PM
Krinkle updated the Trac tickets for D1993: petra: Fix all ESLint and JSHint warnings.
Jul 20 2019, 9:32 PM
Krinkle updated the Trac tickets for D2070: Linting: Remove controversial 'curly' eslint rules.
Jul 20 2019, 9:31 PM
Krinkle added inline comments to D2080: Replace Eval calls with new ScriptInterface CreateObject and CreateArray functions.
Jul 20 2019, 7:25 PM

Jul 17 2019

Krinkle planned changes to D2090: glooxwrapper: Remove unused private 'm_Owned' in Jingle::Session::Jingle.

Causes more warnings. Needs additional changes to either accommodate the absence of this field, or additional changes to make the original version not cause warnings.

Jul 17 2019, 7:06 PM
Krinkle added a comment to D2090: glooxwrapper: Remove unused private 'm_Owned' in Jingle::Session::Jingle.

Okay, it can't just be removed as-is.

Jul 17 2019, 2:43 AM
Krinkle created D2090: glooxwrapper: Remove unused private 'm_Owned' in Jingle::Session::Jingle.
Jul 17 2019, 2:33 AM
Krinkle resigned from rP22487: Remove Vector2D/Vector3D prototype workaround from EngineScriptConversions..

Fixed by rP22488.

Jul 17 2019, 2:18 AM
Krinkle added 1 auditor(s) for rP22487: Remove Vector2D/Vector3D prototype workaround from EngineScriptConversions.: Krinkle.

Note to self: rename FAILVOID to FAIL_VOID.

Jul 17 2019, 12:54 AM
Krinkle added inline comments to rP22487: Remove Vector2D/Vector3D prototype workaround from EngineScriptConversions..
Jul 17 2019, 12:54 AM