--- parser3/src/classes/table.C	2005/11/18 11:00:03	1.208.6.6
+++ 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/18 11:00:03 $";
+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) {
-		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;
 	}
 };
 
@@ -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");
@@ -477,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()));
 }
@@ -939,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)) {