-
Notifications
You must be signed in to change notification settings - Fork 35
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
Type of base in digit decomposition descriptor #183
Comments
I think it was a bigsize so it would only be 1 byte in most cases. However, doing a u8 would accomplish the same goals. |
Ah I see. Yeah u8 would be better IMHO because we would get bound check for free. It's an issue for several other fields actually. |
+1 |
if we are going to do this it should go into #163 and i think its a small enough change that we can sneak it in there? |
This one change in particular is on oracle message so I guess it should maybbe go in #167 |
Oracle announcements go in a contract info so i think it should be included in #163 |
I'm wondering if there is a reason for
base
to be abigsize
in digit decomposition descriptor. I think it's fine not to hardcode it to be2
to allow flexibility but it seems that allowing arbitrary size for it is not a good idea either. As we will update the oracle messages in the near future, would it make sense to change it to be au8
oru16
maybe?The text was updated successfully, but these errors were encountered: