Constants should be policy-based
Nobody has claimed this yet.
Assessment
- Difficulty
- 5/5
- Estimated time
- Over a week
- Newbie friendliness
- 35/100
- Issue type
- Feature
- Clarity
- Mostly clear
- Activity status
- Stale
- Tech stack
- cpp
- Domain
- performance
Research direction
Start with boost/sort/spreadsort/detail/constants.hpp, then trace how spreadsort consumes those constants. Determine the template entry points that would need a policy parameter. Done means callers can customize the tuning values per invocation without modifying installed Boost headers; no tests or specific entry points are named in the issue.
Written by the indexing model from the issue text.
Description
Right now, boost/sort/spreadsort/detail/constants.hpp defines a bunch of performance-tuning constants for spreadsort. Why make these constants global? Because these constants live in a simple enum, they're impossible to change without changing the Boost headers, which is awkward when compiling against a Boost installed system-wide.
Instead, spreadsort should accept a struct containing these constants as a template argument. This way, users would be able to customize these parameters for specific invocations of spreadsort without changing the Boost headers themselves.
- Dominant language
- C++
- Stars
- 58
- Forks
- 47
- 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/sort
-
Difficulty 5/5 Over a week Newbie friendliness 25/100
-
Difficulty 5/5 Over a week Newbie friendliness 22/100
-
Additional memory specification does not seem to consider the index for block indirect sortMay be free again @fjtapia claimed this 1305 days ago, and no pull request is open. Open
-
pivot.hpp std::size_t fixMay be free again @fjtapia claimed this 1346 days ago, and no pull request is open. Open
-
boost/sort/block_indirect_sort/block_indirect_sort.hpp doesn't compile under VS2017May be free again @fjtapia claimed this 1346 days ago, and no pull request is open. Open
Similar issues
-
feature request
Difficulty 2/5 1-3 hours Newbie friendliness 68/100
Maintainers usually reply within 2 days
-
Difficulty 2/5 1-3 hours Newbie friendliness 76/100
Maintainers usually reply within 1 day
-
bug
Difficulty 2/5 1-3 hours Newbie friendliness 78/100
gavinlouuu-kpt/mib-studio-qt#517 ·
Maintainers usually reply within 1 day
-
Difficulty 2/5 1-3 hours Newbie friendliness 68/100
Maintainers usually reply within 1 day
-
bug
Difficulty 2/5 1-3 hours Newbie friendliness 73/100
EchoTools/nevr-runtime#116 ·
Maintainers usually reply within 1 day