--- parser3/src/main/pa_table.C	2007/04/23 10:30:32	1.62
+++ parser3/src/main/pa_table.C	2024/09/07 16:30:27	1.75
@@ -1,49 +1,63 @@
 /** @file
 	Parser: table class.
 
-	Copyright (c) 2001-2005 ArtLebedev Group (http://www.artlebedev.com)
-	Author: Alexandr Petrosian <paf@design.ru> (http://paf.design.ru)
+	Copyright (c) 2001-2023 Art. Lebedev Studio (http://www.artlebedev.com)
+	Authors: Konstantin Morshnev <moko@design.ru>, Alexandr Petrosian <paf@design.ru>
 */
 
-static const char * const IDENT_TABLE_C="$Date: 2007/04/23 10:30:32 $";
-
 #include "pa_table.h"
 
-#include "pa_exception.h"
+volatile const char * IDENT_PA_TABLE_C="$Id: pa_table.C,v 1.75 2024/09/07 16:30:27 moko Exp $" IDENT_PA_TABLE_H;
 
-Table::Table(columns_type acolumns, size_t initial_rows):
-	Array<element_type>(initial_rows),
+#include "pa_exception.h"
 
-	fcurrent(0),
-	fcolumns(acolumns), 
-	name2number(new name2number_hash_class) {
+Table::Table(columns_type acolumns, size_t initial_rows): Array<element_type>(initial_rows), fcurrent(0), fcolumns(acolumns), name2number(NULL){
+	column_names_init();
+}
 
-	if(fcolumns) {
+void Table::column_names_init(){
+	if(fcolumns){
+		name2number = new name2number_hash_class;
 		size_t number=1;
-		for(Array_iterator<const String*> i(*fcolumns); i.has_next(); ) {
+		for(ArrayString::Iterator i(*fcolumns); i; ) {
 			const String& name=*i.next();
 			name2number->put(name, number++);
 		}
 	}
 }
 
+static void append_row(Table& src, Table* dest) {
+	Table::element_type src_row(src[src.current()]);
+	Table::element_type row(new ArrayString(src_row->count()));
+	row->append(*src_row);
+	*dest+=row;
+}
+
 Table::Table(const Table& src, Action_options& options) :
-	Array<element_type>(options.limit==ARRAY_OPTION_LIMIT_ALL?0:options.limit/*may be more than needed, no harm done*/),
+	Array<element_type>( (options.limit==ARRAY_OPTION_LIMIT_ALL || options.limit>src.count()) ?  src.count() : options.limit),
 
 	fcurrent(0),
 	fcolumns(src.fcolumns),
 	name2number(src.name2number) {
 
-	append(src, options.offset, options.limit, options.reverse);
+	((Table &)src).table_for_each(append_row, this, options);
+}
+
+size_t Table::max_cells() const {
+	size_t result=0;
+	for(size_t i=0; i<count(); i++){
+		element_type row=get(i);
+		if(row->count()>result)
+			result=row->count();
+	}
+	return result;
 }
 
 int Table::column_name2index(const String& column_name, bool bark) const {
 	if(fcolumns) {// named
 		int result=name2number->get(column_name)-1; // -1 = column not found
 		if(bark && result<0)
-			throw Exception(PARSER_RUNTIME,
-				&column_name,
-				"column not found");
+			throw Exception(PARSER_RUNTIME, &column_name, "column not found");
 		return result;
 	} else // nameless
 		return column_name.as_int();
@@ -58,6 +72,27 @@ const String* Table::item(size_t column)
 	return 0; // it's OK we don't have row|column, just return nothing
 }
 
+void Table::put_item(size_t column, const String* value) {
+	if(!valid(fcurrent)) {
+		throw Exception(PARSER_RUNTIME, 0, "invalid current row");
+	}
+	element_type row=get(fcurrent);
+	while(row->count()<=column){
+	    *row+=&String::Empty;
+	}
+	row->put(column, value);
+}
+
+void Table::remove_current(){
+	if(!valid(fcurrent)) {
+		throw Exception(PARSER_RUNTIME, 0, "invalid current row");
+	}
+	remove(fcurrent);
+	if(fcurrent==count() && count()>0){
+	    fcurrent--;
+	}
+}
+
 #ifndef DOXYGEN
 struct Locate_int_string_info {
 	int column;
@@ -69,18 +104,17 @@ bool locate_int_string(Table& self, Loca
 	return item_value && *item_value==*info->value;
 }
 
-bool Table::locate(int column, const String& value,
-		   Table::Action_options& options) {
+bool Table::locate(int column, const String& value, Table::Action_options& options) {
 	Locate_int_string_info info={column, &value};
 	return table_first_that(locate_int_string, &info, options);
 }
 
-bool Table::locate(const String& column, const String& value, 
-		   Table::Action_options& options) {
+bool Table::locate(const String& column, const String& value, Table::Action_options& options) {
 	return locate(column_name2index(column, true), value, options);
 }
 
 void Table::offset(bool absolute, int offset) {
+	// not +lcount, but "if either operand is unsigned, the other shall be converted to unsigned" C++ rule works here
 	if(size_t lcount=count())
 		fcurrent=((absolute?0:fcurrent)+offset+lcount)%lcount;
 }