-
Notifications
You must be signed in to change notification settings - Fork 2
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
Feat/node insertion #17
base: master
Are you sure you want to change the base?
Changes from 1 commit
ba3f7b3
4b27f8f
fbe3676
bd9ea68
7db4be5
844d87d
e443905
File filter
Filter by extension
Conversations
Jump to
Diff view
Diff view
There are no files selected for viewing
Original file line number | Diff line number | Diff line change | ||||
---|---|---|---|---|---|---|
@@ -0,0 +1,21 @@ | ||||||
use crate::SmlStr; | ||||||
|
||||||
enum NixPrimitive { | ||||||
Bool(bool), | ||||||
Int(i64), | ||||||
Str(SmlStr), | ||||||
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more.
Suggested change
|
||||||
} | ||||||
|
||||||
trait NixValue {} | ||||||
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. why not model all possible values here (including rather opaque stuff like function calls)? Maybe it would be a good idea to put this into a separate crate. |
||||||
|
||||||
enum NixType { | ||||||
Integer, | ||||||
String, | ||||||
Function, | ||||||
List(Box<NixType>), | ||||||
AttrSet(Vec<Box<NixType>>), | ||||||
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. how is this |
||||||
} | ||||||
|
||||||
//enum NixStructure { | ||||||
//List(Vec<Box<dyn >>), | ||||||
//} |
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -0,0 +1 @@ | ||
pub mod hlir_types; |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.