forked from fl00r/go-tarantool-1.6
-
Notifications
You must be signed in to change notification settings - Fork 59
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
[MP_EXT] Extension encoder and decoder functions are not exportable #420
Comments
rybakit
changed the title
[MP_EXT] Extension encoders and decoders are not exportable
[MP_EXT] Extension encoder and decoder functions are not exportable
Dec 16, 2024
rybakit
added a commit
to rybakit/go-tarantool
that referenced
this issue
Dec 16, 2024
rybakit
added a commit
to rybakit/go-tarantool
that referenced
this issue
Dec 16, 2024
rybakit
added a commit
to rybakit/go-tarantool
that referenced
this issue
Dec 16, 2024
2 tasks
rybakit
added a commit
to rybakit/go-tarantool
that referenced
this issue
Dec 16, 2024
rybakit
added a commit
to rybakit/go-tarantool
that referenced
this issue
Dec 16, 2024
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Consider making the current MessagePack extension encoder and decoder functions exportable. I’m personally interested in the decimal extension, where I would like to register my own extension but still be able to reuse the encoding logic from the tarantool/decimal package. The easiest way is to simply make the
decimalEncoder()
anddecimalDecoder()
functions start with a capital letter. Another option is to make them adhere to theMarshalerUnmarshaler
interface from vmihailenco/msgpack/v5 and rename them toMarshalMsgpack()
andUnmarshalMsgpack()
, respectively.The text was updated successfully, but these errors were encountered: