[Lldb-commits] PATCH for REVIEW: parse ansi color codes for prompt, add use color settings

Mike Sartain mikesart at valvesoftware.com
Thu May 23 13:00:38 PDT 2013


From: Malea, Daniel [daniel.malea at intel.com]
> 
> Hi Mike, this looks promising! I'm having some difficulty applying this patch on the latest trunk though; seeing a conflict in AnsiTerminal.h.
> 
> As an aside, lldb folk don't seem to use this much, but if you have a git workflow, I found LLVM reviews to be pretty painless through the "phabricator" instance that's been set up. If you haven't seen it, check out:

How does this look?

http://llvm-reviews.chandlerc.com/D852

And this is the Linux "platform process list" one...

http://llvm-reviews.chandlerc.com/D853

The UI on chandlerc.com is really slick. So is it best to create a diff there (like the two above) and then send e-mail to the lldb-commits alias for review before checking in?
Thanks Daniel.
 -Mike



More information about the lldb-commits mailing list