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=14965>.
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=14965

Empty comments causes ArrayIndexOutOfBoundsException





------- Additional Comments From [EMAIL PROTECTED]  2002-12-04 00:44 -------
Just a minor comment...

I like your patch, but it may be better to say:

      if (length > 0 && ch[limit-1] == '-')

instead of using limit, because if start > 0 but length = 0, then the test 
would actually be on the wrong array element (I think that the worst case 
scenario is just an extra space in a comment, but I do not think that it costs 
any more to be correct).

If it is desired to have "<!-- -->" instead of "<!---->" Then could use:

      if (length <= 0 || ch[limit-1] == '-')

Art (day late and dollar short)

Reply via email to