Use brace initialization instead of copy initialization
Nobody has claimed this yet.
Assessment
- Difficulty
- 5/5
- Estimated time
- Over a week
- Newbie friendliness
- 25/100
Research direction
The issue names no files or tests, so first locate all C-style copy initialisations across the course repository and review the linked C++ Core Guideline. Replace the applicable uses with uniform brace initialisation throughout the codebase, then verify that the project still builds and its existing checks pass.
Written by the indexing model from the issue text.
Description
@mmcleod89 Let us systematically replace all uses of C-style copy initialisations with uniform brace initialisation. This is a best practice that will prevent narrowing conversions.
See this C++ core guideline here for more explanation from Stroustrup: https://isocpp.github.io/CppCoreGuidelines/CppCoreGuidelines#Res-list
- Dominant language
- Ruby
- Stars
- 31
- Forks
- 20
- 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 UCL/research-computing-with-cpp
-
Difficulty 5/5 Over a week Newbie friendliness 20/100
UCL/research-computing-with-cpp#154 · 1 reaction ·
-
Refactor Week 1 Open
Difficulty 4/5 3-5 days Newbie friendliness 25/100
-
Difficulty 3/5 1-2 days Newbie friendliness 35/100
-
Difficulty 4/5 3-5 days Newbie friendliness 35/100
-
Difficulty 4/5 3-5 days Newbie friendliness 25/100
All issues in UCL/research-computing-with-cpp
Similar issues
-
user-reported
Difficulty 2/5 1-3 hours Newbie friendliness 85/100
Kong/developer.konghq.com#7316 ·
-
Difficulty 2/5 1-3 hours Newbie friendliness 76/100
TheOdinProject/curriculum#31408 ·
-
bug
Difficulty 2/5 1-3 hours Newbie friendliness 84/100
notch8/utk_knapsack#148 ·
-
Difficulty 1/5 Under an hour Newbie friendliness 78/100
Homebrew/homebrew-cask#288729 · 1 comment ·
-
Difficulty 2/5 1-3 hours Newbie friendliness 68/100