Skip to content
This repository has been archived by the owner on Jul 1, 2024. It is now read-only.

[DO NOT MERGE] test unnecessary clang tidy #253

Closed
wants to merge 1 commit into from

Conversation

takayuki5168
Copy link
Contributor

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.

  • There are no open discussions or they are tracked via tickets.

After all checkboxes are checked, anyone who has write access can merge the PR.

Signed-off-by: Takayuki Murooka <[email protected]>
Copy link

@github-actions github-actions bot left a 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)

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

⚠️ google-explicit-constructor ⚠️
constructors that are callable with a single argument must be marked explicit to avoid unintentional implicit conversions

Suggested change
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)

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

⚠️ google-explicit-constructor ⚠️
single-argument constructors must be marked explicit to avoid unintentional implicit conversions

Suggested change
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)

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

⚠️ google-explicit-constructor ⚠️
constructors that are callable with a single argument must be marked explicit to avoid unintentional implicit conversions

Suggested change
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)

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

⚠️ google-explicit-constructor ⚠️
single-argument constructors must be marked explicit to avoid unintentional implicit conversions

Suggested change
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)

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

⚠️ google-explicit-constructor ⚠️
constructors that are callable with a single argument must be marked explicit to avoid unintentional implicit conversions

Suggested change
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)

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

⚠️ google-explicit-constructor ⚠️
single-argument constructors must be marked explicit to avoid unintentional implicit conversions

Suggested change
BOOST_PYTHON_FUNCTION_OVERLOADS(isInLanelet_overload, ::isInLanelet, 2, 3)
BOOST_PYTHON_FUNCTION_OVERLOADS(explicit isInLanelet_overload, ::isInLanelet, 2, 3)

@takayuki5168 takayuki5168 deleted the tmp/test-unnecessary-clang-tidy branch June 16, 2024 14:13
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant