Skip to content

Commit 9f35a59

Browse files
author
Gonzalo Diaz
committed
[REFACTOR] [Hacker Rank]: Warmup: Simple Array Sum. Static check issue solved.
-> src/lib/exercises/src/hackerrank/warmup/simple_array_sum.cpp:20:41: performance: Function parameter 'ar' should be passed by const reference. [passedByValue] int simpleArraySum(std::vector<int> ar) {
1 parent 4bc0c15 commit 9f35a59

File tree

2 files changed

+2
-2
lines changed

2 files changed

+2
-2
lines changed

src/lib/exercises/include/exercises/hackerrank/warmup/simple_array_sum.hpp

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -4,5 +4,5 @@
44
#pragma once
55

66
namespace hackerrank::warmup {
7-
int simpleArraySum(std::vector<int> ar);
7+
int simpleArraySum(const std::vector<int>& ar);
88
}

src/lib/exercises/src/hackerrank/warmup/simple_array_sum.cpp

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -17,7 +17,7 @@ vector<string> split(const string &);
1717

1818
namespace hackerrank::warmup {
1919

20-
int simpleArraySum(std::vector<int> ar) {
20+
int simpleArraySum(const std::vector<int>& ar) {
2121
int accum = 0;
2222

2323
for(const int i : ar) {

0 commit comments

Comments
 (0)