[Bug #744] java.text.CollationElementIterator.next() is broken

[nobody]

=================== BUG #744: LATEST MODIFICATIONS ==================
http://savannah.gnu.org/bugs/?func=detailbug&bug_id=744&group_id=85

Changes by: Brian Jones <address@hidden>
Date: 2003-Jan-25 17:25 (US/Eastern)

            What     | Removed                   | Added
---------------------------------------------------------------------------
          Resolution | None                      | Fixed
         Assigned to | None                      | cbj
              Status | Open                      | Closed


------------------ Additional Follow-up Comments ----------------------------
Fixed by Julian Dolby, checked into CVS on 12/01/2002.  I've created a Mauve 
test for this.



=================== BUG #744: FULL BUG SNAPSHOT ===================


Submitted by: None                    Project: classpath                    
Submitted on: 2002-Jun-24 15:56
Severity:  5 - Major                  Resolution:  Fixed                    
Assigned to:  cbj                     Status:  Closed                       
Platform Version:  Red Hat 7.x        

Summary:  java.text.CollationElementIterator.next() is broken

Original Submission:  The "pos" field is being incremented before the value of 
the character at that position is queried.  Also, this causes a 
StringIndexOutOfBoundsException for zero-length strings.

I would suggest changing the method to work as follows:
1. test pos against string length, if at end return NULLORDER
2. get value of character
3. increment pos
4. Compute the return value and return it

I'm using a CVS version of classpath checked out on June 23rd, 2002.

Follow-up Comments
*******************

-------------------------------------------------------
Date: 2003-Jan-25 17:25             By: cbj
Fixed by Julian Dolby, checked into CVS on 12/01/2002.  I've created a Mauve 
test for this.


CC list is empty


No files currently attached


For detailed info, follow this link:
http://savannah.gnu.org/bugs/?func=detailbug&bug_id=744&group_id=85