Skip to content

Require structs to be initialized before using - #1808

Merged
ubaidsk merged 6 commits into
lcompilers:mainfrom
ubaidsk:struct_init_expr
May 14, 2023
Merged

Require structs to be initialized before using#1808
ubaidsk merged 6 commits into
lcompilers:mainfrom
ubaidsk:struct_init_expr

Conversation

@ubaidsk

Copy link
Copy Markdown
Collaborator

fixes#1788.

Comment on lines +1399 to +1400
numpy:
(Module

Copy link
Copy Markdown
CollaboratorAuthor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

In this test, I imported empty from numpy, hence the reference output increased significantly.

@ubaidsk

Copy link
Copy Markdown
CollaboratorAuthor

Ready.

@ubaidsk
ubaidsk requested a review from certikMay 13, 2023 20:16

@certikcertik left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks great, thanks!

@ubaidsk
ubaidsk merged commit 9941f2c into lcompilers:mainMay 14, 2023
@ubaidsk
ubaidsk deleted the struct_init_expr branch May 14, 2023 01:49
Sign up for freeto join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Require structs to be initialized before using

2 participants

@ubaidsk@certik