From e2ab6d127aa3379ef3ad19779bfe52b273a1a757 Mon Sep 17 00:00:00 2001 From: Tom Tromey Date: Tue, 31 Aug 2004 09:50:40 +0000 Subject: 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/java/text/AttributedString.java | 12 +++++++----- 1 file changed, 7 insertions(+), 5 deletions(-) (limited to 'libjava/java/text/AttributedString.java') diff --git a/libjava/java/text/AttributedString.java b/libjava/java/text/AttributedString.java index 8304ced..41512dc 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()); } -- cgit v1.1