Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Fix ascii marshal, unmarshall #322

Merged
merged 2 commits into from
Nov 25, 2024

Conversation

illia-li
Copy link

@illia-li illia-li commented Oct 27, 2024

Changes:

  1. Unmarshalling zero data into []byte, *[]byte, returned ([]byte)(nil) and (*[]byte)(*[nil]) before, now returns ([]byte)([]) and (*[]byte)(*[])
  2. Unmarshalling into *custom interface{}, was unsupported before, now is supported.
  3. Unmarshalling data with bytes out of the range 0-127, does not return an error before, now return an error.

marshal.go Outdated Show resolved Hide resolved
@illia-li illia-li force-pushed the il/fix/marshal/ascii branch from 6d386e7 to 8e26db1 Compare October 29, 2024 15:21
dkropachev
dkropachev previously approved these changes Oct 30, 2024
serialization/ascii/unmarshal_utils.go Outdated Show resolved Hide resolved
dkropachev
dkropachev previously approved these changes Nov 7, 2024
@dkropachev
Copy link
Collaborator

@illia-li , can you please rebase it

@illia-li illia-li force-pushed the il/fix/marshal/ascii branch 2 times, most recently from 0d759cc to ca3e918 Compare November 14, 2024 13:19
debounce/simple_debouncer_test.go Outdated Show resolved Hide resolved
serialization/ascii/marshal.go Show resolved Hide resolved
serialization/ascii/unmarshal_utils.go Outdated Show resolved Hide resolved
@illia-li illia-li force-pushed the il/fix/marshal/ascii branch from ca3e918 to 88e8914 Compare November 15, 2024 19:43
@illia-li illia-li requested a review from dkropachev November 15, 2024 22:54
dkropachev
dkropachev previously approved these changes Nov 21, 2024
serialization/ascii/unmarshal_utils.go Outdated Show resolved Hide resolved
@dkropachev dkropachev merged commit 8b3548e into scylladb:master Nov 25, 2024
2 checks passed
@illia-li illia-li deleted the il/fix/marshal/ascii branch November 25, 2024 20:09
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.

3 participants