DO NOT REPLY [Bug 8085] - cvschangelog does not correctly detect ends of comments

2003-10-14 Thread bugzilla
DO NOT REPLY TO THIS EMAIL, BUT PLEASE POST YOUR BUG 
RELATED COMMENTS THROUGH THE WEB INTERFACE AVAILABLE AT
.
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=8085

cvschangelog does not correctly detect ends of comments

[EMAIL PROTECTED] changed:

   What|Removed |Added

 Status|REOPENED|RESOLVED
 Resolution||FIXED
   Target Milestone|--- |1.7



--- Additional Comments From [EMAIL PROTECTED]  2003-10-14 08:46 ---
I've turned the startsWith into equals calls for both the --- and === markers.
I'm not sure whether the exact numer of dashes is guaranteed, though, that's why
I don't want to put the change into 1.6 now, as it wouldn't get as much testing 
as
it IMHO should.

-
To unsubscribe, e-mail: [EMAIL PROTECTED]
For additional commands, e-mail: [EMAIL PROTECTED]



DO NOT REPLY [Bug 8085] - cvschangelog does not correctly detect ends of comments

2003-04-16 Thread bugzilla
DO NOT REPLY TO THIS EMAIL, BUT PLEASE POST YOUR BUG 
RELATED COMMENTS THROUGH THE WEB INTERFACE AVAILABLE AT
.
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=8085

cvschangelog does not correctly detect ends of comments

[EMAIL PROTECTED] changed:

   What|Removed |Added

 Status|RESOLVED|REOPENED
 Resolution|FIXED   |



--- Additional Comments From [EMAIL PROTECTED]  2003-04-16 00:44 ---
The current solution fixes all cases where the comment is less than 
"". I had a case where a workmate entered
"-" as part of a comment which confused
CvsChangeLog. Wouldn't a better solution be to use a
.equals("") rather than
a.startsWith("")? This will reduce the likelyhood of
the problem reoccuring in the future...