Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Removed hard limit on module initialization #773

Open
nickbattle opened this issue Jan 16, 2021 · 2 comments
Open

Removed hard limit on module initialization #773

nickbattle opened this issue Jan 16, 2021 · 2 comments
Assignees
@nickbattle
Labels
enhancement Not a bug, but nice to have language Issues in parser, TC, interpreter, POG or CG Mergable A fix is available on a branch to merge for release

Comments

@nickbattle
Copy link
Contributor

This is an issue to reflect the changes, with some amendments, in Simon Fraser's PR #772. I will merge this into ncb/development, which will subsequently be picked up by development and master in the next release.
@nickbattle nickbattle added enhancement Not a bug, but nice to have language Issues in parser, TC, interpreter, POG or CG labels Jan 16, 2021
@nickbattle nickbattle self-assigned this Jan 16, 2021
@nickbattle
Copy link
Contributor Author

Simon's patch was applied, with a small change to force a small number of retries even if the error number does not decrease. This is to protect against the case where N1 errors are produced at one pass, and N2 on the next pass and N1=N2 even though the errors are different. A similar change was also made to class initialization.

@nickbattle nickbattle added the Mergable A fix is available on a branch to merge for release label Jan 16, 2021
@nickbattle
Copy link
Contributor Author

Change now available in ncb/development.

@nickbattle nickbattle mentioned this issue Jan 16, 2021
Closed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@nickbattle nickbattle

Labels
enhancement Not a bug, but nice to have language Issues in parser, TC, interpreter, POG or CG Mergable A fix is available on a branch to merge for release
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
1 participant
@nickbattle