Skip to content

Fix packed string encoding#153

Merged
cswinter merged 3 commits intomasterfrom
string-packer-fix
Jul 3, 2022
Merged

Fix packed string encoding#153
cswinter merged 3 commits intomasterfrom
string-packer-fix

Conversation

@cswinter
Copy link
Copy Markdown
Owner

@cswinter cswinter commented Jul 3, 2022

PackedStrings was encoding strings with a null-terminator, which breaks for strings that contain null bytes. Prefix each string with a length header instead.

Resolves #152

@cswinter cswinter mentioned this pull request Jul 3, 2022
@cswinter cswinter merged commit af55fcf into master Jul 3, 2022
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.

It does not work at all.

1 participant