-
-
Notifications
You must be signed in to change notification settings - Fork 9
Convert op_id and party_id to enums #33
Copy link
Copy link
Open
Labels
Type: Improvement 📈Performance improvement not introducing a new feature or requiring a major refactorPerformance improvement not introducing a new feature or requiring a major refactor
Metadata
Metadata
Assignees
Labels
Type: Improvement 📈Performance improvement not introducing a new feature or requiring a major refactorPerformance improvement not introducing a new feature or requiring a major refactor
Type
Fields
Give feedbackNo fields configured for issues without a type.
Description
Currently, we're working with
op_idandparty_idas ausizevariables, and then we use these variables to "match" values, for example, like the following:sycret/src/lib.rs
Line 14 in fc776e1
I think we could maybe use an enum instead. For example, in the case of
op_id:This would be more idiomatic for Rust and could help us prevent errors my limiting options to the enum variants.
Are you interested in working on this improvement yourself?
Yes.