Skip to content

Commit 35f123a

Browse files
authored
Merge pull request #10250 from dotty-staging/robust-sbt-reporter
sbt reporter: Don't crash with malformed positions
2 parents 088398b + 341d330 commit 35f123a

File tree

1 file changed

+3
-1
lines changed

1 file changed

+3
-1
lines changed

sbt-bridge/src/xsbt/DelegatingReporter.java

Lines changed: 3 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -127,7 +127,9 @@ public Optional<String> pointerSpace() {
127127
String lineContent = this.lineContent();
128128
int pointer = this.pointer().get();
129129
StringBuilder result = new StringBuilder();
130-
for (int i = 0; i < pointer; i++)
130+
// Don't crash if pointer is out-of-bounds (happens with some macros)
131+
int fixedPointer = Math.min(pointer, lineContent.length());
132+
for (int i = 0; i < fixedPointer; i++)
131133
result.append(lineContent.charAt(i) == '\t' ? '\t' : ' ');
132134
return Optional.of(result.toString());
133135
}

0 commit comments

Comments
 (0)