near / borsh-rs

Rust implementation of Binary Object Representation Serializer for Hashing
https://borsh.io/
Apache License 2.0
299 stars 65 forks source link

add implementation for `BorshSerialize`, `BorshDeserialize`, `BorshSchema` for `char` #247

Closed dj8yfo closed 10 months ago

frol commented 10 months ago

I believe it was a deliberate choice to not implement it for char as it is not language-agnostic. What is your motivation to add it? Is there some use-case that drives this feature request?

dj8yfo commented 10 months ago

Primarily the pr was driven by one last so far commented block in borsh_schema tests of cargo near abi compared to json_schema tests of cargo near abi

dj8yfo commented 10 months ago

Closed as not needed atm