Code review comment for lp://staging/~jameinel/subunit/omit-times-623654

Revision history for this message
Jonathan Lange (jml) wrote :

On Tue, Oct 19, 2010 at 10:26 AM, Robert Collins
<email address hidden> wrote:
> I think keeping the first time is actually ok and quite interesting.
> The rest I agree with.

FWIW, the way the branch behaves now it keeps the first timestamp
after the last interesting event and the first timestamp before the
next interesting event. So....

<BEGINNING>
time: 1
time: N
test: event
error: event
time: N + A
time: N + A + B
test: event2
error: event2
time: N + A + B + C
<END>

jml

« Back to merge proposal