Skip to content

feat: bitstream#12

Open
leia-uwu wants to merge 1 commit into
mainfrom
feat/bitstream
Open

feat: bitstream#12
leia-uwu wants to merge 1 commit into
mainfrom
feat/bitstream

Conversation

@leia-uwu
Copy link
Copy Markdown
Contributor

(also i want to eventually move the Buff class outside of it for other stuff to use)

@leia-uwu leia-uwu assigned ei-pi and unassigned ei-pi Jan 23, 2026
@leia-uwu leia-uwu requested a review from ei-pi January 23, 2026 03:41
Comment thread include/gmi/math/BitStream.h Outdated
Comment thread include/gmi/math/BitStream.h Outdated
Comment thread include/gmi/math/BitStream.h Outdated
Comment thread include/gmi/math/BitStream.h Outdated
Comment thread include/gmi/math/BitStream.h Outdated
Comment thread include/gmi/math/BitStream.h Outdated
Comment thread include/gmi/misc/BitStream.h
Comment thread tests/bitStreamTest.cpp
Copy link
Copy Markdown

@ei-pi ei-pi left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

looks overall good, just a few silly typos to fix, and a test case to verify (if it fails, i think I know why)

@leia-uwu leia-uwu force-pushed the feat/bitstream branch 2 times, most recently from 28381fc to 3dbbfe5 Compare January 27, 2026 01:29
@leia-uwu leia-uwu marked this pull request as ready for review January 27, 2026 01:30
@leia-uwu leia-uwu force-pushed the feat/bitstream branch 2 times, most recently from aa6a699 to bfe61de Compare January 28, 2026 21:00
@ei-pi ei-pi self-requested a review January 29, 2026 19:35
Comment thread include/gmi/misc/BitStream.h
Comment thread include/gmi/math/BitStream.h Outdated
Comment thread include/gmi/misc/BitStream.h
Comment thread include/gmi/misc/BitStream.h
Comment thread src/misc/BitStream.cpp
Comment thread src/misc/BitStream.cpp
Comment thread src/math/BitStream.cpp Outdated
Comment thread src/misc/BitStream.cpp
@leia-uwu leia-uwu force-pushed the feat/bitstream branch 3 times, most recently from 31fe30e to 2c51af4 Compare January 30, 2026 23:40
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants