Skip to content
This repository has been archived by the owner on Sep 27, 2023. It is now read-only.

Fix deferred name resolution #947

Merged
merged 2 commits into from
Jul 16, 2021
Merged

Conversation

dchassin
Copy link
Member

@dchassin dchassin commented Jul 14, 2021

This PR fixes issue #946

Current issues

None

Code changes

  • Add support for deferred object resolution in gldcore/load.{h,cpp}
  • Add call to resolve deferred object names in gldcore/main.cpp

Documentation changes

None

Test and Validation Notes

  • Add gldcore/autotest/test_deferred_names.glm and supporting files gldcore/autotest/deferred[12].glm

@dchassin dchassin requested a review from aivanova5 July 14, 2021 18:57
@dchassin dchassin added the bug Request a fix or change to the code label Jul 14, 2021
@dchassin dchassin self-assigned this Jul 14, 2021
@dchassin dchassin changed the base branch from master to develop July 14, 2021 19:04
@dchassin dchassin mentioned this pull request Jul 14, 2021
3 tasks
@dchassin dchassin changed the title Develop fix deferred name resolution Fix deferred name resolution Jul 14, 2021
@aivanova5 aivanova5 merged commit 8f627d7 into develop Jul 16, 2021
@aivanova5 aivanova5 deleted the develop-fix-deferred-name-resolution branch July 16, 2021 15:43
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
bug Request a fix or change to the code
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Deferred name resolution doesn't work across multiple command line GLM files
2 participants