Skip to content

Commit 29f2d5c

Browse files
authored
Merge pull request #20054 from xedin/space-show-interleave
[AST] NFC: Switch `Space` printing to use `interleave` instead of han…
2 parents c2dad28 + 35e2d2e commit 29f2d5c

File tree

1 file changed

+5
-13
lines changed

1 file changed

+5
-13
lines changed

lib/Sema/TypeCheckSwitchStmt.cpp

Lines changed: 5 additions & 13 deletions
Original file line numberDiff line numberDiff line change
@@ -18,6 +18,7 @@
1818
#include "swift/AST/ASTPrinter.h"
1919
#include "swift/AST/DiagnosticsSema.h"
2020
#include "swift/AST/Pattern.h"
21+
#include "swift/Basic/STLExtras.h"
2122

2223
#include <llvm/ADT/APInt.h>
2324
#include <llvm/ADT/APFloat.h>
@@ -684,11 +685,9 @@ namespace {
684685
llvm_unreachable("Attempted to display disjunct to user!");
685686
} else {
686687
buffer << "DISJOIN(";
687-
for (auto &sp : Spaces) {
688-
buffer << "\n";
688+
interleave(Spaces, [&](const Space &sp) {
689689
sp.show(buffer, forDisplay);
690-
buffer << " |";
691-
}
690+
}, [&buffer]() { buffer << " |\n"; });
692691
buffer << ")";
693692
}
694693
}
@@ -707,16 +706,9 @@ namespace {
707706
}
708707

709708
buffer << "(";
710-
bool first = true;
711-
for (auto &param : Spaces) {
712-
if (!first) {
713-
buffer << ", ";
714-
}
709+
interleave(Spaces, [&](const Space &param) {
715710
param.show(buffer, forDisplay);
716-
if (first) {
717-
first = false;
718-
}
719-
}
711+
}, [&buffer]() { buffer << ", "; });
720712
buffer << ")";
721713
}
722714
break;

0 commit comments

Comments
 (0)