M-Labs/nac3
M-Labs
/
nac3
8
0
Fork 10
Code Issues 99 Pull Requests 21 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 +62 -82
derppening commented 2023-12-08 15:56:07 +08:00
Collaborator
Copy Link

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   
future-extension

   
high-priority

   
low-priority

   
UROP0622
No Label
future-extension
high-priority
low-priority
UROP0622
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
No description provided.
Delete Branch "enhance/infer-builtins-list"

Deleting a branch is permanent. Although the deleted branch may continue to exist for a short time before it actually gets removed, it CANNOT be undone in most cases. Continue?