Skip to content
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

Open
Tibo-lg opened this issue Jan 7, 2022 · 7 comments
Open

Type of base in digit decomposition descriptor #183

Tibo-lg opened this issue Jan 7, 2022 · 7 comments
Labels

Comments

@Tibo-lg
Copy link
Member

Tibo-lg commented Jan 7, 2022

I'm wondering if there is a reason for base to be a bigsize in digit decomposition descriptor. I think it's fine not to hardcode it to be 2 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 a u8 or u16 maybe?

@Tibo-lg Tibo-lg added the oracle label Jan 7, 2022
@benthecarman
Copy link
Contributor

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.

@Tibo-lg
Copy link
Member Author

Tibo-lg commented Jan 8, 2022

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.

@benthecarman
Copy link
Contributor

Yeah u8 would be better IMHO because we would get bound check for free.

+1

@Christewart
Copy link
Contributor

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?

@Tibo-lg
Copy link
Member Author

Tibo-lg commented Jan 12, 2022

This one change in particular is on oracle message so I guess it should maybbe go in #167

@benthecarman
Copy link
Contributor

Oracle announcements go in a contract info so i think it should be included in #163

@Tibo-lg
Copy link
Member Author

Tibo-lg commented Jan 12, 2022

For now we agreed not to put oracle changes in #163 to avoid breaking all oracle infrastructure before #167 (so all oracle messages are unchanged in #163 for now)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

3 participants