-
Notifications
You must be signed in to change notification settings - Fork 978
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
Use deparse() to cast calls to string when needed #6027
Conversation
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## master #6027 +/- ##
=======================================
Coverage 97.53% 97.53%
=======================================
Files 80 80
Lines 14915 14917 +2
=======================================
+ Hits 14547 14549 +2
Misses 368 368 ☔ View full report in Codecov by Sentry. |
Generated via commit 5ba6112 Download link for the artifact containing the test results: ↓ atime-results.zip Time taken to finish the standard R installation steps: 11 minutes and 30 seconds Time taken to run |
@jangorecki PTAL, straightforward enough. I do worry about deeper edgecases, would be good to put this through revdeps. |
I would go back to the issue and examine if what the problem exactly is and possible solutions. |
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.
tested and looks good! Can we tweak news entry to not encourage users for this sub-efficient usage?
Closes #6026.
There is some small overhead added here since
jsub %iscall% f
callsis.call()
redundantly a few times.is.call()
is ~50ns to run so I think this can be ignored.