--- parser3/src/classes/table.C	2005/11/25 10:47:26	1.208.6.9
+++ parser3/src/classes/table.C	2005/08/09 08:14:48	1.210
@@ -5,7 +5,7 @@
 	Author: Alexandr Petrosian <paf@design.ru> (http://paf.design.ru)
 */
 
-static const char * const IDENT_TABLE_C="$Date: 2005/11/25 10:47:26 $";
+static const char * const IDENT_TABLE_C="$Date: 2005/08/09 08:14:48 $";
 
 #include "classes.h"
 #include "pa_vmethod_frame.h"
@@ -21,7 +21,7 @@ static const char * const IDENT_TABLE_C=
 
 class MTable: public Methoded {
 public: // VStateless_class
-	Value* create_new_value(Pool&) { return new VTable(); }
+	Value* create_new_value(Pool&, HashStringValue&) { return new VTable(); }
 
 public:
 	MTable();
@@ -37,6 +37,8 @@ DECLARE_CLASS_VAR(table, new MTable, 0);
 // defines for globals
 
 #define SQL_BIND_NAME "bind"
+#define SQL_LIMIT_NAME "limit"
+#define SQL_OFFSET_NAME "offset"
 #define SQL_DEFAULT_NAME "default"
 #define SQL_DISTINCT_NAME "distinct"
 #define TABLE_REVERSE_NAME "reverse"
@@ -44,12 +46,18 @@ DECLARE_CLASS_VAR(table, new MTable, 0);
 // globals
 
 String sql_bind_name(SQL_BIND_NAME);
-String sql_limit_name(PA_SQL_LIMIT_NAME);
-String sql_offset_name(PA_SQL_OFFSET_NAME);
+String sql_limit_name(SQL_LIMIT_NAME);
+String sql_offset_name(SQL_OFFSET_NAME);
 String sql_default_name(SQL_DEFAULT_NAME);
 String sql_distinct_name(SQL_DISTINCT_NAME);
 String table_reverse_name(TABLE_REVERSE_NAME);
 
+// local defines
+
+#define COLUMN_SEPARATOR_NAME "separator"
+#define COLUMN_ENCLOSER_NAME "encloser"
+
+
 // methods
 
 static Table::Action_options get_action_options(Request& r, MethodParams& params, 
@@ -65,7 +73,6 @@ static Table::Action_options get_action_
 	]
 	but there is ^table.locate[name;value]
 
-	...if(voptions.is_defined() && !voptions.is_string()))
 	if(maybe_options.is_string()) { // allow empty options
 		result.defined=true;
 		return result;
@@ -139,11 +146,6 @@ static void _create(Request& r, MethodPa
 	size_t raw_pos_after=0;
 	Table::columns_type columns;
 	if(params.count()==2) {
-		const String& snameless=params.as_string(0, "called with two params, first param may only be string 'nameless'");
-		if(snameless!="nameless")
-			throw Exception("parser.runtime",
-				&snameless,
-				"table::create called with two params, first param may only be 'nameless'");
 		columns=Table::columns_type(0); // nameless
 	} else {
 		columns=Table::columns_type(new ArrayString);
@@ -268,27 +270,25 @@ struct TableSeparators {
 		column('\t'), scolumn(new String("\t", false)),
 		encloser(0), sencloser(0)
 	{}
-	int load( HashStringValue& options ) {
-		int result=0;
-		if(Value* vseparator=options.get(PA_COLUMN_SEPARATOR_NAME)) {
+	void load( HashStringValue& options ) {
+		if(Value* vseparator=options.get(COLUMN_SEPARATOR_NAME)) {
+			options.remove(COLUMN_SEPARATOR_NAME);
 			scolumn=&vseparator->as_string();
 			if(scolumn->length()!=1)
 				throw Exception("parser.runtime",
 					scolumn,
 					"separator must be one character long");
 			column=scolumn->first_char();
-			result++;
 		}
-		if(Value* vencloser=options.get(PA_COLUMN_ENCLOSER_NAME)) {
+		if(Value* vencloser=options.get(COLUMN_ENCLOSER_NAME)) {
+			options.remove(COLUMN_ENCLOSER_NAME);
 			sencloser=&vencloser->as_string();
 			if(sencloser->length()!=1)
 				throw Exception("parser.runtime",
 					sencloser,
 					"encloser must be one character long");
 			encloser=sencloser->first_char();
-			result++;
 		}
-		return result;
 	}
 };
 
@@ -334,7 +334,7 @@ static void _load(Request& r, MethodPara
 	Table& table=*new Table(columns);
 
 	// parse cells
-	Table::element_type row(new ArrayString(columns_count));
+	Table::element_type row(new ArrayString);
 	skip_empty_and_comment_lines(&data);
 	while( lsplit_result sr=lsplit(&data, separators.column, '\n', separators.encloser) ) {
 		if(!*sr.piece && !sr.delim && !row->count()) // append last empty column [if without \n]
@@ -342,7 +342,7 @@ static void _load(Request& r, MethodPara
 		*row+=new String(sr.piece, 0, true);
 		if(sr.delim=='\n') {
 			table+=row;
-			row=new ArrayString(columns_count);
+			row=new ArrayString;
 			skip_empty_and_comment_lines(&data);
 		}
 	}
@@ -361,7 +361,7 @@ static void maybe_enclose( String& to, c
 		size_t pos_after=0;
 		for( size_t pos_before; (pos_before=from.pos( encloser, pos_after ))!=STRING_NOT_FOUND; pos_after=pos_before+1) {
             to<<from.mid(pos_before+1/*+found encloser*/, pos_after-pos_before);
-			to<<*sencloser;  to<<*sencloser; // doubling encloser
+			to<<*sencloser; // doubling encloser
 		}
 		// last piece
 		size_t from_length=from.length();
@@ -394,8 +394,8 @@ static void _save(Request& r, MethodPara
 	TableSeparators separators;
 	if(param_index<params.count()) {
 		if(HashStringValue *options=params.as_no_junction(param_index++, "additional params must be hash").get_hash()) {
-			int valid_options=separators.load(*options);
-			if(valid_options!=options->count())
+			separators.load(*options);
+			if(options->count())
 				throw Exception("parser.runtime",
 					0,
 					"invalid option passed");
@@ -447,13 +447,8 @@ static void _save(Request& r, MethodPara
 	}
 
 	// write
-	{
-		const char* data_cstr=sdata.cstr();
-		file_write(r.absolute(file_name), 
-			data_cstr, sdata.length(), true, do_append);
-		if(*data_cstr) // not empty (when empty it's not heap memory)
-			pa_free((void*)data_cstr); // not needed anymore
-	}
+	file_write(r.absolute(file_name), 
+		sdata.cstr(), sdata.length(), true, do_append);
 }
 
 static void _count(Request& r, MethodParams&) {
@@ -482,8 +477,8 @@ static void _offset(Request& r, MethodPa
 					"is invalid whence, valid are 'cur' or 'set'");
 		}
 		
-		int offset=params.as_int(params.count()-1, "offset must be expression", r);
-		table.offset(absolute, offset);
+		Value& offset_expr=params.as_junction(params.count()-1, "offset must be expression");
+		table.offset(absolute, r.process_to_value(offset_expr).as_int());
 	} else
 		r.write_no_lang(*new VInt(table.current()));
 }
@@ -846,7 +841,6 @@ static void _join(Request& r, MethodPara
 #ifndef DOXYGEN
 class Table_sql_event_handlers: public SQL_Driver_query_event_handlers {
 	ArrayString& columns;
-	int columns_count;
 	ArrayString* row;
 public:
 	Table* table;
@@ -867,7 +861,6 @@ public:
 	bool before_rows(SQL_Error& error) { 
 		try {
 			table=new Table(&columns);
-			columns_count=columns.count();
 			return false;
 		} catch(...) {
 			error=SQL_Error("exception occured in Table_sql_event_handlers::before_rows");
@@ -876,7 +869,7 @@ public:
 	}
 	bool add_row(SQL_Error& error) {
 		try {
-			*table+=row=new ArrayString(columns_count);
+			*table+=row=new ArrayString;
 			return false;
 		} catch(...) {
 			error=SQL_Error("exception occured in Table_sql_event_handlers::add_row");
@@ -946,7 +939,7 @@ static void _sql(Request& r, MethodParam
 	ulong offset=0;
 	if(params.count()>1) {
 		Value& voptions=params.as_no_junction(1, "options must be hash, not code");
-		if(voptions.is_defined() && !voptions.is_string())
+		if(!voptions.is_string())
 			if(HashStringValue* options=voptions.get_hash()) {
 				int valid_options=0;
 				if(Value* vbind=options->get(sql_bind_name)) {