-
Notifications
You must be signed in to change notification settings - Fork 1.9k
Closed
Labels
E-mediumS-actionableSomeone could pick this issue up and work on it right nowSomeone could pick this issue up and work on it right nowgood first issue
Description
We currently have ast::String for "" strings and ast::RawString for r"". We don't have a type to represent either of those.
Steps:
- Find out places in grammar where we need this. One is
extern "ABI". Are there anything else? - If the answer is no, we should change the grammar of ABI to contain
ast::Literal(like we do for attributes). - If the answer is yes, we should consider adding a StringOrRawString type, with a better name, and maybe re-design literal to account for that (and then we should also introduce
ByteStringOrRawByteString). - Finally, for all places that can have strings (extern ABI), we should add the requisite accessor.
But lets not overthink this and figure out 1 first.
Metadata
Metadata
Assignees
Labels
E-mediumS-actionableSomeone could pick this issue up and work on it right nowSomeone could pick this issue up and work on it right nowgood first issue