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
Fix Insert (Alternative Method) #4922
Conversation
I prefer the method of #4920, but the modified grammar will cause test failures. This is a workaround. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This looks good!
What do you prefer about the other version? (Using AtomicExpr
doesn't seem like the right thing)
This version may be better if AtomicExpr is not the right thing. Shall I go ahead and merge? |
Go for it |
Changed inserts to take a single object reference.