-
Notifications
You must be signed in to change notification settings - Fork 2
Should Result and Option be enums? #64
Copy link
Copy link
Open
Labels
help wantedExtra attention is neededExtra attention is neededneed discussion/reflexionYet to decide architectural-level changesYet to decide architectural-level changesperformanceperformance-improvement enhancementperformance-improvement enhancement
Description
Metadata
Metadata
Assignees
Labels
help wantedExtra attention is neededExtra attention is neededneed discussion/reflexionYet to decide architectural-level changesYet to decide architectural-level changesperformanceperformance-improvement enhancementperformance-improvement enhancement
Ironically, we implement those types as separate structs instead of enums. This is legacy from original python implementation where it was much faster, but is this still the case in rust?
Hopefully we can migrate to enums, as this would very likely make Iterators who use those constructs much faster, and the surrounding code much more readable.