-
Notifications
You must be signed in to change notification settings - Fork 53
Drop redundant try/catch exception handling #193
Copy link
Copy link
Open
Description
There are two functions under src/utils.cpp that still add a second layer of try/catch for exception handling (on top of what BEGIN_RCPP/END_RCPP does). I'm referring to
Lines 557 to 559 in 03f3efd
| } catch (std::exception& ex) { | |
| forward_exception_to_r(ex); | |
| } |
and
Lines 596 to 598 in 03f3efd
| } catch (std::exception& ex){ | |
| forward_exception_to_r(ex); | |
| } |
These try/catch blocks can be removed. The motivation is that I'm revising dependencies to try to phase out this old forward_exception_to_r function in the medium-term.
Reactions are currently unavailable
Metadata
Metadata
Assignees
Labels
No labels