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

Errors in NodeSet.java removeElement and removeElementAt

[EMAIL PROTECTED] changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
                 CC|                            |[EMAIL PROTECTED]
         AssignedTo|[EMAIL PROTECTED]    |[EMAIL PROTECTED]



------- Additional Comments From [EMAIL PROTECTED]  2003-06-25 21:28 -------
Patch looks good, except I think the length argument in the arraycopy should be:
  m_firstFree - i - 1

Also, I'd argue that throwing an AIOOBE is the proper behaviour if i >= 
m_firstFree is passed into removeElementAt(), instead of silently swallowing 
the error.

Reply via email to