Skip to content
GitLab
Projects Groups Snippets
  • /
  • Help
    • Help
    • Support
    • Community forum
    • Submit feedback
    • Contribute to GitLab
  • Sign in
  • asm asm
  • Project information
    • Project information
    • Activity
    • Labels
    • Members
  • Repository
    • Repository
    • Files
    • Commits
    • Branches
    • Tags
    • Contributors
    • Graph
    • Compare
  • Issues 2
    • Issues 2
    • List
    • Boards
    • Service Desk
    • Milestones
  • Merge requests 0
    • Merge requests 0
  • CI/CD
    • CI/CD
    • Pipelines
    • Jobs
    • Schedules
  • Deployments
    • Deployments
    • Environments
    • Releases
  • Packages and registries
    • Packages and registries
    • Container Registry
  • Monitor
    • Monitor
    • Incidents
  • Analytics
    • Analytics
    • Value stream
    • CI/CD
    • Repository
  • Wiki
    • Wiki
  • Snippets
    • Snippets
  • Activity
  • Graph
  • Create a new issue
  • Jobs
  • Commits
  • Issue Boards
Collapse sidebar
  • asmasm
  • asmasm
  • Issues
  • #317981
Closed
Open
Issue created Aug 17, 2022 by raphw@raphw

Add public getDelegate method to all visitor classes.

Currently, only the RecordComponentVisitor offers a public method for reading the underlying visitor. Sometimes, it is however desirable to communicate with a visitor that lies beneath another visitor also for the other visitor implementations. Could an equivalent getter be added for the other visitors, too?

Also, would you consider to make the delegate field for the record component visitor protected as with the other visitors? This can be handy when a subclass of a visitor wants to change its delegate after visiting some properties, for example after all attributes were passed. Ideally, ASM would be consistent here.

Assignee
Assign to
Time tracking