--- parser3/src/include/pa_string.h	2002/02/20 09:13:07	1.125
+++ parser3/src/include/pa_string.h	2002/02/22 12:14:11	1.131
@@ -4,7 +4,7 @@
 	Copyright (c) 2001, 2002 ArtLebedev Group (http://www.artlebedev.com)
 	Author: Alexandr Petrosian <paf@design.ru> (http://paf.design.ru)
 
-	$Id: pa_string.h,v 1.125 2002/02/20 09:13:07 paf Exp $
+	$Id: pa_string.h,v 1.131 2002/02/22 12:14:11 paf Exp $
 */
 
 #ifndef PA_STRING_H
@@ -109,6 +109,7 @@ public:
 
 	String(Pool& apool, const char *src=0, size_t src_size=0, bool tainted=false);
 	String(const String& src);
+	bool is_empty() const { return append_here==head.rows; }
 	size_t size() const;
 	/// convert to C string. if 'lang' known, forcing 'lang' to it
 	char *cstr(Untaint_lang lang=UL_AS_IS, 
@@ -225,10 +226,14 @@ public:
 	const Origin& origin() const;
 #endif
 
-public: // debug private:
+private:
 
-	/// several String fragments
-	struct Chunk {
+	/** several String fragments
+		
+		'mutable' because can write after it's end, after it was appended to somebody 
+		@see String::append
+	*/
+	mutable struct Chunk {
 		typedef uchar count_type;
 		count_type count; ///< the number of rows in chunk
 		/// string fragment or a link to next chunk union
@@ -244,31 +249,23 @@ public: // debug private:
 #endif
 			} item;
 			Chunk *link;  ///< link to the next chunk in chain
-		} rows[CR_PREALLOCATED_COUNT];
+		} rows[CR_PREALLOCATED_COUNT+1/*for head.rows[CR_PREALLOCATED_COUNT].link*/];
 	}
 		head;  ///< the head chunk of the chunk chain
-	Chunk *head_link; ///< next rows are here must go right after 'head'
 
 	/// next append would write to this record
 	Chunk::Row *append_here;
 	
-	/** the address of place where lies address 
-		of the link to the next chunk to allocate
-	*/
-	Chunk::Row *link_row;
-
 private:
 	/// last chunk
 	mutable Chunk *last_chunk;
 
-public: //debug
-	uint used_rows() const;
 private:
 
 	bool chunk_is_full() {
-		return append_here == link_row;
+		return append_here == last_chunk->rows+last_chunk->count;
 	}
-	//uint used_rows() const;
+	uint used_rows() const;
 	void expand();
 	
 	Untaint_lang lang_of(size_t offset) const;
@@ -283,7 +280,7 @@ private:
 
 	String& reconstruct(Pool& pool) const;
 	void join_chain(Pool& pool, 
-					   uint& ai, const Chunk*& achunk, const Chunk::Row*& arow,
+					   const Chunk*& achunk, const Chunk::Row*& arow, uint& acountdown, 
 					   uchar& joined_lang, const char *& joined_ptr, size_t& joined_size) const;
 	String& replace_in_reconstructed(Pool& pool, Dictionary& dict) const;
 
@@ -294,18 +291,22 @@ private: //disabled
 };
 #include "pa_pragma_pack_end.h"
 
-#define STRING_PREFIX_FOREACH_ROW(src, body) { \
-	const Chunk *chunk=&(src).head;  \
-	do { \
-		const Chunk::Row *row=chunk->rows; \
-		for(uint i=0; i<chunk->count; i++, row++) { \
-			if(row==(src).append_here) \
-				goto break2; \
-			\
-			body \
-		} \
-		chunk=row->link; \
-	} while(chunk); \
+#define STRING_PREPARED_FOREACH_ROW(self, body) \
+	while(row!=(self).append_here) { \
+		if(countdown==0) { \
+			chunk=row->link; \
+			row=chunk->rows; \
+			countdown=chunk->count; \
+		}; \
+		{ body } \
+		row++; countdown--; \
+	} 
+
+#define STRING_PREFIX_FOREACH_ROW(self, body) { \
+	const Chunk *chunk=&(self).head;  \
+	const Chunk::Row *row=chunk->rows; \
+	uint countdown=chunk->count; \
+	STRING_PREPARED_FOREACH_ROW(self, body) \
 }
 
 #define STRING_FOREACH_ROW(body) STRING_PREFIX_FOREACH_ROW(*this, body)