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

IIncrementExpression exposes both UnaryOperatorKind and BinaryOperatorKind #18056

Closed
mavasani opened this issue Mar 22, 2017 · 5 comments
Closed
Assignees
Labels
Area-Analyzers Bug Concept-API This issue involves adding, removing, clarification, or modification of an API. Feature - IOperation IOperation Urgency-Soon
Milestone

Comments

@mavasani
Copy link
Contributor

mavasani commented Mar 22, 2017

IIncrementExpression exposes UnaryOperatorKind but it's base type ICompoundAssignmentExpression expose BinaryOperatorKind.

Tagging @dotnet/analyzer-ioperation and @heejaechang who is working on unary operator tests.

@mavasani mavasani added Bug Concept-API This issue involves adding, removing, clarification, or modification of an API. Feature - IOperation IOperation labels Mar 22, 2017
@jinujoseph
Copy link
Contributor

jinujoseph commented Apr 18, 2017

Design Team Notes

We need to simplify the hierarchy, The intend here is we are not going to have the complex inheritance hierarchy where in the same hierarchy there is lot of base. We should try to make it more flat and narrow. IExpression might have Iincrementexpression, IcompondAssignment , IAssignementExpression and those themselves can implement side hierarchy, which can be call as infos which are not Ioperation Kind , Say IAssignmentInfos and some one can uniformally operate on this things.

Also look at #8834,#8835 while fixing this as well.

@jinujoseph jinujoseph modified the milestones: 15.3, 15.later Apr 18, 2017
@jinujoseph jinujoseph modified the milestones: 15.6, 15.later May 25, 2017
@mavasani
Copy link
Contributor Author

@heejaechang Did we check in unit tests for unary operations?

@jinujoseph
Copy link
Contributor

Nope, it's still pending #18136

@heejaechang
Copy link
Contributor

@mavasani ha ha not yet!

@jinujoseph
Copy link
Contributor

@heejaechang , you can close this issue onces the test is checked in.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
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
Development

No branches or pull requests

4 participants