[Lldb-commits] [PATCH] D75070: [lldb] Color the line marker

[Greg Clayton via Phabricator via lldb-commits]

clayborg requested changes to this revision.
clayborg added a comment.
This revision now requires changes to proceed.

If we are going to use this to to ansi stuff we should ensure the suffix works too. The current testing example will turn the line green along with all lines that follow if all other ansi stuff is removed. Right now I am guessing we are getting lucky because something else is doing the ansi reset. Once that is tested, this will be good to go.



================
Comment at: lldb/test/Shell/Settings/TestLineMarkerColor.test:4
+settings set use-color true
+settings set stop-show-line-ansi-prefix ""
+b foo
----------------
test suffix?


================
Comment at: lldb/test/Shell/Settings/TestLineMarkerColor.test:8
+c
+settings set stop-show-line-ansi-prefix ${ansi.fg.green}
+run
----------------
test suffix?


CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D75070/new/

https://reviews.llvm.org/D75070