[func.wrap.ref.class] Rename the parameter `c` in `constant_wrapper<c, F>`?
Nobody has claimed this yet.
- Dominant language
- TeX
- Stars
- 221
- Forks
- 813
- Avg merge
- 16h 4m
- Merged PRs (30d)
- 36
Description
https://eel.is/c++draft/func.wrap.ref#class:
template<auto c, class F> constexpr function_ref(constant_wrapper<c, F>) noexcept;
The use of a lowercase c as the template parameter name here seems a bit odd to me; in libraries, we rarely seem to use a single lowercase letter as a template parameter. The c here also overlaps with the c in cv.
Would it be better to use X, or even _ (because we didn't use it at all)?
Contributor guide
No contributing guide indexed for this repository
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.
Research direction
Start with the [func.wrap.ref] class wording linked in the issue and review the surrounding declaration and comment thread for the naming conventions being applied. The work is done when the template parameter in the constant_wrapper<c, F> declaration has an agreed replacement name and the draft wording reflects it.
Written by the indexing model from the issue text.
Assessment
- Tech stack
- cpp
- Domain
- documentation
- Issue type
- Documentation
- Difficulty
- 1/5
- Estimated time
- 1-3 hours
- Activity status
- Quiet
- Clarity
- Clearly specified
- Newbie friendliness
- 55/100