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.

Sign up for GitHub

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

Jump to bottom

IOperation API for nameof expression #19954

Closed
mavasani opened this issue Jun 1, 2017 · 2 comments
Closed

IOperation API for nameof expression #19954

mavasani opened this issue Jun 1, 2017 · 2 comments
Assignees
@mavasani
Labels
Area-Analyzers Bug Concept-API This issue involves adding, removing, clarification, or modification of an API. Feature - IOperation IOperation Urgency-Soon
Milestone
15.5

Comments

@mavasani
Copy link
Contributor

mavasani commented Jun 1, 2017

Proposed API:

    public interface INameOfExpression : IOperation
    {
        IOperation Argument { get; }
    }
@mavasani mavasani added Area-Analyzers Bug Concept-API This issue involves adding, removing, clarification, or modification of an API. Discussion Feature - IOperation IOperation labels Jun 1, 2017
@mavasani mavasani added this to the 15.later milestone Jun 1, 2017
@mavasani mavasani changed the title IOperation support for nameof expression IOperation API for nameof expression Jun 1, 2017
@jinujoseph jinujoseph mentioned this issue Jun 1, 2017
Closed
48 tasks
@CyrusNajmabadi
Copy link
Member

CyrusNajmabadi commented Jun 1, 2017 •
edited by jinujoseph
Loading

Design Team Decision
We should do this.

@CyrusNajmabadi CyrusNajmabadi mentioned this issue Jun 1, 2017
Open
@jinujoseph jinujoseph modified the milestones: 15.6, 15.later Jun 4, 2017
@mavasani mavasani mentioned this issue Jul 7, 2017
Closed
mavasani added a commit to mavasani/roslyn that referenced this issue Jul 8, 2017
@mavasani
Add IOperation APIs for nameof and throw expressions
def58aa 
Fixes dotnet#19954 and dotnet#20125
@mavasani mavasani mentioned this issue Jul 8, 2017
Merged
@mavasani
Copy link
Contributor Author

Fixed with #20722

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@mavasani mavasani

Labels
Area-Analyzers Bug Concept-API This issue involves adding, removing, clarification, or modification of an API. Feature - IOperation IOperation Urgency-Soon
Projects
None yet
Milestone
15.5
Development

No branches or pull requests
3 participants
@CyrusNajmabadi @mavasani @jinujoseph