Requirements on keys for bulk visitation are overly restrictive
Nobody has claimed this yet.
Assessment
- Difficulty
- 4/5
- Estimated time
- 3-5 days
- Newbie friendliness
- 35/100
Research direction
Start by tracing the bulk visitation requirements for FwdIterator value types, key_type, Hash, and Pred in the unordered module. Check the existing tests or add focused coverage for keys implicitly convertible to key_type, and consider the work complete when that scenario is accepted without weakening the stated transparent-hash requirements.
Written by the indexing model from the issue text.
Description
Currently, bulk visitation requires that
[f]or K = std::iterator_traits::value_type, either K [be] key_type or else Hash::is_transparent and Pred::is_transparent [be] valid member typedefs.
This precludes the scenario where the keys are implicitly convertible to key_type, which seems to be overly restrictive. The drawback of allowing this is that conversion from std::iterator_traits<FwdIterator>::value_type to key_type will happen more than once for each key (twice, to be precise).
- Dominant language
- C++
- Stars
- 92
- Forks
- 88
- PR merge metrics
- No merged PRs in 30d
Getting set up
This project ships no dev container, Dockerfile or contributing guide, so setting up is up to you: start from its README, and see our first-contribution guide for the general steps.
First steps
- Read the whole issue, then the project's contributing guide.
- Comment on the issue to say you are picking it up — it saves two people doing the same work.
- Fork the repository and make your change on a branch.
- Open a pull request that references the issue number.
More from boostorg/unordered
-
Difficulty 2/5 1-3 hours Newbie friendliness 68/100
-
Difficulty 4/5 3-5 days Newbie friendliness 35/100
-
Difficulty 5/5 Over a week Newbie friendliness 35/100
-
Documentation omnibusMay be free again @cmazakas claimed this 613 days ago, and no pull request is open. Open
-
Difficulty 5/5 Over a week Newbie friendliness 25/100
All issues in boostorg/unordered
Similar issues
-
bug chart-audit
Difficulty 1/5 Under an hour Newbie friendliness 92/100
Maintainers usually reply within 1 day
-
Difficulty 2/5 1-3 hours Newbie friendliness 85/100
godotengine/godot#124120 ·
Maintainers usually reply within 1 day
-
Component: R Type: bug
Difficulty 2/5 1-3 hours Newbie friendliness 72/100
apache/arrow#51695 · 1 comment ·
Maintainers usually reply within 1 day
-
HasBacktrace Priority-Critical
Difficulty 2/5 1-3 hours Newbie friendliness 78/100
azerothcore/azerothcore-wotlk#27921 ·
Maintainers usually reply within 1 day
-
area/ysql kind/bug priority/medium
Difficulty 2/5 1-3 hours Newbie friendliness 86/100
yugabyte/yugabyte-db#34584 ·
Maintainers usually reply within 1 day