Skip to content

fixed bugs#309

Open
freeliver wants to merge 2 commits into
vmihailenco:v5from
freeliver:v5_fixed_bugs
Open

fixed bugs#309
freeliver wants to merge 2 commits into
vmihailenco:v5from
freeliver:v5_fixed_bugs

Conversation

@freeliver

Copy link
Copy Markdown

fixed when use msgpack.rawmessage panic reflect: reflect.Value.Set using unaddressable value

@ghost

ghost commented Jul 1, 2021

Copy link
Copy Markdown

@freeliver Would you mind making the necessary adjustments to your code so that we can accept your pull request? See https://travis-ci.org/github/vmihailenco/msgpack/jobs/773364119#L292 and following for details.

Let's get rid of that nasty #310 =)

@vmihailenco

Copy link
Copy Markdown
Owner

Sorry for the 2 years delay, but this change requires a test :)

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