Skip to content

VB -> C#: Don't generate unnecessary properties for WithEvents field #572

Closed
@GrahamTheCoder

Description

Is your feature request related to a problem? Please describe.
In order to convert a WithEvents field, the converter turns it into a property with a backing field (as the VB compiler also does).
If the field is only internally visible and has no Handles methods, this is redundant, and non-idiomatic C#.

Describe the solution you'd like
Use the field directly in the case described.

Describe alternatives you've considered
Leave it as it is now for consistency and simplicity of implementation

Additional context
It surprised someone in its current state, which is usually a bad sign:
#569 (comment)

Metadata

Assignees

No one assigned

    Labels

    Type

    No type

    Projects

    No projects

    Milestone

    No milestone

    Relationships

    None yet

    Development

    No branches or pull requests

    Issue actions