Correct class names for KeysView, ValuesView and ItemsView in bind_map - #4353

Merged

rwgk merged 20 commits into

pybind:masterfrom

aimir:map-view-types

Dec 9, 2022

Commits on Dec 9, 2022

Create templated abstract classes KeysView, ValuesView and ItemsView, and implement them on-the-fly when wrapping any specific map type

Amir
authored and
rwgk
committed
db09d94
We don't want to wrap different ValuesView objects for double values and const double, for example, as both wrappers will be named ValuesView[float]

Amir
authored and
rwgk
committed
6bbde12
Fallback to C++ names if key or values types are not wrapped

Amir
authored and
rwgk
committed
f405e11
Added a test for .keys(), .values() and .items() returning the same types for similarly-typed maps

Amir
authored and
rwgk
committed
72a645e
Fixed wrong use of auto in a declarator list: the two descriptions might have different types

Amir
authored and
rwgk
committed
3e1b653
Fixes for clang-tidy issues: explicit single-argument constructor, using the 'override' keyword when overriding functions

aimir
authored and
rwgk
committed
c2846b4
Bugfix for old versions of clang++, which seem to have trouble with the struct being defined inside a module, which was also needlessly ugly anyway

aimir
authored and
rwgk
committed
dfc3ee1
Bugfix for clang++, which doesn't have some of the names in runtime uness they are specified to be static

aimir
authored and
rwgk
committed
42d8694
A fix for clang-tidy performance-inefficient-string-concatenation issues - I personally think this looks uglier, but it's probably worth it for clang-tidy to be happy

aimir
authored and
rwgk
committed
c558073
Possible fix for clang++ linking issues - make the descriptions static constexpr to make sure they are known before linking

Amir
authored and
rwgk
committed
f0294f1
Correct names for previously-wrapped types as keys/values of maps

Amir
authored and
rwgk
committed
ebf874d
Bugfix - typo in type info names which caused things to segfault

Amir
authored and
rwgk
committed
24019ae
Apply suggestions from code review

authored and
rwgk
committed
ddcc4e2
Use detail::remove_cvref_t instead of doing remove_cv and remove_reference separately

aimir
authored and
rwgk
committed
21ec3ec
Avoid names with double underscore, as they are reserved

Amir
authored and
rwgk
committed
8abc0ae
Improved testing for KeysView, ValuesView and ItemsView: check type names + stricter asserts

Amir
authored and
rwgk
committed
60d754f
Moved description logic to helper function in type_caster_base.h

Amir
authored and
rwgk
committed
e054752
style: pre-commit fixes

pre-commit-ci[bot]
authored and
rwgk
committed
b4041b8
Fix a clang-tidy issue: do not use 'else' after 'return'

Amir
authored and
rwgk
committed
3a3b9f2
Apply suggestion by @Skylion007, with additional trivial simplification.
rwgk
committed
4525c5d

Navigation Menu

Correct class names for KeysView, ValuesView and ItemsView in bind_map - #4353

Correct class names for KeysView, ValuesView and ItemsView in bind_map#4353
rwgk merged 20 commits into
pybind:masterfrom
aimir:map-view-types

Commits on Dec 9, 2022

Create templated abstract classes KeysView, ValuesView and ItemsView, and implement them on-the-fly when wrapping any specific map type

We don't want to wrap different ValuesView objects for double values and const double, for example, as both wrappers will be named ValuesView[float]

Fallback to C++ names if key or values types are not wrapped

Added a test for .keys(), .values() and .items() returning the same types for similarly-typed maps

Fixed wrong use of auto in a declarator list: the two descriptions might have different types

Fixes for clang-tidy issues: explicit single-argument constructor, using the 'override' keyword when overriding functions

Bugfix for old versions of clang++, which seem to have trouble with the struct being defined inside a module, which was also needlessly ugly anyway

Bugfix for clang++, which doesn't have some of the names in runtime uness they are specified to be static

A fix for clang-tidy performance-inefficient-string-concatenation issues - I personally think this looks uglier, but it's probably worth it for clang-tidy to be happy

Possible fix for clang++ linking issues - make the descriptions static constexpr to make sure they are known before linking

Correct names for previously-wrapped types as keys/values of maps

Bugfix - typo in type info names which caused things to segfault

Apply suggestions from code review

Use detail::remove_cvref_t instead of doing remove_cv and remove_reference separately

Avoid names with double underscore, as they are reserved

Improved testing for KeysView, ValuesView and ItemsView: check type names + stricter asserts

Moved description logic to helper function in type_caster_base.h

style: pre-commit fixes

Fix a clang-tidy issue: do not use 'else' after 'return'

Apply suggestion by @Skylion007, with additional trivial simplification.