commit-classpath
[Top][All Lists]
Advanced

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

[patch #1658] [Patch #1658] Bug fix for CollationElementIterator.next()


From: noreply
Subject: [patch #1658] [Patch #1658] Bug fix for CollationElementIterator.next()
Date: Thu, 26 Jun 2003 12:13:57 -0400
User-agent: Mozilla/5.0 (compatible; Konqueror/2.2.2-3; Linux)

Patch #1658 has been updated. 

Project: 
Category: None
Status: Open
Summary: Bug fix for CollationElementIterator.next()

Follow-Ups:

Date: Wed 06/25/2003 at 01:47
By: mkoch

Comment:
Are you okay with this revised patch to make clearer what value of index is 
used and when its incremented ?
-------------------------------------------------------

Date: Thu 06/26/2003 at 01:48
By: crawley

Comment:
I don't think your change makes any difference to
the readability for someone used to reading Java
(or C or C++).  But if you prefer the code that way,
I don't mind.

However, in uploading a new version of the patch, 
you have clobbered all of my explanations, and my
comments for the ChangeLog file.  I hope you kept
a copy ...
-------------------------------------------------------

Date: Fri 06/27/2003 at 02:13
By: crawley

Comment:
(This replaces the lost description, etc)

This patch fixes a bug in the next() method that caused 
it to skip over the first character of the iterator's
string.  (The 'index' was incremented to early.)

2003-06-xx  Stephen Crawley <address@hidden>

        * java/text/CollationElementIterator.java (next) :
          fix bug that caused the first character of the
          Iterator's text to be skipped.

-------------------------------------------------------

-------------------------------------------------------
For more info, visit:

http://savannah.gnu.org/patch/?func=detailpatch&patch_id=1658&group_id=85

_______________________________________________
  Message sent via/by Savannah
  http://savannah.gnu.org/





reply via email to

[Prev in Thread] Current Thread [Next in Thread]