#bitfighter IRC Log

Index Search ←Prev date Next date→

IRC Log for 2026-04-19

Timestamps are in GMT/BST.

06:23:01BFLogBot Repo: bitfighter | Commit: 2172031bf6 | Author: eykamp | Message: Fix crashes with null strings
06:23:02BFLogBot Repo: bitfighter | Commit: aeb148a988 | Author: eykamp | Message: Improved vehicle designer
06:24:56BFLogBot Repo: bitfighter | Commit: 36f479e0e1 | Author: eykamp | Message: Fix crashes with null strings
06:32:05BFLogBot Repo: bitfighter | Commit: 9150f4027c | Author: Copilot | Message: 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>
06:37:25BFLogBot Repo: bitfighter | Commit: a9a56b91ec | Author: google-labs-jules[bot] | Message: Fix potential buffer overflow and invalid format in ftos The ftos(F32 f, S32 digits) function in zap/stringUtils.cpp had a potential buffer overflow when a large number of digits was requested, and it produced an invalid format string for negative digits. This commit adds bounds checking to clamp digits between 0 and 30 and handles negative values by falling back
06:37:26BFLogBot to the default ftos implementation. A unit test case is added to verify the fix. Identifying as an AI agent. Co-authored-by: eykamp <1133323+eykamp@users.noreply.github.com>
06:37:29BFLogBot Pull request #743 opened by eykamp | Title: Fix potential buffer overflow and invalid format in ftos | https://github.com/bitfighter/bitfighter/pull/743
06:37:53BFLogBot Repo: bitfighter | Commit: 2296ca50c7 | Author: Copilot | Message: Finish xtank design propagation and suspension turn-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>
17:38:01BFLogBot Repo: bitfighter | Commit: 0244ac7660 | Author: google-labs-jules[bot] | Message: Fix bugs in countCharInString and sanitizeForJson with unit tests This commit fixes two bugs in `zap/stringUtils.cpp`: 1. `countCharInString` now correctly handles strings with embedded null characters by iterating using `source.length()` instead of checking for a null terminator. 2. `sanitizeForJson` now includes a NULL check to prevent crashes when a NULL pointer
17:38:03BFLogBot is passed. Corresponding unit tests have been added to `bitfighter_test/TestStringUtils.cpp` to verify these fixes. As an AI agent, I identified these issues through code analysis and verified them with a new test case that demonstrated the failures before applying the fixes. Co-authored-by: eykamp <1133323+eykamp@users.noreply.github.com>
17:38:04BFLogBot Pull request #744 opened by eykamp | Title: Fix string utility bugs and add unit tests | https://github.com/bitfighter/bitfighter/pull/744
19:06:47BFLogBot Pull request #744 closed by eykamp | Title: Fix string utility bugs and add unit tests | https://github.com/bitfighter/bitfighter/pull/744
19:06:47BFLogBot Repo: bitfighter | Commit: 8c4be334dc | Author: eykamp | Message: Fix bugs in countCharInString and sanitizeForJson with unit tests (#744) This commit fixes two bugs in `zap/stringUtils.cpp`: 1. `countCharInString` now correctly handles strings with embedded null characters by iterating using `source.length()` instead of checking for a null terminator. 2. `sanitizeForJson` now includes a NULL check to prevent crashes when a NULL pointer is passe
19:06:49BFLogBot d. Corresponding unit tests have been added to `bitfighter_test/TestStringUtils.cpp` to verify these fixes. As an AI agent, I identified these issues through code analysis and verified them with a new test case that demonstrated the failures before applying the fixes. 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:08:35BFLogBot Pull request #743 closed by eykamp | Title: Fix potential buffer overflow and invalid format in ftos | https://github.com/bitfighter/bitfighter/pull/743
19:08:35BFLogBot Repo: bitfighter | Commit: dec4d30006 | Author: eykamp | Message: Fix potential buffer overflow and invalid format in ftos (#743) The ftos(F32 f, S32 digits) function in zap/stringUtils.cpp had a potential buffer overflow when a large number of digits was requested, and it produced an invalid format string for negative digits. This commit adds bounds checking to clamp digits between 0 and 30 and handles negative values by falling back to the de
19:08:36BFLogBot fault ftos implementation. A unit test case is added to verify the fix. Identifying as an AI agent. 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>
21:28:12BFLogBot Repo: bitfighter | Commit: 21b6594b4d | Author: eykamp | Message: Add libtommath so project will build again
21:28:14BFLogBot Repo: bitfighter | Commit: d484f10080 | Author: eykamp | Message: Rename tomcrypt to libtomcrypt
21:55:23BFLogBot Pull request #745 opened by Copilot | Title: Add Xtank vehicle designer specials, directional armor, and space/weight budget UI | https://github.com/bitfighter/bitfighter/pull/745
21:57:34BFLogBot Repo: bitfighter | Commit: 7387f84f7f | Author: Copilot | Message: 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 bitmask and U8 armorSides[4] (per-side armor points). XtankDesign() and initForBody() initialize both new fields; initFor
21:57:35BFLogBot Body 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.specials from mCurrentMove. XtankBodyMask pack/unpack include specials. damageObject() applies directional armor reducti
21:57:37BFLogBot on 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-state items with cost/weight/space help text. All phase dispatch methods (render, currentPhaseItemCount, getItemsForPhas
21:57:38BFLogBot e, 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 names; sPhaseTitles includes "Specials" at index 6. renderPreviewPanel() tracks space and weight budget (with over-budget wa
21:57:40BFLogBot rnings 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>
21:57:41BFLogBot Repo: bitfighter | Commit: c8afad0075 | Author: Copilot | Message: Add xtank specials, per-side armor, budget UI, and directional armor damage Agent-Logs-Url: https://github.com/bitfighter/bitfighter/sessions/8ed13a22-11af-4f1c-80d0-049b90be913b Co-authored-by: eykamp <1133323+eykamp@users.noreply.github.com>
21:59:29BFLogBot Repo: bitfighter | Commit: fbb93e2935 | Author: Copilot | Message: 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>

Index Search ←Prev date Next date→

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