luke
/
sbt
mirror of https://github.com/sbt/sbt.git
1
0
Fork 0
Code Issues Packages Projects Releases Wiki Activity

Make EscHelpers.removeEscapeSequences handle partial escape sequences 

Fixes #67
This commit is contained in:
Dale Wijnand 2017-12-21 17:09:30 +00:00
parent 83f8eabfab
commit 5ed25cbc56
No known key found for this signature in database
GPG Key ID: 4F256E3D151DF5EF
2 changed files with 12 additions and 1 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

3
internal/util-logging/src/main/scala/sbt/internal/util/EscHelpers.scala
View File

@ -66,8 +66,9 @@ object EscHelpers {
} else { } else {
sb.append(s, start, escIndex) sb.append(s, start, escIndex)
val next: Int = val next: Int =
if (escIndex + 1 >= s.length) skipESC(s, escIndex + 1)
// If it's a CSI we skip past it and then look for a terminator. // If it's a CSI we skip past it and then look for a terminator.
if (isCSI(s.charAt(escIndex + 1))) skipESC(s, escIndex + 2) else if (isCSI(s.charAt(escIndex + 1))) skipESC(s, escIndex + 2)
else if (isAnsiTwoCharacterTerminator(s.charAt(escIndex + 1))) escIndex + 2 else if (isAnsiTwoCharacterTerminator(s.charAt(escIndex + 1))) escIndex + 2
else { else {
// There could be non-ANSI character sequences we should make sure we handle here. // There could be non-ANSI character sequences we should make sure we handle here.

10
internal/util-logging/src/test/scala/Escapes.scala
View File

@ -34,6 +34,16 @@ object Escapes extends Properties("Escapes") {
!hasEscapeSequence(removed) !hasEscapeSequence(removed)
} }
private[this] final val ecs = ESC.toString
private val partialEscapeSequences =
Gen.oneOf(Gen const ecs, Gen const ecs ++ "[", Gen.choose('@', '_').map(ecs :+ _))
property("removeEscapeSequences handles partial escape sequences") =
forAll(partialEscapeSequences) { s =>
val removed: String = removeEscapeSequences(s)
s"Escape sequence removed: '$removed'" |: !hasEscapeSequence(removed)
}
property("removeEscapeSequences returns string without escape sequences") = property("removeEscapeSequences returns string without escape sequences") =
forAllNoShrink(genWithoutEscape, genEscapePairs) { forAllNoShrink(genWithoutEscape, genEscapePairs) {
(start: String, escapes: List[EscapeAndNot]) => (start: String, escapes: List[EscapeAndNot]) =>