Use braces in tryCatch() calls
#1839
Conversation
Codecov Report
@@ Coverage Diff @@
## main #1839 +/- ##
=======================================
Coverage 98.85% 98.85%
=======================================
Files 112 112
Lines 4814 4815 +1
=======================================
+ Hits 4759 4760 +1
Misses 55 55
Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here. |
R/get_source_expressions.R
Outdated
| # This object should be named `e` because, if invalid characters are present, | ||
| # the error needs to be assigned in the parent frame, where it is named `e`. |
There was a problem hiding this comment.
For this reason, the content can't be assigned to parsed_content or some other, more informative name.
There was a problem hiding this comment.
isn't the parent's name given by the first argument to assign()?
so assign("e", parsed_content, ...) works?
There was a problem hiding this comment.
Ah, yes, true that.
Sorry, I was doing search and replace, which changed both e and "e", and that's why it wasn't working.
I have made the necessary change now and the object is called parsed_content.
Thanks!
Inspired by discussion in #1817.