M-Labs
/
nac3
8
0
Fork 5
Code Issues 87 Pull Requests 3 Releases Wiki Activity

Simplify get_builtins to only return top-level built-in declarations #362

Merged
sb10q merged 1 commits from enhance/infer-builtins-list into master 2023-12-08 17:29:55 +08:00
Conversation 0 Commits 1 Files Changed 5
derppening commented 2023-12-08 15:56:07 +08:00
Collaborator

The top-level builtin declaration list is used at multiple places, but the names are only used at one location (TopLevelComposer::new). This change also has the effect of allowing Option to be used as a class name, as previously Option is not registered in the TopLevelComposer due to how the names of builtins are processed.

The top-level builtin declaration list is used at multiple places, but the names are only used at one location (`TopLevelComposer::new`). This change also has the effect of allowing `Option` to be used as a class name, as previously `Option` is not registered in the `TopLevelComposer` due to how the names of builtins are processed.
derppening added 1 commit 2023-12-08 15:56:08 +08:00
sb10q merged commit 68b97347b1 into master 2023-12-08 17:29:53 +08:00
sb10q deleted branch enhance/infer-builtins-list 2023-12-08 17:30:06 +08:00
Sign in to join this conversation.
Reviewers
No reviewers
Labels
Clear labels   
UROP0622
   
future-extension
   
high-priority
   
low-priority
No Label
UROP0622
future-extension
high-priority
low-priority
Milestone
Clear milestone
No items
No Milestone
Assignees
Clear assignees
No Assignees
1 Participants
Notifications
Due Date
The due date is invalid or out of range. Please use the format 'yyyy-mm-dd'.

No due date set.

Dependencies

No dependencies set.

Reference: M-Labs/nac3#362
There is no content yet.
Delete Branch "enhance/infer-builtins-list"

Deleting a branch is permanent. Although the deleted branch may exist for a short time before cleaning up, in most cases it CANNOT be undone. Continue?