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

[c#] identify assignments in JsonAst; add operatorCallNode #5278

Merged

xavierpinho merged 1 commit into master from xavierp/c#-identify-assignments+misc-refactor

Feb 3, 2025

Contributor

xavierpinho commented Feb 2, 2025

Identifies AssignmentExpr within BinaryExpr inside DotNetJsonAst. This is preparatory work for handling assignments whose LHS is a setter property, which no longer means a traditional assignment but rather a setter call.
Replaces createCallNodeForOperator by a more general AstNodeBuilder operatorCallNode that is guaranteed to respect the code/line/column primitives.


          [c#] identify assignments in JsonAst; add operatorCallNode

acc2dd3

xavierpinho added the c# label

xavierpinho requested review from max-leuthaeuser and DavidBakerEffendi

February 2, 2025 21:11

max-leuthaeuser approved these changes

View reviewed changes

xavierpinho merged commit 50465d2 into master

5 checks passed

xavierpinho deleted the xavierp/c#-identify-assignments+misc-refactor branch

February 3, 2025 08:43

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

Labels

c#