The type derived from `_ERROR_` should not be considered as `__msuccess`
Nobody has claimed this yet.
Assessment
- Difficulty
- 3/5
- Estimated time
- 1-2 days
- Newbie friendliness
- 45/100
Research direction
Start with include/stdexec/__detail/__meta.hpp at lines 172-178 and reproduce the reported behavior using the linked Godbolt example. Replace the specialization approach with the proposed overload-based lookup, and verify that a type derived from ERROR is no longer considered __msuccess.
Written by the indexing model from the issue text.
Description
Use overload instead of specialization:
consteval __msuccess __get_ok_type(...);
template <class What, class... With>
consteval _ERROR_<What, With...> __get_ok_type(const _ERROR_<What, With...>*);
template <class _Ty>
extern decltype(__get_ok_type((_Ty*)nullptr)) __ok_v; // Use overload instead of specialization
Use case: https://godbolt.org/z/P1eGccsEY
- Dominant language
- C++
- Stars
- 2.4k
- Forks
- 270
- Avg merge
- 2d 17h
- Merged PRs (30d)
- 37
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.
More from NVIDIA/stdexec
-
Difficulty 2/5 1-3 hours Newbie friendliness 68/100
-
Difficulty 3/5 1-2 days Newbie friendliness 68/100
-
Difficulty 4/5 3-5 days Newbie friendliness 45/100
-
Difficulty 4/5 3-5 days Newbie friendliness 66/100
-
Difficulty 3/5 1-2 days Newbie friendliness 74/100
Similar issues
-
enhancement
Difficulty 1/5 Under an hour Newbie friendliness 88/100
QuantStack/git2cpp#187 ·
-
Difficulty 2/5 1-3 hours Newbie friendliness 86/100
-
Difficulty 2/5 1-3 hours Newbie friendliness 84/100
mlcommons/mobile_app_open#1182 ·
-
Needs-Triage
Difficulty 2/5 1-3 hours Newbie friendliness 78/100
microsoft/winget-cli#6547 ·
-
Difficulty 1/5 Under an hour Newbie friendliness 90/100
AXERA-TECH/ax-llm#77 ·