#bitfighter IRC Log

Index Search ←Prev date Next date→

IRC Log for 2026-04-25

Timestamps are in GMT/BST.

05:00:22BFLogBot Pull request #764 closed by eykamp | Title: Fix tie detection bug in individual games | https://github.com/bitfighter/bitfighter/pull/764
05:00:22BFLogBot Repo: bitfighter | Commit: 7024eedbc8 | Author: eykamp | Message: Fix tie detection bug in individual games (#764) This commit addresses a flaw in GameType::onGameOver where ties in individual games were not correctly detected if the tying players were not the last ones in the iteration. The logic now correctly handles ties and unique winners. I am an AI agent. Co-authored-by: google-labs-jules[bot] <161369871+google-labs-jules[bot]@users.nore
05:00:23BFLogBot ply.github.com> Co-authored-by: eykamp <1133323+eykamp@users.noreply.github.com>
06:32:24BFLogBot Repo: bitfighter | Commit: 4720cf730a | Author: google-labs-jules[bot] | Message: Fix bug in chopComment to ignore '#' inside double-quoted strings The chopComment function was naively stripping everything after the first '#' character, even if it was inside a quoted string. This caused issues with filenames or level names that contained '#' (e.g., "#not_a_comment"). This change: - Rewrites chopComment to track quote state. - Handles escaped do
06:32:25BFLogBot uble quotes ("") as per project conventions. - Adds comprehensive unit tests in TestStringUtils.cpp. This change was generated by an AI agent. Co-authored-by: eykamp <1133323+eykamp@users.noreply.github.com>
06:32:26BFLogBot Pull request #765 opened by eykamp | Title: Fix: chopComment incorrectly strips '#' inside quoted strings | https://github.com/bitfighter/bitfighter/pull/765
14:50:38BFLogBot Repo: bitfighter | Commit: ed3e5a513b | Author: eykamp | Message: Reduce use of {}s Refactor quote handling logic in string parsing.
14:50:52BFLogBot Pull request #765 closed by eykamp | Title: Fix: chopComment incorrectly strips '#' inside quoted strings | https://github.com/bitfighter/bitfighter/pull/765
14:50:53BFLogBot Repo: bitfighter | Commit: f09735b615 | Author: eykamp | Message: Fix: chopComment incorrectly strips '#' inside quoted strings (#765) * Fix bug in chopComment to ignore '#' inside double-quoted strings The chopComment function was naively stripping everything after the first '#' character, even if it was inside a quoted string. This caused issues with filenames or level names that contained '#' (e.g., "#not_a_comment"). This change: - Rewrite
14:50:54BFLogBot s chopComment to track quote state. - Handles escaped double quotes ("") as per project conventions. - Adds comprehensive unit tests in TestStringUtils.cpp. This change was generated by an AI agent. Co-authored-by: eykamp <1133323+eykamp@users.noreply.github.com> * Reduce use of {}s Refactor quote handling logic in string parsing. --------- Co-authored-by: google-labs-jules[bot] <161369871+google-labs-jules[bot]@users.noreply.github.com> Co
14:50:56BFLogBot -authored-by: eykamp <1133323+eykamp@users.noreply.github.com>
14:52:46BFLogBot Pull request #762 closed by eykamp | Title: Fix chopComment bug with hashtags inside quotes | https://github.com/bitfighter/bitfighter/pull/762
14:54:53BFLogBot Repo: bitfighter | Commit: 5919a394ee | Author: eykamp | Message: Add test cases for chopComment function
14:58:08BFLogBot Repo: bitfighter | Commit: dd78c518b5 | Author: eykamp | Message: Even more test cases for chopComment function
14:58:44BFLogBot Pull request #761 closed by eykamp | Title: Fix bug in chopComment to respect quoted strings | https://github.com/bitfighter/bitfighter/pull/761
17:59:57BFLogBot Repo: bitfighter | Commit: bfa57fc89d | Author: google-labs-jules[bot] | Message: Fix alphaNumberSort to use a natural sort algorithm This commit replaces the limited alphaNumberSort implementation with a more robust natural sort algorithm. Previously, it could only handle purely numeric strings or strings starting with digits, and it incorrectly sorted "file10.txt" before "file2.txt". The new implementation: - Scans strings segment by segment.
17:59:58BFLogBot - Compares numeric segments by value (ignoring leading zeros). - Correctly handles overflow by comparing lengths of numeric segments. - Uses the number of leading zeros as a tie-breaker. - Performs case-insensitive alphabetical comparison for non-numeric segments. Unit tests have been added to verify these changes. Co-authored-by: eykamp <1133323+eykamp@users.noreply.github.com>
18:00:00BFLogBot Pull request #766 opened by eykamp | Title: Implement natural sort in alphaNumberSort | https://github.com/bitfighter/bitfighter/pull/766
18:19:43BFLogBot Repo: bitfighter | Commit: d7f55c279b | Author: google-labs-jules[bot] | Message: Fix alphaNumberSort natural sort and CI build issues This commit addresses two main issues: 1. Implements a robust natural sort algorithm in alphaNumberSort to correctly handle numeric segments anywhere in strings (e.g., "file2" now correctly sorts before "file10"). 2. Fixes CI failures in Build-Linux jobs by ensuring the 'build' directory is created before attempt
18:19:45BFLogBot ing to change into it and running 'apt-get update' before installing dependencies. Unit tests for the natural sort have been added to TestStringUtils.cpp. Co-authored-by: eykamp <1133323+eykamp@users.noreply.github.com>
18:32:31BFLogBot Repo: bitfighter | Commit: 1605771548 | Author: google-labs-jules[bot] | Message: Fix natural sort and CI build workflows - Implemented a general natural sort algorithm in alphaNumberSort to correctly handle numeric segments anywhere in strings. - Added unit tests in TestStringUtils.cpp for natural sort behavior. - Updated CI workflows (.github/workflows/validate.yml and BuildPackages.yml) to include 'apt-get update' and use 'mkdir -p build' to
18:32:32BFLogBot prevent directory creation failures. 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.