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.

Sign up for GitHub

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

Jump to bottom

fix unmarshaling null into []byte #41

Merged

warpfork merged 2 commits into polydawn:master from Stebalien:fix/40

Nov 1, 2018

Contributor

Stebalien commented Nov 1, 2018

fixes #40

Stebalien added 2 commits

November 1, 2018 16:28


          fix unmarshaling null into []byte

c6752ca

fixes polydawn#40


          test: add a test for polydawn#40

6b34486

Stebalien commented

View reviewed changes

obj/unmarshalBuiltins.go

               				mach.rv.Index(i).SetUint(uint64(tok.Bytes[i]))
               			}
               			return true, nil
+              		case TNull:

Contributor Author

Stebalien Nov 1, 2018

Maybe not needed but this matches the behavior of [N]int.

warpfork merged commit 2700834 into polydawn:master

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment