You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
Amino allows to encode int for convenience and while the underlying varint encoding doesn't waste bytes on the wire it remains unclear how this translates to the existing proto3 types (int32, int64). Disallowing int would resolve this and probably is not a bad idea anyways (as int is platform dependant). The SDK and the tendermint team seem onboard with doing the necessary changes.
Alternatively, always assume (proto) int64 when int is used on the go side (and err on 32 b it platforms).
The text was updated successfully, but these errors were encountered:
Amino allows to encode
int
for convenience and while the underlying varint encoding doesn't waste bytes on the wire it remains unclear how this translates to the existing proto3 types (int32, int64). Disallowingint
would resolve this and probably is not a bad idea anyways (asint
is platform dependant). The SDK and the tendermint team seem onboard with doing the necessary changes.Alternatively, always assume (proto)
int64
whenint
is used on the go side (and err on 32 b it platforms).The text was updated successfully, but these errors were encountered: