Skip to content

Commit 7334f2b

Browse files
committed
Use range-based for loop
1 parent 31cf805 commit 7334f2b

File tree

2 files changed

+9
-11
lines changed

2 files changed

+9
-11
lines changed

ir/Function.cpp

Lines changed: 5 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -13,14 +13,13 @@ Function::Function(std::string name, std::vector<Parameter> parameters,
1313
llvm::raw_ostream &operator<<(llvm::raw_ostream &s, const Function &func) {
1414
s << " def " << handleReservedWords(func.name)
1515
<< "(";
16-
for (std::vector<Parameter>::size_type i = 0; i < func.parameters.size(); i++) {
17-
const Parameter &param = func.parameters[i];
18-
s << handleReservedWords(param.getName())
16+
std::string sep = "";
17+
for (const auto &param : func.parameters) {
18+
s << sep
19+
<< handleReservedWords(param.getName())
1920
<< ": "
2021
<< param.getType();
21-
if (i + 1 != func.parameters.size()) {
22-
s << ", ";
23-
}
22+
sep = ", ";
2423
}
2524
if (func.isVariadic) {
2625
/* the C Iso require at least one argument in a variadic function, so the comma is fine */

ir/Struct.cpp

Lines changed: 4 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -24,11 +24,10 @@ TypeDef Struct::generateTypeDef() const {
2424

2525
std::string Struct::getFieldsTypes() const {
2626
std::stringstream s;
27-
for (std::vector<Field>::size_type i = 0; i < fields.size(); i++) {
28-
s << fields[i].getType();
29-
if (i + 1 != fields.size()) {
30-
s << ", ";
31-
}
27+
std::string sep = "";
28+
for (const auto &field : fields) {
29+
s << sep << field.getType();
30+
sep = ", ";
3231
}
3332
return s.str();
3433
}

0 commit comments

Comments
 (0)