DO NOT REPLY TO THIS EMAIL, BUT PLEASE POST YOUR BUG 
RELATED COMMENTS THROUGH THE WEB INTERFACE AVAILABLE AT
<http://nagoya.apache.org/bugzilla/show_bug.cgi?id=16879>.
ANY REPLY MADE TO THIS MESSAGE WILL NOT BE COLLECTED AND 
INSERTED IN THE BUG DATABASE.

http://nagoya.apache.org/bugzilla/show_bug.cgi?id=16879

CvsChangeLog attr "start" causes IllegalArgumentException

[EMAIL PROTECTED] changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
             Status|NEW                         |ASSIGNED



------- Additional Comments From [EMAIL PROTECTED]  2003-02-20 14:24 -------
I've committed a slightly different patch. Changing the signature of setStart()
is not a backward compatible change for anyone who has extended this task. While
that is likely to be the empty set, it was not really necessary. Instead I just
made setStateDate take the String argument.

Since this attribute is new, and not subject to backward compat, I made the
default date format the CVS default yyyy-MM-dd.

I introduced a new method determineDates which sets m_start from m_startDate, as
well as detecting any attempts to set the start date with two different methods.

Finally I did not apply the date format specified to the output date formatting
passed to CVS. This needs to be in a consistent format so CVS understands it.

I have not applied the doc and test patches yet. Can you check out if this code
works for you (you'll need to use "startdate" in place of "start", etc. If you
OK, I'll do the remaining work tomorrow.

Reply via email to