master #311

Closed
Aadityavardhan wants to merge 8 commits from (deleted):master into master
First-time contributor

WIP: Static Class Attributes

Enabled class attribute definition syntax in the class body (Imp: Type checking/expression folding needs to be done).
Modified init checker to ensure that static fields do not need to be initialised in init.

**WIP: Static Class Attributes** Enabled class attribute definition syntax in the class body (Imp: Type checking/expression folding needs to be done). Modified init checker to ensure that static fields do not need to be initialised in __init__.
Aadityavardhan added 5 commits 2022-08-21 12:35:05 +08:00
sb10q reviewed 2022-08-21 16:31:16 +08:00
.gitignore Outdated
@ -4,0 +6,4 @@
interpreted.log
nac3standalone/demo/module.o
nac3standalone/demo/demo
nac3standalone/demo/run.log
Owner

Remove

Remove
Owner

Please fix the file permission issues that pollute this patch.

Please fix the file permission issues that pollute this patch.
sb10q reviewed 2022-08-21 16:32:18 +08:00
@ -175,6 +177,8 @@ pub fn get_builtins(primitives: &mut (PrimitiveStore, Unifier)) -> BuiltinInfo {
type_vars: Default::default(),
fields: exception_fields,
methods: Default::default(),
// Make TopLevelDef::Class initlializer compatible
Owner

Typo
What does that mean anyway?

Typo What does that mean anyway?
Author
First-time contributor

To call out the newly introduced static_fields data member in the TopLevelDef::Class struct.

To call out the newly introduced static_fields data member in the TopLevelDef::Class struct.
sb10q reviewed 2022-08-21 16:32:40 +08:00
@ -514,3 +514,3 @@
fn unify_impl(&mut self, a: Type, b: Type, swapped: bool) -> Result<(), TypeError> {
use TypeEnum::*;
Owner

Remove

Remove
Owner

Please title this PR properly. "master" is definitely not a good title.

Please title this PR properly. "master" is definitely not a good title.
Aadityavardhan added 1 commit 2022-08-21 17:19:49 +08:00
fc8b69c3cd Updated .gitignore
Removed unnecessary filenames
Aadityavardhan added 2 commits 2022-08-21 18:24:11 +08:00
Aadityavardhan closed this pull request 2022-08-21 20:30:33 +08:00

Pull request closed

Sign in to join this conversation.
No reviewers
No Milestone
No Assignees
2 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#311
No description provided.