Adrian Crum wrote:
> Adam Heath wrote:
>> adri...@apache.org wrote:
>>> Author: adrianc
>>> Date: Wed Feb 17 21:40:41 2010
>>> New Revision: 911182
>>>
>>> URL: http://svn.apache.org/viewvc?rev=911182&view=rev
>>> Log:
>>> Added a test for TimeDuration pre-epoch elapsed time, plus added SVN
>>> properties.
>>
>> I was just talking to Ean that there needed to be a test for your fix.
>>   Have you verified that the test case breaks before you added your fix?
> 
> No. The original code assumed both millisecond values were positive, so
> it was a pretty safe assumption that it would break with negative numbers.

Ok, I'll add it to my list of things to try.

Reply via email to