--- parser3/src/main/pa_string.C 2001/01/27 13:09:45 1.9 +++ parser3/src/main/pa_string.C 2001/01/29 12:21:35 1.12 @@ -1,10 +1,10 @@ /* - $Id: pa_string.C,v 1.9 2001/01/27 13:09:45 paf Exp $ + $Id: pa_string.C,v 1.12 2001/01/29 12:21:35 paf Exp $ */ #include -#include "pa_pool.h" +#include "pa_string.h" #include "pa_hash.h" void *String::operator new(size_t size, Pool *apool) { @@ -23,7 +23,7 @@ void String::construct(Pool *apool) { void String::expand() { curr_chunk_rows+=curr_chunk_rows*CR_GROW_PERCENT/100; Chunk *chunk=static_cast( - pool->malloc(sizeof(Chunk::Row)*curr_chunk_rows+sizeof(Chunk *))); + pool->malloc(sizeof(int)+sizeof(Chunk::Row)*curr_chunk_rows+sizeof(Chunk *))); chunk->count=curr_chunk_rows; link_row->link=chunk; append_here=chunk->rows; @@ -37,15 +37,15 @@ String::String(String& src) { int src_used_rows=src.used_rows(); if(src_used_rows<=head.count) { - // new rows fit into preallocated area + // all new rows fit into preallocated area curr_chunk_rows=head.count; memcpy(head.rows, src.head.rows, sizeof(Chunk::Row)*src_used_rows); append_here=&head.rows[src_used_rows]; link_row=&head.rows[curr_chunk_rows]; } else { // warning: - // heavy relies on the fact - // that preallocated area is the same for all strings + // heavily relies on the fact + // "preallocated area is the same for all strings" // // info: // allocating only enough mem to fit src string rows @@ -57,7 +57,7 @@ String::String(String& src) { // remaining rows into new_chunk curr_chunk_rows=src_used_rows-head.count; Chunk *new_chunk=static_cast( - pool->malloc(sizeof(Chunk::Row)*curr_chunk_rows+sizeof(Chunk *))); + pool->malloc(sizeof(int)+sizeof(Chunk::Row)*curr_chunk_rows+sizeof(Chunk *))); new_chunk->count=curr_chunk_rows; head.preallocated_link=new_chunk; append_here=link_row=&new_chunk->rows[curr_chunk_rows]; @@ -159,8 +159,8 @@ bool String::operator == (String& src) { int b_offset=0; Chunk::Row *a_end=append_here; Chunk::Row *b_end=src.append_here; - int a_count=a_chunk->count; - int b_count=b_chunk->count; + int a_countdown=a_chunk->count; + int b_countdown=b_chunk->count; bool a_break=false; bool b_break=false; while(true) { @@ -171,18 +171,18 @@ bool String::operator == (String& src) { if(size_diff==0) { // a has same size as b if(memcmp(a_row->item.ptr+a_offset, b_row->item.ptr+b_offset, a_row->item.size-a_offset)!=0) return false; - a_row++; a_count--; a_offset=0; - b_row++; b_count--; b_offset=0; + a_row++; a_countdown--; a_offset=0; + b_row++; b_countdown--; b_offset=0; } else if (size_diff>0) { // a longer if(memcmp(a_row->item.ptr+a_offset, b_row->item.ptr+b_offset, b_row->item.size-b_offset)!=0) return false; a_offset+=b_row->item.size-b_offset; - b_row++; b_count--; b_offset=0; + b_row++; b_countdown--; b_offset=0; } else { // b longer if(memcmp(a_row->item.ptr+a_offset, b_row->item.ptr+b_offset, a_row->item.size-a_offset)!=0) return false; b_offset+=a_row->item.size-a_offset; - a_row++; a_count--; a_offset=0; + a_row++; a_countdown--; a_offset=0; } a_break=a_row==a_end; @@ -190,15 +190,15 @@ bool String::operator == (String& src) { if(a_break || b_break) break; - if(!a_count) { + if(!a_countdown) { a_chunk=a_row->link; a_row=a_chunk->rows; - a_count=a_chunk->count; + a_countdown=a_chunk->count; } - if(!b_count) { + if(!b_countdown) { b_chunk=b_row->link; b_row=b_chunk->rows; - b_count=b_chunk->count; + b_countdown=b_chunk->count; } } return a_break==b_break;