From e2ab6d127aa3379ef3ad19779bfe52b273a1a757 Mon Sep 17 00:00:00 2001 From: Tom Tromey Date: Tue, 31 Aug 2004 09:50:40 +0000 Subject: [PATCH] AttributedString.java (AttributedString): Use ArrayList to build array of attribute ranges. 2004-08-31 Tom Tromey * java/text/AttributedString.java (AttributedString): Use ArrayList to build array of attribute ranges. Don't use `attribs' before it is set. From-SVN: r86825 --- libjava/ChangeLog | 6 ++++++ libjava/java/text/AttributedString.java | 12 +++++++----- 2 files changed, 13 insertions(+), 5 deletions(-) diff --git a/libjava/ChangeLog b/libjava/ChangeLog index 5cdc6f4783e..9cf9a202c1b 100644 --- a/libjava/ChangeLog +++ b/libjava/ChangeLog @@ -1,3 +1,9 @@ +2004-08-31 Tom Tromey + + * java/text/AttributedString.java (AttributedString): Use + ArrayList to build array of attribute ranges. Don't use + `attribs' before it is set. + 2004-08-30 Andreas Tobler * HACKING: Remove reference to special automake. No longer needed. diff --git a/libjava/java/text/AttributedString.java b/libjava/java/text/AttributedString.java index 8304cedf7bc..41512dc6c5e 100644 --- a/libjava/java/text/AttributedString.java +++ b/libjava/java/text/AttributedString.java @@ -39,6 +39,7 @@ exception statement from your version. */ package java.text; import java.util.Arrays; +import java.util.ArrayList; import java.util.HashMap; import java.util.Hashtable; import java.util.Iterator; @@ -224,6 +225,7 @@ AttributedString(AttributedCharacterIterator aci, int begin_index, // Loop through and extract the attributes char c = aci.setIndex(begin_index); + ArrayList accum = new ArrayList(); do { sb.append(c); @@ -272,17 +274,17 @@ AttributedString(AttributedCharacterIterator aci, int begin_index, Map new_map = new Hashtable(); new_map.put(attrib, attrib_obj); - // Add it to the attribute list. Yes this is a bad way to do things. - AttributeRange[] new_list = new AttributeRange[attribs.length + 1]; - System.arraycopy(attribs, 0, new_list, 0, attribs.length); - attribs = new_list; - attribs[attribs.length - 1] = new AttributeRange(new_map, rs, rl); + // Add it to the attribute list. + accum.add(new AttributeRange(new_map, rs, rl)); } c = aci.next(); } while(c != CharacterIterator.DONE); + attribs = new AttributeRange[accum.size()]; + attribs = (AttributeRange[]) accum.toArray(attribs); + sci = new StringCharacterIterator(sb.toString()); } -- 2.30.2