Changeset 26520

Timestamp:

2012-11-27T00:06:10+13:00 (11 years ago)

Author:

davidb

Message:

Append string functions changed to a doubling algorithm for reserved capacity. This can lead to some over allocation at times, however it helps avoid the *highly* *undesirable* resource debilitating situation where a large text string is being built up by appending a short bits of text one at a time -- as happens when lucenesource::get_document() is required to retrieve a large section of text from a doc.xml file

File:

• main/trunk/greenstone2/common-src/src/lib/text_t.cpp (modified) (2 diffs)

main/trunk/greenstone2/common-src/src/lib/text_t.cpp

@@ -219 +219 @@
   unsigned char *us = (unsigned char *)s;
   if (text.capacity() < (text.size() + len + 2)) {
-    text.reserve(text.size() + len + 2);
+    text.reserve((2*text.size()) + len + 2);
   }
 
@@ -234 +234 @@
   size_t len = strlen(s);
   if (text.capacity() < (text.size() + len + 2)) {
-    text.reserve(text.size() + len + 2);
+    text.reserve((2*text.size()) + len + 2);
   }