darthrubik/drharness
1
0
Fork 0
Code Issues 13 Pull Requests 1 Actions Packages Projects Releases 3 Wiki Activity

bugfix/utf8 #38

Merged
darthrubik merged 2 commits from bugfix/utf8 into master 2026-07-10 16:02:35 -05:00
Conversation 0 Commits 2 Files Changed 6
+265 -1
darthrubik commented 2026-07-10 16:02:18 -05:00
Owner
Copy Link
Copy Source
No description provided.
darthrubik added 2 commits 2026-07-10 16:02:18 -05:00
Adding the ability to handle non utf8 encodings 11484fec1e
Implement issue 180: Add unit tests for encoding (commit 11484fe) 4b3a277a15 
- Add test_encoding.sh with 11 tests for UTF-8 encoding validation and conversion
- Add -8 debug flag to test ensure_utf8 function with hex-encoded input
- Fix bug in ensure_utf8: check null terminator at loop start to prevent
  undefined behavior when reading past the end of the string
- Tests cover:
  - Valid ASCII passthrough
  - Valid UTF-8 2-byte, 3-byte, and 4-byte sequences
  - Latin-1 byte conversion (0xe9 -> c3a9)
  - Multiple Latin-1 bytes (café)
  - Isolated continuation byte conversion (0x80 -> c280)
  - Mixed valid/invalid bytes
  - Empty string handling
  - Integration test: read_file with non-UTF8 content
darthrubik merged commit eb161931bb into master 2026-07-10 16:02:35 -05:00
darthrubik deleted branch bugfix/utf8 2026-07-10 16:02:35 -05:00
Sign in to join this conversation.
Reviewers
No Reviewers
Labels
Clear labels
bug
Something is not working
 duplicate
This issue or pull request already exists
 enhancement
New feature
 help wanted
Need some help
 invalid
Something is wrong
 question
More information is needed
 wontfix
This won't be fixed
No Label
Milestone
No items
No Milestone
Projects
Clear projects
No project
Assignees
Clear assignees
hal9000 darthrubik
No Assignees
1 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: darthrubik/drharness#38
Delete Branch "bugfix/utf8"

Deleting a branch is permanent. Although the deleted branch may continue to exist for a short time before it actually gets removed, it CANNOT be undone in most cases. Continue?