Support C++17
Nobody has claimed this yet.
Assessment
- Difficulty
- 1/5
- Estimated time
- 1-3 hours
- Newbie friendliness
- 68/100
Research direction
Open src/MeshEditor.cc and locate the two uses of std::bind2nd described in the issue. Replace them with the shown C++17-compatible lambda form, then build the project with C++17 enabled. Done means both instances are updated and the project builds successfully without std::bind2nd.
Written by the indexing model from the issue text.
Description
C++17 removes std::bind2nd from the std library. It's easy to replace the two instances of it in src/MeshEditor.cc.
Original code:
replace_if(mNodeMask.begin(), mNodeMask.end(), bind2nd(equal_to<unsigned>(), 2), 1);
New code:
replace_if(mNodeMask.begin(), mNodeMask.end(), [] (unsigned val) { return val == 2; }, 1);
- Dominant language
- C++
- Stars
- 10
- Forks
- 4
- PR merge metrics
- No merged PRs in 30d
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 llnl/polytope
-
Difficulty 4/5 3-5 days Newbie friendliness 42/100
-
Difficulty 4/5 3-5 days Newbie friendliness 25/100
-
Difficulty 2/5 1-3 hours Newbie friendliness 55/100
-
testing
-
bug testing
Difficulty 5/5 Over a week Newbie friendliness 20/100
Similar issues
-
Difficulty 1/5 Under an hour Newbie friendliness 90/100
AXERA-TECH/ax-llm#77 ·
-
Difficulty 1/5 Under an hour Newbie friendliness 90/100
games-on-whales/wolf#509 ·
-
Difficulty 2/5 1-3 hours Newbie friendliness 74/100
-
bug-unconfirmed
Difficulty 2/5 1-3 hours Newbie friendliness 76/100
-
Difficulty 2/5 1-3 hours Newbie friendliness 74/100
NVIDIA/cuda-samples#453 ·