boostorg / boostorg/sort

Constants should be policy-based

Open
#36 3 comments 1 reaction 0 assignees View on GitHub
Dominant language
C++
Stars
58
Forks
47
PR merge metrics
No merged PRs in 30d

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.

Contributor guide

No contributing guide indexed for this repository

Assessment

This issue has not been assessed yet.

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.