Conversation
Member
daveallie
commented
Dec 6, 2025
- Rebuild HTML parsing using expat
- Potential to swap out the rest of the parsing to use expat in the future
martinbrook
pushed a commit
to martinbrook/crosspoint-reader
that referenced
this pull request
Jan 30, 2026
Address review comments crosspoint-reader#2, crosspoint-reader#3, and crosspoint-reader#10: - Extract duplicated bayer4x4 matrix, applyBayerDither4Level(), and drawPixelWithRenderMode() into shared DitherUtils.h - Extract duplicated PixelCache struct into shared PixelCache.h so both JPEG and PNG decoders use the same implementation - Add MAX_CACHE_BYTES (256KB) size limit to PixelCache::allocate() to proactively guard against oversized allocations on embedded targets
martinbrook
pushed a commit
to martinbrook/crosspoint-reader
that referenced
this pull request
Jan 30, 2026
Address review comments crosspoint-reader#2 and crosspoint-reader#3: - Remove duplicated bayer4x4, applyBayerDither4Level(), drawPixelWithRenderMode(), and PixelCache from JPEG decoder - Include shared DitherUtils.h and PixelCache.h instead
martinbrook
added a commit
to martinbrook/crosspoint-reader
that referenced
this pull request
Jan 30, 2026
Address review comments crosspoint-reader#2, crosspoint-reader#3, and crosspoint-reader#10: - Extract duplicated bayer4x4 matrix, applyBayerDither4Level(), and drawPixelWithRenderMode() into shared DitherUtils.h - Extract duplicated PixelCache struct into shared PixelCache.h so both JPEG and PNG decoders use the same implementation - Add MAX_CACHE_BYTES (256KB) size limit to PixelCache::allocate() to proactively guard against oversized allocations on embedded targets
martinbrook
added a commit
to martinbrook/crosspoint-reader
that referenced
this pull request
Jan 30, 2026
Address review comments crosspoint-reader#2 and crosspoint-reader#3: - Remove duplicated bayer4x4, applyBayerDither4Level(), drawPixelWithRenderMode(), and PixelCache from JPEG decoder - Include shared DitherUtils.h and PixelCache.h instead
martinbrook
added a commit
to martinbrook/crosspoint-reader
that referenced
this pull request
Feb 1, 2026
Address review comments crosspoint-reader#2, crosspoint-reader#3, and crosspoint-reader#10: - Extract duplicated bayer4x4 matrix, applyBayerDither4Level(), and drawPixelWithRenderMode() into shared DitherUtils.h - Extract duplicated PixelCache struct into shared PixelCache.h so both JPEG and PNG decoders use the same implementation - Add MAX_CACHE_BYTES (256KB) size limit to PixelCache::allocate() to proactively guard against oversized allocations on embedded targets
martinbrook
added a commit
to martinbrook/crosspoint-reader
that referenced
this pull request
Feb 1, 2026
Address review comments crosspoint-reader#2 and crosspoint-reader#3: - Remove duplicated bayer4x4, applyBayerDither4Level(), drawPixelWithRenderMode(), and PixelCache from JPEG decoder - Include shared DitherUtils.h and PixelCache.h instead
lukestein
pushed a commit
that referenced
this pull request
Feb 3, 2026
Address review comments #2, #3, and #10: - Extract duplicated bayer4x4 matrix, applyBayerDither4Level(), and drawPixelWithRenderMode() into shared DitherUtils.h - Extract duplicated PixelCache struct into shared PixelCache.h so both JPEG and PNG decoders use the same implementation - Add MAX_CACHE_BYTES (256KB) size limit to PixelCache::allocate() to proactively guard against oversized allocations on embedded targets
martinbrook
added a commit
to martinbrook/crosspoint-reader
that referenced
this pull request
Feb 5, 2026
Address review comments crosspoint-reader#2, crosspoint-reader#3, and crosspoint-reader#10: - Extract duplicated bayer4x4 matrix, applyBayerDither4Level(), and drawPixelWithRenderMode() into shared DitherUtils.h - Extract duplicated PixelCache struct into shared PixelCache.h so both JPEG and PNG decoders use the same implementation - Add MAX_CACHE_BYTES (256KB) size limit to PixelCache::allocate() to proactively guard against oversized allocations on embedded targets
martinbrook
added a commit
to martinbrook/crosspoint-reader
that referenced
this pull request
Feb 5, 2026
Address review comments crosspoint-reader#2 and crosspoint-reader#3: - Remove duplicated bayer4x4, applyBayerDither4Level(), drawPixelWithRenderMode(), and PixelCache from JPEG decoder - Include shared DitherUtils.h and PixelCache.h instead
martinbrook
added a commit
to martinbrook/crosspoint-reader
that referenced
this pull request
Feb 5, 2026
Address review comments crosspoint-reader#2, crosspoint-reader#3, and crosspoint-reader#10: - Extract duplicated bayer4x4 matrix, applyBayerDither4Level(), and drawPixelWithRenderMode() into shared DitherUtils.h - Extract duplicated PixelCache struct into shared PixelCache.h so both JPEG and PNG decoders use the same implementation - Add MAX_CACHE_BYTES (256KB) size limit to PixelCache::allocate() to proactively guard against oversized allocations on embedded targets
martinbrook
added a commit
to martinbrook/crosspoint-reader
that referenced
this pull request
Feb 5, 2026
Address review comments crosspoint-reader#2 and crosspoint-reader#3: - Remove duplicated bayer4x4, applyBayerDither4Level(), drawPixelWithRenderMode(), and PixelCache from JPEG decoder - Include shared DitherUtils.h and PixelCache.h instead
martinbrook
added a commit
to martinbrook/crosspoint-reader
that referenced
this pull request
Feb 8, 2026
Address review comments crosspoint-reader#2, crosspoint-reader#3, and crosspoint-reader#10: - Extract duplicated bayer4x4 matrix, applyBayerDither4Level(), and drawPixelWithRenderMode() into shared DitherUtils.h - Extract duplicated PixelCache struct into shared PixelCache.h so both JPEG and PNG decoders use the same implementation - Add MAX_CACHE_BYTES (256KB) size limit to PixelCache::allocate() to proactively guard against oversized allocations on embedded targets
martinbrook
added a commit
to martinbrook/crosspoint-reader
that referenced
this pull request
Feb 8, 2026
Address review comments crosspoint-reader#2 and crosspoint-reader#3: - Remove duplicated bayer4x4, applyBayerDither4Level(), drawPixelWithRenderMode(), and PixelCache from JPEG decoder - Include shared DitherUtils.h and PixelCache.h instead
madeddie
pushed a commit
to madeddie/crosspoint-reader
that referenced
this pull request
Feb 11, 2026
Address review comments crosspoint-reader#2, crosspoint-reader#3, and crosspoint-reader#10: - Extract duplicated bayer4x4 matrix, applyBayerDither4Level(), and drawPixelWithRenderMode() into shared DitherUtils.h - Extract duplicated PixelCache struct into shared PixelCache.h so both JPEG and PNG decoders use the same implementation - Add MAX_CACHE_BYTES (256KB) size limit to PixelCache::allocate() to proactively guard against oversized allocations on embedded targets
madeddie
pushed a commit
to madeddie/crosspoint-reader
that referenced
this pull request
Feb 11, 2026
Address review comments crosspoint-reader#2 and crosspoint-reader#3: - Remove duplicated bayer4x4, applyBayerDither4Level(), drawPixelWithRenderMode(), and PixelCache from JPEG decoder - Include shared DitherUtils.h and PixelCache.h instead
martinbrook
added a commit
to martinbrook/crosspoint-reader
that referenced
this pull request
Feb 13, 2026
Address review comments crosspoint-reader#2, crosspoint-reader#3, and crosspoint-reader#10: - Extract duplicated bayer4x4 matrix, applyBayerDither4Level(), and drawPixelWithRenderMode() into shared DitherUtils.h - Extract duplicated PixelCache struct into shared PixelCache.h so both JPEG and PNG decoders use the same implementation - Add MAX_CACHE_BYTES (256KB) size limit to PixelCache::allocate() to proactively guard against oversized allocations on embedded targets
martinbrook
added a commit
to martinbrook/crosspoint-reader
that referenced
this pull request
Feb 13, 2026
Address review comments crosspoint-reader#2 and crosspoint-reader#3: - Remove duplicated bayer4x4, applyBayerDither4Level(), drawPixelWithRenderMode(), and PixelCache from JPEG decoder - Include shared DitherUtils.h and PixelCache.h instead
Danbrose
pushed a commit
to Danbrose/crosspoint-reader
that referenced
this pull request
Feb 24, 2026
* feat: add extra small fonz sizes * added spanish string * fix formatting * reodered `FONT_SIZE` enum for keep old order * add romanian string * fix: setting now corresponds to font (error introduced in 072a44c) * Update russian.yaml Added line 176 with the translation of "extra small" font size --------- Co-authored-by: Ariel Lindemann <[email protected]>
vjapolitzer
pushed a commit
to vjapolitzer/crosspoint-reader
that referenced
this pull request
Feb 26, 2026
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.