#bitfighter IRC Log

Index Search ←Prev date Next date→

IRC Log for 2026-05-04

Timestamps are in GMT/BST.

05:30:35BFDiscordBridge has joined


06:47:51BFLogBot Repo: bitfighter | Commit: 76535cfbcf | Author: google-labs-jules[bot] | Message: Fix TNL::Vector memory layout and improve build system portability.  This commit addresses a bug in the TNL::Vector specialization where it was using std::vector for internal storage. Since sizeof(bool) is typically 1, this caused incorrect memory indexing and potential corruption when using the address() method to access elements. We now use std::v


06:47:53BFLogBot ector to ensure the memory layout matches expectations.  Included changes: - Modified tnl/tnlVector.h to use std::vector for Vector. - Added bitfighter_test/TestTnlVectorBool.cpp with comprehensive tests for Vector. - Registered the new test in zap/bitfighter_test.cmake and removed a duplicate entry. - Removed a redundant block of include statements in tnl/tnlString.h. - Fixed the libtomcrypt dependency name in master/CMakeLis


06:47:54BFLogBot Pull request #781 opened by eykamp | Title: Fix TNL::Vector memory layout and build system issues | https://github.com/bitfighter/bitfighter/pull/781


06:47:54BFLogBot ts.txt to match the actual target name ('tomcrypt').  Note: I am an AI agent assisting with this task.  Co-authored-by: eykamp <1133323+eykamp@users.noreply.github.com>


16:39:14BFLogBot Repo: bitfighter | Commit: 1cf7663238 | Author: google-labs-jules[bot] | Message: Fix memory alignment bug in TNL::Vector  Modified the `TNL::Vector` specialization to use `U8` for its underlying storage instead of `S32`. This ensures that the `address()` method returns a pointer compatible with standard `bool*` pointer arithmetic (typically 1-byte aligned), preventing incorrect memory access when iterating via the raw pointer.  Inclu


16:39:15BFLogBot ded a new test suite `TestTnlVectorBool.cpp` to verify correct alignment and basic vector operations.  Also cleaned up redundant include statements in `tnl/tnlString.h` and removed a duplicate test entry in `zap/bitfighter_test.cmake`.  Co-authored-by: eykamp <1133323+eykamp@users.noreply.github.com>


16:39:17BFLogBot Pull request #782 opened by eykamp | Title: Fix memory alignment bug in TNL::Vector | https://github.com/bitfighter/bitfighter/pull/782


17:09:05BFLogBot Repo: bitfighter | Commit: bec3d26f06 | Author: google-labs-jules[bot] | Message: Fix memory alignment bug in TNL::Vector  Modified the `TNL::Vector` specialization to use `U8` for its underlying storage instead of `S32`. This ensures that the `address()` method returns a pointer compatible with standard `bool*` pointer arithmetic (typically 1-byte aligned), preventing incorrect memory access when iterating via the raw pointer.  Inclu


17:09:06BFLogBot ded a new test suite `TestTnlVectorBool.cpp` to verify correct alignment and basic vector operations.  Also cleaned up redundant include statements in `tnl/tnlString.h` and removed duplicate test entries in `zap/bitfighter_test.cmake`.  Reverted an unrelated build fix in `master/CMakeLists.txt` per user instructions.  Co-authored-by: eykamp <1133323+eykamp@users.noreply.github.com>


17:27:16BFLogBot Repo: bitfighter | Commit: 4923ee9335 | Author: google-labs-jules[bot] | Message: Fix memory alignment bug in TNL::Vector  Modified the `TNL::Vector` specialization to use `U8` for its underlying storage instead of `S32`. This ensures that the `address()` method returns a pointer compatible with standard `bool*` pointer arithmetic (typically 1-byte aligned), preventing incorrect memory access when iterating via the raw pointer.  Inclu


17:27:18BFLogBot ded a new test suite `TestTnlVectorBool.cpp` to verify correct alignment and basic vector operations.  Also cleaned up redundant include statements in `tnl/tnlString.h` and removed duplicate test entries in `zap/bitfighter_test.cmake`.  Note: This PR excludes the fix for the `libtomcrypt` dependency in `master/CMakeLists.txt` as requested.  Co-authored-by: eykamp <1133323+eykamp@users.noreply.github.com>


18:05:56BFLogBot Repo: bitfighter | Commit: 06d608030f | Author: google-labs-jules[bot] | Message: Fix TNL::Vector storage and address() behavior  Changed VectorBase storage from std::vector to std::vector to ensure that pointer arithmetic via address() works correctly. Added unit tests to verify the fix and performed minor cleanup in build files.  Author: AI Agent  Co-authored-by: eykamp <1133323+eykamp@users.noreply.github.com>


18:06:00BFLogBot Pull request #783 opened by eykamp | Title: Fix TNL::Vector storage and address() behavior | https://github.com/bitfighter/bitfighter/pull/783


21:50:48BFLogBot Repo: bitfighter | Commit: f042bd63df | Author: google-labs-jules[bot] | Message: Fix memory alignment bug in TNL::Vector  Modified the `TNL::Vector` specialization to use `U8` for its underlying storage instead of `S32`. This ensures that the `address()` method returns a pointer compatible with standard `bool*` pointer arithmetic (typically 1-byte aligned), preventing incorrect memory access when iterating via the raw pointer.  Inclu


21:50:50BFLogBot ded a new test suite `TestTnlVectorBool.cpp` to verify correct alignment and basic vector operations.  Also cleaned up redundant include statements in `tnl/tnlString.h` and removed duplicate test entries in `zap/bitfighter_test.cmake`.  Note: This PR excludes the fix for the `libtomcrypt` dependency in `master/CMakeLists.txt` as requested. I have confirmed that the necessary includes in `tnl/tnlString.h` are preserved.  Co-authored-by: eykamp <11


21:50:51BFLogBot 33323+eykamp@users.noreply.github.com>

Index Search ←Prev date Next date→

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