[lldb-dev] marking new summary output for expected timeouts

Todd Fiala via lldb-dev lldb-dev at lists.llvm.org
Fri Dec 11 16:20:33 PST 2015


Hey Pavel and/or Tamas,

Let me know when we're definitely all clear on the expected timeout support
I added to the (now once again) newer default test results.

As soon as we don't need the legacy summary results anymore, I'm going to
strip out the code that manages it.  It is quite messy and duplicates the
content that is better handled by the test event system.

Thanks!

-Todd

On Fri, Dec 11, 2015 at 2:03 PM, Todd Fiala <todd.fiala at gmail.com> wrote:

> I went ahead and added the expected timeout support in r255363.
>
> I'm going to turn back on the new BasicResultsFormatter as the default.
> We can flip this back off if it is still not doing everything we need, but
> I *think* we cover the issue you saw now.
>
> -Todd
>
> On Fri, Dec 11, 2015 at 10:14 AM, Todd Fiala <todd.fiala at gmail.com> wrote:
>
>> Hi Pavel,
>>
>> I'm going to adjust the new summary output for expected timeouts.  I hope
>> to do that in the next hour or less.  I'll put that in and flip the default
>> back on for using the new summary output.
>>
>> I'll do those two changes separately, so you can revert the flip back on
>> to flip it back off if we still have an issue.
>>
>> Sound good?
>>
>> (This can be orthogonal to the new work to mark up expected timeouts).
>> --
>> -Todd
>>
>
>
>
> --
> -Todd
>



-- 
-Todd
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.llvm.org/pipermail/lldb-dev/attachments/20151211/d1162236/attachment.html>


More information about the lldb-dev mailing list