#bitfighter IRC Log

Index Search ←Prev date Next date→

IRC Log for 2026-04-20

Timestamps are in GMT/BST.

03:40:03BFLogBot Pull request #745 closed by Copilot | Title: Add Xtank vehicle designer specials, directional armor, and space/weight budget UI | https://github.com/bitfighter/bitfighter/pull/745
03:40:03BFLogBot Repo: bitfighter | Commit: fcc0bc5a14 | Author: Copilot | Message: Add Xtank vehicle designer specials, directional armor, and space/weight budget UI (#745) * Propagate armor/suspension/bumper through xtank move and ship sync Agent-Logs-Url: https://github.com/bitfighter/bitfighter/sessions/f9156017-6219-4f8f-88c5-18fef90d50b1 Co-authored-by: eykamp <1133323+eykamp@users.noreply.github.com> * Finish xtank design propagation and suspension tur
03:40:05BFLogBot n-rate support Agent-Logs-Url: https://github.com/bitfighter/bitfighter/sessions/f9156017-6219-4f8f-88c5-18fef90d50b1 Co-authored-by: eykamp <1133323+eykamp@users.noreply.github.com> * Add Xtank specials, directional armor, and space/weight budget UI - XtankShape.h/cpp: Add XtankSpecial enum (12 specials), XtankSpecialInfo struct with xtankSpecialInfos data, hasSpecial/toggleSpecial inline helpers. Update XtankDesign to include U16 specials b
03:40:06BFLogBot itmask and U8 armorSides[4] (per-side armor points). XtankDesign() and initForBody() initialize both new fields; initForBody sets armorSides from 10*body.size/4. - move.h/cpp: Add U16 specials field. initialize() zeros it; isEqualMove() compares it; pack()/unpack() serialize it as 16-bit int inside the bodyIndex >= 0 block. - UIGame.cpp: applyXtankDesign() syncs mCurrentMove.specials from design. - ship.cpp: processMove() syncs mXtankDesign.
03:40:08BFLogBot specials from mCurrentMove. XtankBodyMask pack/unpack include specials. damageObject() applies directional armor reduction based on hit side and armorSides[hitSide]; adds TODO for RAMPLATE collision damage. - UIXtankHelper.h: Insert PHASE_SPECIALS = 6 between BUMPERS and HEATSINK (HEATSINK = 7, WEAPONS = 8). Add mSpecialsItems vector, width members, and buildSpecialsItems() declaration. - UIXtankHelper.cpp: buildSpecialsItems() builds toggle-
03:40:09BFLogBot state items with cost/weight/space help text. All phase dispatch methods (render, currentPhaseItemCount, getItemsForPhase, widthForPhase, setHighlightedIndexForPhase, getSelectedLabelForPhase, commitHighlightedSelection) handle PHASE_SPECIALS. processInputCode() toggles specials on ENTER and hotkeys without auto-advancing. renderItemStatsColumn() shows active/inactive state and stats. renderCard() prepends [X]/[ ] checkbox to specials item name
03:40:11BFLogBot s; sPhaseTitles includes "Specials" at index 6. renderPreviewPanel() tracks space and weight budget (with over-budget warnings in red), adds armor/suspension/bumper/specials to totals, and shows active specials count. Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com> Co-authored-by: eykamp <1133323+eykamp@users.noreply.github.com> * Add xtank specials, per-side armor, budget UI, and directional armor damage Agent-Logs-Url:
03:40:12BFLogBot https://github.com/bitfighter/bitfighter/sessions/8ed13a22-11af-4f1c-80d0-049b90be913b Co-authored-by: eykamp <1133323+eykamp@users.noreply.github.com> * Untrack cmake-build directory Agent-Logs-Url: https://github.com/bitfighter/bitfighter/sessions/8ed13a22-11af-4f1c-80d0-049b90be913b Co-authored-by: eykamp <1133323+eykamp@users.noreply.github.com> --------- Co-authored-by: copilot-swe-agent[bot] <198982749+Copilot@users.noreply.github.co
03:40:14BFLogBot m> Co-authored-by: eykamp <1133323+eykamp@users.noreply.github.com>
04:34:16BFLogBot Repo: bitfighter | Commit: bcbbf813c7 | Author: Copilot | Message: Add side-based weapon assignment UI for xtank vehicle designer (UIXtankHelper.h / .cpp) Navigation model changes: - Tab/Shift-Tab always navigate the phase carousel - LEFT/RIGHT in PHASE_WEAPONS cycles turret slots (sides); elsewhere = phase nav - Hotkeys in weapon phase auto-advance to next slot, then exit the phase New in UIXtankHelper: - getTurretSideLabel(): labels each turr
04:35:29BFLogBot Pull request #746 opened by Copilot | Title: Xtank vehicle designer: side-based weapon assignment via LEFT/RIGHT slot cycling | https://github.com/bitfighter/bitfighter/pull/746
04:36:06BFLogBot Pull request #746 closed by Copilot | Title: Xtank vehicle designer: side-based weapon assignment via LEFT/RIGHT slot cycling | https://github.com/bitfighter/bitfighter/pull/746
04:36:06BFLogBot Repo: bitfighter | Commit: c9464067ee | Author: Copilot | Message: Add side-based weapon assignment UI for xtank vehicle designer (UIXtankHelper.h / .cpp) (#746) Navigation model changes: - Tab/Shift-Tab always navigate the phase carousel - LEFT/RIGHT in PHASE_WEAPONS cycles turret slots (sides); elsewhere = phase nav - Hotkeys in weapon phase auto-advance to next slot, then exit the phase New in UIXtankHelper: - getTurretSideLabel(): labels ea
04:37:28BFLogBot Pull request #747 opened by eykamp | Title: Revert "Xtank vehicle designer: side-based weapon assignment via LEFT/RIGHT slot cycling" | https://github.com/bitfighter/bitfighter/pull/747
04:37:34BFLogBot Pull request #747 closed by eykamp | Title: Revert "Xtank vehicle designer: side-based weapon assignment via LEFT/RIGHT slot cycling" | https://github.com/bitfighter/bitfighter/pull/747
05:16:31BFLogBot Repo: bitfighter | Commit: 609a7daaad | Author: eykamp | Message: Add side-based weapon assignment UI for xtank vehicle designer (UIXtankHelper.h / .cpp) Navigation model changes: - Tab/Shift-Tab always navigate the phase carousel - LEFT/RIGHT in PHASE_WEAPONS cycles turret slots (sides); elsewhere = phase nav - Hotkeys in weapon phase auto-advance to next slot, then exit the phase New in UIXtankHelper: - getTurretSideLabel(): labels each turre
05:38:19BFLogBot Repo: bitfighter | Commit: 99b39680c4 | Author: Copilot | Message: feat: per-side armor networking + UI allocation phase Agent-Logs-Url: https://github.com/bitfighter/bitfighter/sessions/ee0b06ed-3317-4c00-a7c1-90cfd2b2c06a Co-authored-by: eykamp <1133323+eykamp@users.noreply.github.com>
05:40:47BFLogBot Repo: bitfighter | Commit: e11959e3e3 | Author: Copilot | Message: fix: address code review feedback (constant, naming, hint text, comment) Agent-Logs-Url: https://github.com/bitfighter/bitfighter/sessions/ee0b06ed-3317-4c00-a7c1-90cfd2b2c06a Co-authored-by: eykamp <1133323+eykamp@users.noreply.github.com>
05:43:36BFLogBot Pull request #748 opened by Copilot | Title: Xtank: replicate per-side armor over network and add UI allocation phase | https://github.com/bitfighter/bitfighter/pull/748
05:46:18BFLogBot Repo: bitfighter | Commit: 609a7daaad | Author: eykamp | Message: Add side-based weapon assignment UI for xtank vehicle designer (UIXtankHelper.h / .cpp) Navigation model changes: - Tab/Shift-Tab always navigate the phase carousel - LEFT/RIGHT in PHASE_WEAPONS cycles turret slots (sides); elsewhere = phase nav - Hotkeys in weapon phase auto-advance to next slot, then exit the phase New in UIXtankHelper: - getTurretSideLabel(): labels each turre
05:49:05BFLogBot Pull request #748 closed by Copilot | Title: Xtank: replicate per-side armor over network and add UI allocation phase | https://github.com/bitfighter/bitfighter/pull/748
05:49:05BFLogBot Repo: bitfighter | Commit: 3cd34be30b | Author: Copilot | Message: Xtank: replicate per-side armor over network and add UI allocation phase (#748) * feat: per-side armor networking + UI allocation phase Agent-Logs-Url: https://github.com/bitfighter/bitfighter/sessions/ee0b06ed-3317-4c00-a7c1-90cfd2b2c06a Co-authored-by: eykamp <1133323+eykamp@users.noreply.github.com> * fix: address code review feedback (constant, naming, hint text, comment)
05:49:07BFLogBot Agent-Logs-Url: https://github.com/bitfighter/bitfighter/sessions/ee0b06ed-3317-4c00-a7c1-90cfd2b2c06a Co-authored-by: eykamp <1133323+eykamp@users.noreply.github.com> --------- Co-authored-by: copilot-swe-agent[bot] <198982749+Copilot@users.noreply.github.com> Co-authored-by: eykamp <1133323+eykamp@users.noreply.github.com> Co-authored-by: eykamp
05:50:57BFLogBot Comment on issue #748 | User: eykamp | Comment: I merged this PR and it doesn't build -- lots of undefined symbols. Can you check the xtank branch and fix the problems ... | https://github.com/bitfighter/bitfighter/pull/748#issuecomment-4279557752
06:14:39BFLogBot Repo: bitfighter | Commit: e4baa53476 | Author: Copilot | Message: Restore UIXtankHelper files and fix broken phase declarations Agent-Logs-Url: https://github.com/bitfighter/bitfighter/sessions/bef97f59-dec4-4135-a810-11fc55a7a779 Co-authored-by: eykamp <1133323+eykamp@users.noreply.github.com>
06:15:50BFLogBot Repo: bitfighter | Commit: 88fa2e5f88 | Author: Copilot | Message: Fix heat sink fire-rate format string in UIXtankHelper Agent-Logs-Url: https://github.com/bitfighter/bitfighter/sessions/bef97f59-dec4-4135-a810-11fc55a7a779 Co-authored-by: eykamp <1133323+eykamp@users.noreply.github.com>
06:17:23BFLogBot Repo: bitfighter | Commit: 18b9716c79 | Author: Copilot | Message: Polish UIXtankHelper phase/comment consistency and render call formatting Agent-Logs-Url: https://github.com/bitfighter/bitfighter/sessions/bef97f59-dec4-4135-a810-11fc55a7a779 Co-authored-by: eykamp <1133323+eykamp@users.noreply.github.com>
06:18:15BFLogBot Repo: bitfighter | Commit: 6aa4d2cb61 | Author: Copilot | Message: Clarify weapon phase constants and remove dead TOTAL_PHASES guard Agent-Logs-Url: https://github.com/bitfighter/bitfighter/sessions/bef97f59-dec4-4135-a810-11fc55a7a779 Co-authored-by: eykamp <1133323+eykamp@users.noreply.github.com>
06:19:41BFLogBot Repo: bitfighter | Commit: 70d4a81d71 | Author: Copilot | Message: Fix phase-title bounds check in UIXtankHelper render path Agent-Logs-Url: https://github.com/bitfighter/bitfighter/sessions/bef97f59-dec4-4135-a810-11fc55a7a779 Co-authored-by: eykamp <1133323+eykamp@users.noreply.github.com>
06:21:20BFLogBot Repo: bitfighter | Commit: 07853a7b9f | Author: Copilot | Message: Clarify phase comments in UIXtankHelper header and renderer Agent-Logs-Url: https://github.com/bitfighter/bitfighter/sessions/bef97f59-dec4-4135-a810-11fc55a7a779 Co-authored-by: eykamp <1133323+eykamp@users.noreply.github.com>
06:22:35BFLogBot Repo: bitfighter | Commit: 2b0d3778ec | Author: Copilot | Message: Make phase-title bounds check cross-platform and drop windows-only include Agent-Logs-Url: https://github.com/bitfighter/bitfighter/sessions/bef97f59-dec4-4135-a810-11fc55a7a779 Co-authored-by: eykamp <1133323+eykamp@users.noreply.github.com>
06:23:39BFLogBot Pull request #749 opened by Copilot | Title: Fix xtank helper regression after merged PR | https://github.com/bitfighter/bitfighter/pull/749
08:58:35BFLogBot Repo: bitfighter | Commit: 10ac0d9d88 | Author: google-labs-jules[bot] | Message: Fix several bugs in string and geometry utilities This commit addresses several safety and correctness issues in `zap/stringUtils.cpp` and `zap/GeomUtils.cpp`: - Added NULL pointer checks to `safeFilename`, `findPointerOfArg`, and `parseStringAndStripLeadingSlash`. - Added a bounds check to `concatenate` to handle negative starting indices. - Fixed potential undefi
08:58:36BFLogBot ned behavior in `lcase` and `ucase` by casting characters to `unsigned char` before calling `tolower`/`toupper`. - Added a size check to `isWoundClockwise` to prevent out-of-bounds access when called with an empty vector. New unit tests have been added to `bitfighter_test/TestStringUtils.cpp` and `bitfighter_test/TestGeomUtils.cpp` to verify these fixes and prevent regressions. I am an AI agent and these changes were generated to improve codeba
08:58:38BFLogBot Pull request #750 opened by eykamp | Title: Fix bugs and add regression tests in string and geometry utilities | https://github.com/bitfighter/bitfighter/pull/750
08:58:38BFLogBot se robustness. Co-authored-by: eykamp <1133323+eykamp@users.noreply.github.com>
13:07:37BFLogBot Pull request #749 closed by Copilot | Title: Fix xtank helper regression after merged PR | https://github.com/bitfighter/bitfighter/pull/749
13:07:38BFLogBot Repo: bitfighter | Commit: 25965230c1 | Author: Copilot | Message: Fix xtank helper regression after merged PR (#749) * Restore UIXtankHelper files and fix broken phase declarations Agent-Logs-Url: https://github.com/bitfighter/bitfighter/sessions/bef97f59-dec4-4135-a810-11fc55a7a779 Co-authored-by: eykamp <1133323+eykamp@users.noreply.github.com> * Fix heat sink fire-rate format string in UIXtankHelper Agent-Logs-Url: https://github.com/bit
13:07:39BFLogBot fighter/bitfighter/sessions/bef97f59-dec4-4135-a810-11fc55a7a779 Co-authored-by: eykamp <1133323+eykamp@users.noreply.github.com> * Polish UIXtankHelper phase/comment consistency and render call formatting Agent-Logs-Url: https://github.com/bitfighter/bitfighter/sessions/bef97f59-dec4-4135-a810-11fc55a7a779 Co-authored-by: eykamp <1133323+eykamp@users.noreply.github.com> * Clarify weapon phase constants and remove dead TOTAL_PHASES guard Ag
13:07:41BFLogBot ent-Logs-Url: https://github.com/bitfighter/bitfighter/sessions/bef97f59-dec4-4135-a810-11fc55a7a779 Co-authored-by: eykamp <1133323+eykamp@users.noreply.github.com> * Fix phase-title bounds check in UIXtankHelper render path Agent-Logs-Url: https://github.com/bitfighter/bitfighter/sessions/bef97f59-dec4-4135-a810-11fc55a7a779 Co-authored-by: eykamp <1133323+eykamp@users.noreply.github.com> * Clarify phase comments in UIXtankHelper header an
13:07:42BFLogBot d renderer Agent-Logs-Url: https://github.com/bitfighter/bitfighter/sessions/bef97f59-dec4-4135-a810-11fc55a7a779 Co-authored-by: eykamp <1133323+eykamp@users.noreply.github.com> * Make phase-title bounds check cross-platform and drop windows-only include Agent-Logs-Url: https://github.com/bitfighter/bitfighter/sessions/bef97f59-dec4-4135-a810-11fc55a7a779 Co-authored-by: eykamp <1133323+eykamp@users.noreply.github.com> --------- Co-author
13:07:44BFLogBot ed-by: copilot-swe-agent[bot] <198982749+Copilot@users.noreply.github.com> Co-authored-by: eykamp <1133323+eykamp@users.noreply.github.com>
13:08:18BFLogBot Repo: bitfighter | Commit: 4e387ef9cc | Author: eykamp | Message: Revert "Fix xtank helper regression after merged PR (#749)" This reverts commit 25965230c1d512f26fb696ffe73164709589ef7f.
13:08:23BFLogBot Pull request #751 opened by eykamp | Title: Revert "Fix xtank helper regression after merged PR" | https://github.com/bitfighter/bitfighter/pull/751
13:08:51BFLogBot Pull request #751 closed by eykamp | Title: Revert "Fix xtank helper regression after merged PR" | https://github.com/bitfighter/bitfighter/pull/751
13:08:51BFLogBot Comment on issue #751 | User: eykamp | Comment: Please ... | https://github.com/bitfighter/bitfighter/pull/751#issuecomment-4282800798
13:14:23BFLogBot Repo: bitfighter | Commit: 8aeeb318f7 | Author: Copilot | Message: Fix UIXtankHelper header merge omissions for xtank designer Agent-Logs-Url: https://github.com/bitfighter/bitfighter/sessions/c19e9051-34dd-470e-a1e3-5a73e3a03fcb Co-authored-by: eykamp <1133323+eykamp@users.noreply.github.com>
13:14:55BFLogBot Pull request #752 opened by Copilot | Title: Fix UIXtankHelper header merge omissions that broke xtank client builds | https://github.com/bitfighter/bitfighter/pull/752
13:23:31BFLogBot Pull request #750 closed by eykamp | Title: Fix bugs and add regression tests in string and geometry utilities | https://github.com/bitfighter/bitfighter/pull/750
13:23:32BFLogBot Repo: bitfighter | Commit: a95f145849 | Author: eykamp | Message: Fix several bugs in string and geometry utilities (#750) This commit addresses several safety and correctness issues in `zap/stringUtils.cpp` and `zap/GeomUtils.cpp`: - Added NULL pointer checks to `safeFilename`, `findPointerOfArg`, and `parseStringAndStripLeadingSlash`. - Added a bounds check to `concatenate` to handle negative starting indices. - Fixed potential undefined behav
13:23:33BFLogBot ior in `lcase` and `ucase` by casting characters to `unsigned char` before calling `tolower`/`toupper`. - Added a size check to `isWoundClockwise` to prevent out-of-bounds access when called with an empty vector. New unit tests have been added to `bitfighter_test/TestStringUtils.cpp` and `bitfighter_test/TestGeomUtils.cpp` to verify these fixes and prevent regressions. I am an AI agent and these changes were generated to improve codebase robust
13:23:35BFLogBot ness. Co-authored-by: google-labs-jules[bot] <161369871+google-labs-jules[bot]@users.noreply.github.com> Co-authored-by: eykamp <1133323+eykamp@users.noreply.github.com>
19:50:00BFLogBot Repo: bitfighter | Commit: a82b89e9b9 | Author: google-labs-jules[bot] | Message: Fix UB in tolower/toupper by casting char to unsigned char I am an AI agent. This commit fixes a common C++ bug where tolower and toupper from are called with signed char arguments. According to the C++ standard, these functions expect an int that is representable as unsigned char or is EOF. If char is signed, passing a negative value results in undefined
19:50:02BFLogBot behavior. Changes: - Added (unsigned char) casts to tolower and toupper calls in: - zap/stringUtils.cpp - tnl/platform.cpp - zap/UIMenuItems.cpp - zap/md5wrapper.cpp - Added caseInsensitiveStringCompareNonASCII test to bitfighter_test/TestStringUtils.cpp to verify the fix. Co-authored-by: eykamp <1133323+eykamp@users.noreply.github.com>
19:50:03BFLogBot Pull request #753 opened by eykamp | Title: Fix undefined behavior when calling tolower/toupper with signed char | https://github.com/bitfighter/bitfighter/pull/753
20:10:39BFLogBot Repo: bitfighter | Commit: 3b1487b796 | Author: google-labs-jules[bot] | Message: Fix UB in tolower/toupper by providing safe char overloads I am an AI agent. This commit fixes a widespread bug where tolower and toupper from are called with signed char arguments, which is undefined behavior for negative values. Instead of adding manual casts at every call site, I have provided safe overloads for tolower(char) and toupper(char) in the T
20:10:41BFLogBot NL namespace within tnl/tnlPlatform.h. These overloads perform the necessary cast to unsigned char before calling the standard library functions. Changes: - Modified tnl/tnlPlatform.h to include and provide safe tolower(char) and toupper(char) overloads. - Updated tnl/platform.cpp to use TNL::toupper explicitly to avoid potential ambiguity with std::toupper(int). - Modified zap/md5wrapper.cpp to include tnlPlatform.h and use the TNL nam
20:10:42BFLogBot espace, enabling the safe overloads. - Added a new unit test caseInsensitiveStringCompareNonASCII to bitfighter_test/TestStringUtils.cpp to verify that comparisons correctly handle characters with the high bit set. - Reverted manual (unsigned char) casts from previous implementation attempts in favor of the new overload-based approach. Co-authored-by: eykamp <1133323+eykamp@users.noreply.github.com>

Index Search ←Prev date Next date→

These logs were automatically created by BFLogBot on irc.freenode.net.