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.

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

Fix selective import #290

Closed
wants to merge 7 commits into from
Closed

Fix selective import #290

wants to merge 7 commits into from

Conversation

jcp19
Copy link
Contributor

@jcp19 jcp19 commented May 23, 2021

NOT READY FOR REVIEW

@jcp19 jcp19 linked an issue Jun 3, 2021 that may be closed by this pull request
@jcp19
Copy link
Contributor Author

jcp19 commented Jun 11, 2021

Also, modify TypeInfoImpl.scala after merging the PR for the preserves keyword to include the nodes refered in preserves clauses.

private def addTransitiveDeps(r: SymbolTable.Regular): Unit = {
    def checkedNodesFromFuncSpec(spec: PFunctionSpec): Vector[PNode] =
      spec.pres ++ spec.preserves ++ spec.posts ++ (spec.pres ++ spec.preserves ++ spec.posts).flatMap(_.subNodes)
        // TODO: check why this causes node not in tree exception
        // (try { (spec.pres ++ spec.posts) flatMap allChildren } catch { case e => Seq.empty })

@jcp19 jcp19 closed this Oct 27, 2021
@jcp19 jcp19 deleted the fix-selective-import branch January 11, 2022 11:54
Sign up for free to 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.

Problems with selective importing
1 participant