-
Notifications
You must be signed in to change notification settings - Fork 72
[DO NOT MERGE] test unnecessary clang tidy #253
Conversation
Signed-off-by: Takayuki Murooka <[email protected]>
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.
Clang-Tidy
found issue(s) with the introduced code (1/1)
@@ -377,7 +377,7 @@ BOOST_PYTHON_FUNCTION_OVERLOADS( | |||
BOOST_PYTHON_FUNCTION_OVERLOADS( | |||
getLeftBoundWithOffset_overload, lanelet::utils::getLeftBoundWithOffset, 2, 3) |
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.
constructors that are callable with a single argument must be marked explicit to avoid unintentional implicit conversions
getLeftBoundWithOffset_overload, lanelet::utils::getLeftBoundWithOffset, 2, 3) | |
explicit getLeftBoundWithOffset_overload, lanelet::utils::getLeftBoundWithOffset, 2, 3) |
@@ -377,7 +377,7 @@ | |||
BOOST_PYTHON_FUNCTION_OVERLOADS( | |||
getLeftBoundWithOffset_overload, lanelet::utils::getLeftBoundWithOffset, 2, 3) |
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.
single-argument constructors must be marked explicit to avoid unintentional implicit conversions
getLeftBoundWithOffset_overload, lanelet::utils::getLeftBoundWithOffset, 2, 3) | |
explicit getLeftBoundWithOffset_overload, lanelet::utils::getLeftBoundWithOffset, 2, 3) |
@@ -377,7 +377,7 @@ | |||
BOOST_PYTHON_FUNCTION_OVERLOADS( | |||
getLeftBoundWithOffset_overload, lanelet::utils::getLeftBoundWithOffset, 2, 3) | |||
BOOST_PYTHON_FUNCTION_OVERLOADS( | |||
overwriteLaneletsCenterline_overload, lanelet::utils::overwriteLaneletsCenterline, 1, 3) | |||
overwriteLaneletsCenterline_overload, lanelet::utils::overwriteLaneletsCenterline, 2, 3) |
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.
constructors that are callable with a single argument must be marked explicit to avoid unintentional implicit conversions
overwriteLaneletsCenterline_overload, lanelet::utils::overwriteLaneletsCenterline, 2, 3) | |
explicit overwriteLaneletsCenterline_overload, lanelet::utils::overwriteLaneletsCenterline, 2, 3) |
@@ -377,7 +377,7 @@ | |||
BOOST_PYTHON_FUNCTION_OVERLOADS( | |||
getLeftBoundWithOffset_overload, lanelet::utils::getLeftBoundWithOffset, 2, 3) | |||
BOOST_PYTHON_FUNCTION_OVERLOADS( | |||
overwriteLaneletsCenterline_overload, lanelet::utils::overwriteLaneletsCenterline, 1, 3) | |||
overwriteLaneletsCenterline_overload, lanelet::utils::overwriteLaneletsCenterline, 2, 3) |
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.
single-argument constructors must be marked explicit to avoid unintentional implicit conversions
overwriteLaneletsCenterline_overload, lanelet::utils::overwriteLaneletsCenterline, 2, 3) | |
explicit overwriteLaneletsCenterline_overload, lanelet::utils::overwriteLaneletsCenterline, 2, 3) |
@@ -377,7 +377,7 @@ | |||
BOOST_PYTHON_FUNCTION_OVERLOADS( | |||
getLeftBoundWithOffset_overload, lanelet::utils::getLeftBoundWithOffset, 2, 3) | |||
BOOST_PYTHON_FUNCTION_OVERLOADS( | |||
overwriteLaneletsCenterline_overload, lanelet::utils::overwriteLaneletsCenterline, 1, 3) | |||
overwriteLaneletsCenterline_overload, lanelet::utils::overwriteLaneletsCenterline, 2, 3) | |||
BOOST_PYTHON_FUNCTION_OVERLOADS(isInLanelet_overload, ::isInLanelet, 2, 3) |
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.
constructors that are callable with a single argument must be marked explicit to avoid unintentional implicit conversions
BOOST_PYTHON_FUNCTION_OVERLOADS(isInLanelet_overload, ::isInLanelet, 2, 3) | |
BOOST_PYTHON_FUNCTION_OVERLOADS(explicit isInLanelet_overload, ::isInLanelet, 2, 3) |
@@ -377,7 +377,7 @@ | |||
BOOST_PYTHON_FUNCTION_OVERLOADS( | |||
getLeftBoundWithOffset_overload, lanelet::utils::getLeftBoundWithOffset, 2, 3) | |||
BOOST_PYTHON_FUNCTION_OVERLOADS( | |||
overwriteLaneletsCenterline_overload, lanelet::utils::overwriteLaneletsCenterline, 1, 3) | |||
overwriteLaneletsCenterline_overload, lanelet::utils::overwriteLaneletsCenterline, 2, 3) | |||
BOOST_PYTHON_FUNCTION_OVERLOADS(isInLanelet_overload, ::isInLanelet, 2, 3) |
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.
single-argument constructors must be marked explicit to avoid unintentional implicit conversions
BOOST_PYTHON_FUNCTION_OVERLOADS(isInLanelet_overload, ::isInLanelet, 2, 3) | |
BOOST_PYTHON_FUNCTION_OVERLOADS(explicit isInLanelet_overload, ::isInLanelet, 2, 3) |
Description
Tests performed
Not applicable.
Effects on system behavior
Not applicable.
Pre-review checklist for the PR author
The PR author must check the checkboxes below when creating the PR.
In-review checklist for the PR reviewers
The PR reviewers must check the checkboxes below before approval.
Post-review checklist for the PR author
The PR author must check the checkboxes below before merging.
After all checkboxes are checked, anyone who has write access can merge the PR.