Skip to content
Snippets Groups Projects

Explicit visibility for Instances

Merged Ralf Jung requested to merge ralf/instances into master
All threads resolved!

Our existing check for this missed some cases:

  • Existing Instance
  • Program Instance
  • Instance: (for anonymous instances)

Those are all detected now, so fix all the code accordingly.

Merge request reports

Loading
Loading

Activity

Filter activity
  • Approvals
  • Assignees & reviewers
  • Comments (from bots)
  • Comments (from users)
  • Commits & branches
  • Edits
  • Labels
  • Lock status
  • Mentions
  • Merge request status
  • Tracking
  • Ralf Jung
  • Ralf Jung added 1 commit

    added 1 commit

    • c1248edd - also make sure Opaque/Transparent have visibility qualifiers

    Compare with previous version

  • Ralf Jung
  • Ralf Jung added 1 commit

    added 1 commit

    Compare with previous version

  • Ralf Jung mentioned in merge request iris!684 (merged)

    mentioned in merge request iris!684 (merged)

  • Ralf Jung added 1 commit

    added 1 commit

    Compare with previous version

  • mentioned in commit cb1daa49

  • Ralf Jung added 9 commits

    added 9 commits

    • 03507275...cb1daa49 - 3 commits from branch master
    • d0183a61 - fix warnings about implicitly Global Instance
    • 6c010757 - make local instances explicit
    • 56001965 - also make sure Opaque/Transparent have visibility qualifiers
    • 531fe4dd - show line numbers in errors
    • eb035887 - improve error message
    • 89ebe23f - Local Instance → Definition

    Compare with previous version

  • Ralf Jung resolved all threads

    resolved all threads

  • Forgot to say, modulo my comments LGTM!

  • Ralf Jung enabled an automatic merge when the pipeline for 89ebe23f succeeds

    enabled an automatic merge when the pipeline for 89ebe23f succeeds

  • Ralf Jung mentioned in commit 54d7f380

    mentioned in commit 54d7f380

  • merged

  • Please register or sign in to reply
    Loading