-
Notifications
You must be signed in to change notification settings - Fork 67
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
Rename declarations (tuples, nil
, string
, nonzero_xx
and arrays)
#231
Comments
I do have draft commits for those changes so can send PRs soonish for that. There’s actually more naming changes that I’ve been wondering about though they are a bit philosophical, i.e. it depends what the purpose of borsh schema is. Is it supposed to represent how the Rust type looks or just describe the binary format. The name changes I have in mind are:
And this all leads to further questions about keeping field names when describing structs and variant names when describing enums. |
The logic for renaming |
nil
and arrays)nil
, string
and arrays)
taking following subtasks for resolution in scope of one pr
|
nil
, string
and arrays)nil
, string
, nonzero_xx
and arrays)
taking following subtask for resolution in scope of one pr:
|
taking following subtask for resolution in scope of one pr:
|
"nil"
->"()"
"Tuple<T0, T1, T2>"
->"(T0, T1, T2)"
string
->String
"Array<T0, N>"
->"[T0; N]"
nonzero_i8
->NonZeroI8
...The text was updated successfully, but these errors were encountered: