Skip to content

Commit

Permalink
Pass py::module_ by reference to resolve clang-tidy errors (this is e…
Browse files Browse the repository at this point in the history
…ntirely inconsequential otherwise for all practical purposes).
  • Loading branch information
rwgk committed Mar 9, 2025
1 parent bab3348 commit c22104f
Showing 1 changed file with 5 additions and 5 deletions.
10 changes: 5 additions & 5 deletions tests/test_native_enum.cpp
Original file line number Diff line number Diff line change
Expand Up @@ -150,7 +150,7 @@ TEST_SUBMODULE(native_enum, m) {
.value(malformed_utf8, fake::x);
});

m.def("double_registration_native_enum", [](py::module_ m) {
m.def("double_registration_native_enum", [](py::module_ &m) {
enum fake { x };
py::native_enum<fake>(
m, "fake_double_registration_native_enum", py::native_enum_kind::IntEnum)
Expand All @@ -160,14 +160,14 @@ TEST_SUBMODULE(native_enum, m) {
m, "fake_double_registration_native_enum", py::native_enum_kind::Enum);
});

m.def("native_enum_name_clash", [](py::module_ m) {
m.def("native_enum_name_clash", [](py::module_ &m) {
enum fake { x };
py::native_enum<fake>(m, "fake_native_enum_name_clash", py::native_enum_kind::IntEnum)
.value("x", fake::x)
.finalize();
});

m.def("native_enum_value_name_clash", [](py::module_ m) {
m.def("native_enum_value_name_clash", [](py::module_ &m) {
enum fake { x };
py::native_enum<fake>(
m, "fake_native_enum_value_name_clash", py::native_enum_kind::IntEnum)
Expand All @@ -176,15 +176,15 @@ TEST_SUBMODULE(native_enum, m) {
.finalize();
});

m.def("double_registration_enum_before_native_enum", [](const py::module_ &m) {
m.def("double_registration_enum_before_native_enum", [](py::module_ &m) {
enum fake { x };
py::enum_<fake>(m, "fake_enum_first").value("x", fake::x);
py::native_enum<fake>(m, "fake_enum_first", py::native_enum_kind::IntEnum)
.value("x", fake::x)
.finalize();
});

m.def("double_registration_native_enum_before_enum", [](py::module_ m) {
m.def("double_registration_native_enum_before_enum", [](py::module_ &m) {
enum fake { x };
py::native_enum<fake>(m, "fake_native_enum_first", py::native_enum_kind::IntEnum)
.value("x", fake::x)
Expand Down

0 comments on commit c22104f

Please sign in to comment.