Re: [Lldb-commits] [lldb] r254946 - Rename test_results.py to result_formatter.py.

2015-12-07 Thread Todd Fiala via lldb-commits
Yep all good. We'll get to it soon enough. On Mon, Dec 7, 2015 at 4:30 PM, Zachary Turner wrote: > Yea, that actually came up on my radar too, but when I tried to do more > things at once it started turning into a rats nest. Refactoring Python is > kind of hard so I want to make the changes as

Re: [Lldb-commits] [lldb] r254946 - Rename test_results.py to result_formatter.py.

2015-12-07 Thread Zachary Turner via lldb-commits
Yea, that actually came up on my radar too, but when I tried to do more things at once it started turning into a rats nest. Refactoring Python is kind of hard so I want to make the changes as small as possible that if anything breaks, bisecting will narrow the problem down as much as possible. On

Re: [Lldb-commits] [lldb] r254946 - Rename test_results.py to result_formatter.py.

2015-12-07 Thread Todd Fiala via lldb-commits
This is okay, but I was planning on breaking that out into multiple files. Right now the test_results module also has the EventBuilder in it, which is not a results formatter. But we can break that out separately. On Mon, Dec 7, 2015 at 1:23 PM, Zachary Turner via lldb-commits < lldb-commits@list

[Lldb-commits] [lldb] r254946 - Rename test_results.py to result_formatter.py.

2015-12-07 Thread Zachary Turner via lldb-commits
Author: zturner Date: Mon Dec 7 15:23:41 2015 New Revision: 254946 URL: http://llvm.org/viewvc/llvm-project?rev=254946&view=rev Log: Rename test_results.py to result_formatter.py. There is already a class called LLDBTestResults which I would like to move into a separate file, but the most approp