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.