In an erlier message Eric Kow wrote:
> On 29 janv. 06, at 21:30, Tommy Pettersson wrote:
> >I get an error from test/dateparser.sh:
> >
> >  try_date '4 score, 7 years ago'
> >  darcs failed:  Time.toClockTime: invalid input
> 
> Interesting.  And on my computer '105 years ago' fails (yet 104 years
> ago passes).  This being said, the ability to parse scores of years is
> absolutely needless.  I can't imagine somebody in 2100 wanting to look
> at patches from a repository 87 years back, much less using the
> expression.

On Sun, Jan 29, 2006 at 10:10:32PM +0100, Eric Kow wrote:
> Sun Jan 29 22:09:33 CET 2006  Eric Kow <[EMAIL PROTECTED]>
>   * Remove silliness from date parser.
>
>   Remove ability to parse 'score', 'tea time', 'bed time', and
>   'proper bed time'.  More code means more room for error.

While I agree more code means more errors I find this sort of
"silliness" rather cute. While to "fix" the test suite it is
enough to remove the big time interval from it, I think the real
problem is the exception is thrown at all on some dates. The
user might accidently type such date. Is it possible to catch
the exception and have darcs print something nicer, like "Either
your repo or your computer is too old, troglodyte!"?


-- 
Tommy Pettersson <[EMAIL PROTECTED]>

_______________________________________________
darcs-devel mailing list
[email protected]
http://www.abridgegame.org/cgi-bin/mailman/listinfo/darcs-devel

Reply via email to