--- parser3/src/classes/table.C	2011/11/19 04:15:31	1.282
+++ parser3/src/classes/table.C	2015/04/06 22:27:25	1.302
@@ -1,12 +1,10 @@
 /** @file
 	Parser: @b table parser class.
 
-	Copyright (c) 2001-2009 ArtLebedev Group (http://www.artlebedev.com)
+	Copyright (c) 2001-2012 Art. Lebedev Studio (http://www.artlebedev.com)
 	Author: Alexandr Petrosian <paf@design.ru> (http://paf.design.ru)
 */
 
-static const char * const IDENT_TABLE_C="$Date: 2011/11/19 04:15:31 $";
-
 #if (!defined(NO_STRINGSTREAM) && !defined(FREEBSD4))
 #include <sstream>
 #endif
@@ -23,6 +21,8 @@ static const char * const IDENT_TABLE_C=
 #include "pa_vbool.h"
 #include "pa_array.h"
 
+volatile const char * IDENT_TABLE_C="$Id: table.C,v 1.302 2015/04/06 22:27:25 moko Exp $";
+
 // class
 
 class MTable: public Methoded {
@@ -56,20 +56,7 @@ static Table::Action_options get_action_
 	if(params.count() <= options_index)
 		return result;
 
-	Value& maybe_options=params[options_index];
-/* can not do it: 
-	want to enable ^table::create[$source;
-#		$.option[]
-	]
-	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;
-	}
-*/
-	HashStringValue* options=maybe_options.get_hash();
+	HashStringValue* options=params.as_hash(options_index);
 	if(!options)
 		return result;
 
@@ -137,11 +124,15 @@ struct TableSeparators {
 		}
 		if(Value* vencloser=options.get(PA_COLUMN_ENCLOSER_NAME)) {
 			sencloser=&vencloser->as_string();
+			if(sencloser->is_empty()){
+				encloser=0;
+			} else {
 			if(sencloser->length()!=1)
 				throw Exception(PARSER_RUNTIME,
 					sencloser,
 					"encloser must be one character long");
 			encloser=sencloser->first_char();
+			}
 			result++;
 		}
 		return result;
@@ -179,7 +170,7 @@ static void _create(Request& r, MethodPa
 	size_t options_param_index=data_param_index+1;
 	if(
 		options_param_index<params.count()
-		&& (options=params.as_no_junction(options_param_index, "additional params must be hash").get_hash())
+		&& (options=params.as_hash(options_param_index))
 	) {
 		// cloning, so that we could change
 		options=new HashStringValue(*options);
@@ -194,7 +185,7 @@ static void _create(Request& r, MethodPa
 	// data
 	Temp_lang temp_lang(r, String::L_PASS_APPENDED);
 	const String&  data=
-		r.process_to_string(params.as_junction(data_param_index, "body must be code"));
+		r.process_to_string(params.as_junction(data_param_index, "body must be table or code"));
 
 	// parse columns
 	size_t raw_pos_after=0;
@@ -209,7 +200,10 @@ static void _create(Request& r, MethodPa
 		data.split(head, raw_pos_after, "\n", String::L_AS_IS, 1);
 		if(head.count()) {
 			size_t col_pos_after=0;
-			head[0]->split(*columns, col_pos_after, *separators.scolumn, String::L_AS_IS);
+			if(head[0]->is_empty())
+				*columns += new String();
+			else
+				head[0]->split(*columns, col_pos_after, *separators.scolumn, String::L_AS_IS);
 		}
 	}
 
@@ -278,20 +272,24 @@ static lsplit_result lsplit(char** strin
 			char *write;
 			write=read=string;
 			char c;
-			while((c=*read++)) {
+			// we are enclosed, searching for second encloser
+			while(c=*read++) {
 				if(c==encloser) {
-					char n=*read;
-					if(n==encloser) // double-encloser stands for encloser
-						read++;
-					else if(n==delim1 || n==delim2) {
-						result.delim=n;
+					if(*read==encloser) // double-encloser stands for encloser
 						read++;
-						break;
-					}
+					else
+						break; // note: skipping encloser
 				}
-
 				*write++=c;
 			}
+			// we are no longer enclosed, searching for delimiter, skipping extra enclosers
+			while(c=*read++) {
+				if(c==delim1 || c==delim2) {
+					result.delim=c;
+					break;
+				} else 	if(c!=encloser)
+					*write++=c;
+			}
 			*write=0; // terminate
 			*string_ref=c? read: 0;
 			result.piece=string;
@@ -344,7 +342,7 @@ static void _load(Request& r, MethodPara
 	HashStringValue *options=0;
 	TableSeparators separators;
 	if(options_param_index<params.count()
-		&& (options=params.as_no_junction(options_param_index, "additional params must be hash").get_hash())) {
+		&& (options=params.as_hash(options_param_index))) {
 		// cloning, so that we could change [to simplify checks on params inside file_read_text
 		options=new HashStringValue(*options);
 		separators.load(*options);
@@ -399,56 +397,173 @@ static void _load(Request& r, MethodPara
 }
 
 #if (!defined(NO_STRINGSTREAM) && !defined(FREEBSD4))
+	#define USE_STRINGSTREAM
+#endif
 
+#ifdef USE_STRINGSTREAM
 #include "gc_allocator.h"
 
 typedef std::basic_stringstream<char, std::char_traits<char>, gc_allocator<char> > pa_stringstream;
 typedef std::basic_string<char, std::char_traits<char>, gc_allocator<char> > pa_string;
 
-void maybe_enclose( pa_stringstream& to, const String& from, char encloser ) {
-	if(encloser) {
+static void enclose( pa_stringstream& to, const String* from, char encloser ) {
+	if(from){
 		to<<encloser;
 		// while we have 'encloser'...
 		size_t pos_after=0;
-		for( size_t pos_before; (pos_before=from.pos( encloser, pos_after ))!=STRING_NOT_FOUND; pos_after=pos_before) {
+		for( size_t pos_before; (pos_before=from->pos( encloser, pos_after ))!=STRING_NOT_FOUND; pos_after=pos_before) {
 			pos_before++; // including first encloser (and skipping it for next pos)
-			to<<from.mid(pos_after, pos_before).cstr();
+			to<<from->mid(pos_after, pos_before).cstr();
 			to<<encloser; // doubling encloser
 		}
 		// last piece
-		size_t from_length=from.length();
+		size_t from_length=from->length();
 		if(pos_after<from_length)
-			to<<from.mid(pos_after, from_length).cstr();
-
+			to<<from->mid(pos_after, from_length).cstr();
 		to<<encloser;
-	} else
-		to<<from.cstr();
+	} else {
+		to<<encloser<<encloser;
+	}
+}
+
+static void table_to_csv(pa_stringstream& result, Table& table, TableSeparators& separators, bool output_column_names) {
+	if(output_column_names) {
+		if(table.columns()) { // named table
+			if(separators.encloser){
+				for(Array_iterator<const String*> i(*table.columns()); i.has_next(); ) {
+					enclose( result, i.next(), separators.encloser );
+					if(i.has_next())
+						result<<separators.column;
+				}
+			} else {
+				for(Array_iterator<const String*> i(*table.columns()); i.has_next(); ) {
+					result<<i.next()->cstr();
+					if(i.has_next())
+						result<<separators.column;
+				}
+			}
+		} else { // nameless table [we were asked to output column names]
+			if(int lsize=table.count()?table[0]->count():0)
+				for(int column=0; column<lsize; column++) {
+					if(separators.encloser) {
+						result<<separators.encloser<<column<<separators.encloser;
+					} else {
+						result<<column;
+					}
+					if(column<lsize-1){
+						result<<separators.column;
+					}
+				}
+			else
+				result<<"empty nameless table";
+		}
+		result<<'\n';
+	}
+
+	// process data lines
+	Array_iterator<ArrayString*> i(table);
+	if(separators.encloser){
+		while(i.has_next()) {
+			for(Array_iterator<const String*> c(*i.next()); c.has_next(); ) {
+				enclose( result, c.next(), separators.encloser );
+				if(c.has_next())
+					result<<separators.column;
+			}
+			result<<'\n';
+		}
+	} else {
+		while(i.has_next()) {
+			for(Array_iterator<const String*> c(*i.next()); c.has_next(); ) {
+				result<<c.next()->cstr();
+				if(c.has_next())
+					result<<separators.column;
+			}
+			result<<'\n';
+		}
+	}
 }
 
 #else
 
-void maybe_enclose( String& to, const String& from, char encloser, const String* sencloser ) {
-	if(encloser) {
+void enclose( String& to, const String* from, char encloser, const String* sencloser ) {
+	if(from){
 		to<<*sencloser;
 		// while we have 'encloser'...
 		size_t pos_after=0;
-		for( size_t pos_before; (pos_before=from.pos( encloser, pos_after ))!=STRING_NOT_FOUND; pos_after=pos_before) {
+		for( size_t pos_before; (pos_before=from->pos( encloser, pos_after ))!=STRING_NOT_FOUND; pos_after=pos_before) {
 			pos_before++; // including first encloser (and skipping it for next pos)
-			to<<from.mid(pos_after, pos_before);
+			to<<from->mid(pos_after, pos_before);
 			to<<*sencloser; // doubling encloser
 		}
 		// last piece
-		size_t from_length=from.length();
+		size_t from_length=from->length();
 		if(pos_after<from_length)
-			to<<from.mid(pos_after, from_length);
-
+			to<<from->mid(pos_after, from_length);
 		to<<*sencloser;
-	} else
-		to<<from;
+	} else {
+		to<<*sencloser<<*sencloser;
+	}
 }
 
+static void table_to_csv(String& result, Table& table, TableSeparators& separators, bool output_column_names) {
+	if(output_column_names) {
+		if(table.columns()) { // named table
+			if(separators.encloser) {
+				for(Array_iterator<const String*> i(*table.columns()); i.has_next(); ) {
+					enclose( result, i.next(), separators.encloser, separators.sencloser );
+					if(i.has_next())
+						result<<*separators.scolumn;
+				}
+			} else {
+				for(Array_iterator<const String*> i(*table.columns()); i.has_next(); ) {
+					result<<*i.next();
+					if(i.has_next())
+						result<<*separators.scolumn;
+				}
+			}
+		} else { // nameless table [we were asked to output column names]
+			if(int lsize=table.count()?table[0]->count():0)
+				for(int column=0; column<lsize; column++) {
+					if(separators.encloser) {
+						result<<*separators.sencloser<<String::Body::Format(column)<<*separators.sencloser;
+					} else {
+						result<<String::Body::Format(column);
+					}
+					if(column<lsize-1){
+						result<<*separators.scolumn;
+					}
+				}
+			else
+				result.append_help_length("empty nameless table", 0, String::L_CLEAN);
+		}
+		result.append_know_length("\n", 1, String::L_CLEAN);
+	}
+
+	// data lines
+	Array_iterator<ArrayString*> i(table);
+	if(separators.encloser){
+		while(i.has_next()) {
+			for(Array_iterator<const String*> c(*i.next()); c.has_next(); ) {
+				enclose( result, c.next(), separators.encloser, separators.sencloser );
+				if(c.has_next())
+					result<<*separators.scolumn;
+			}
+			result.append_know_length("\n", 1, String::L_CLEAN);
+		}
+	} else {
+		while(i.has_next()) {
+			for(Array_iterator<const String*> c(*i.next()); c.has_next(); ) {
+				result<<*c.next();
+				if(c.has_next())
+					result<<*separators.scolumn;
+			}
+			result.append_know_length("\n", 1, String::L_CLEAN);
+		}
+	}
+}
 #endif // don't use stringstream
 
+
 static void _save(Request& r, MethodParams& params) {
 	const String& first_arg=params.as_string(0, FIRST_ARG_MUST_NOT_BE_CODE);
 	size_t param_index=1;
@@ -472,10 +587,7 @@ static void _save(Request& r, MethodPara
 
 	TableSeparators separators;
 	if(param_index<params.count())
-		if(HashStringValue* options=params.as_hash(param_index++, 
-				"additional params must be hash",
-				"additional params must be hash (did you spell mode parameter correctly?)"
-		)) {
+		if(HashStringValue* options=params.as_hash(param_index++)) {
 			int valid_options=separators.load(*options);
 			if(valid_options!=options->count())
 				throw Exception(PARSER_RUNTIME, 0, CALLED_WITH_INVALID_OPTION);
@@ -488,101 +600,65 @@ static void _save(Request& r, MethodPara
 
 	Table& table=GET_SELF(r, VTable).table();
 
-#if (!defined(NO_STRINGSTREAM) && !defined(FREEBSD4))
-
+#ifdef USE_STRINGSTREAM
 	pa_stringstream ost(std::stringstream::out);
 
-	// process header
-	if(output_column_names) {
-		if(table.columns()) { // named table
-			for(Array_iterator<const String*> i(*table.columns()); i.has_next(); ) {
-				maybe_enclose( ost, *i.next(), separators.encloser );
-				if(i.has_next()){
-					ost<<separators.column;
-				}
-			}
-		} else { // nameless table [we were asked to output column names]
-			if(int lsize=table.count()?table[0]->count():0)
-				for(int column=0; column<lsize; column++) {
-					if(separators.encloser) {
-						ost<<separators.encloser<<column<<separators.encloser;
-					} else {
-						ost<<column;
-					}
-					if(column<lsize-1){
-						ost<<separators.column;
-					}
-				}
-			else
-				ost<<"empty nameless table";
-		}
-		ost<<'\n';
-	}
-
-	// process data lines
-	Array_iterator<ArrayString*> i(table);
-	while(i.has_next()) {
-		for(Array_iterator<const String*> c(*i.next()); c.has_next(); ) {
-			maybe_enclose( ost, *c.next(), separators.encloser );
-			if(c.has_next())
-				ost<<separators.column;
-		}
-		ost<<'\n';
-	}
+	table_to_csv(ost, table, separators, output_column_names);
 
 	// write
-	{
 		pa_string data=ost.str();
 		const char* data_cstr=data.c_str();
-
 		file_write(r.charsets, file_spec, data_cstr, data.length(), true /* as text */, do_append);
-	}
-
 #else
-
 	String sdata;
-	if(output_column_names) {
-		if(table.columns()) { // named table
-			for(Array_iterator<const String*> i(*table.columns()); i.has_next(); ) {
-				maybe_enclose( sdata, *i.next(), separators.encloser, separators.sencloser );
-				if(i.has_next())
-					sdata<<*separators.scolumn;
-			}
-		} else { // nameless table [we were asked to output column names]
-			if(int lsize=table.count()?table[0]->count():0)
-				for(int column=0; column<lsize; column++) {
-					char *cindex_tab=new(PointerFreeGC) char[MAX_NUMBER];
-					sdata.append_know_length(cindex_tab, 
-						snprintf(cindex_tab, MAX_NUMBER, 
-							column<lsize-1?"%d%c":"%d", column, separators.column),
-							String::L_CLEAN);
-				}
-			else
-				sdata.append_help_length("empty nameless table", 0, String::L_CLEAN);
-		}
-		sdata.append_know_length("\n", 1, String::L_CLEAN);
-	}
 
-	// data lines
-	Array_iterator<ArrayString*> i(table);
-	while(i.has_next()) {
-		for(Array_iterator<const String*> c(*i.next()); c.has_next(); ) {
-			maybe_enclose( sdata, *c.next(), separators.encloser, separators.sencloser );
-			if(c.has_next())
-				sdata<<*separators.scolumn;
-		}
-		sdata.append_know_length("\n", 1, String::L_CLEAN);
-	}
+	table_to_csv(sdata, table, separators, output_column_names);
 
 	// write
-	{
 		const char* data_cstr=sdata.cstr();
-		file_write(r.charsets, file_spec, data_cstr, sdata.length(), true, do_append);
+	file_write(r.charsets, file_spec, data_cstr, sdata.length(), true /* as text */, do_append);
 		if(*data_cstr) // not empty (when empty it's not heap memory)
 			pa_free((void*)data_cstr); // not needed anymore
+#endif
+}
+
+static void _csv_string(Request& r, MethodParams& params) {
+	bool output_column_names=true;
+	size_t param_index=0;
+	if(params.count()>0 && params[0].is_string()) {
+		if(params.as_string(0, FIRST_ARG_MUST_NOT_BE_CODE)=="nameless") {
+			output_column_names=false;
+			param_index++;
+		} else {
+			throw Exception(PARSER_RUNTIME,
+				0,
+				"bad mode (must be nameless)");
+		}
 	}
 
-#endif // don't use stringstream
+	TableSeparators separators;
+	if(param_index<params.count())
+		if(HashStringValue* options=params.as_hash(param_index++)) {
+			int valid_options=separators.load(*options);
+			if(valid_options!=options->count())
+				throw Exception(PARSER_RUNTIME, 0, CALLED_WITH_INVALID_OPTION);
+		}
+
+	Table& table=GET_SELF(r, VTable).table();
+
+#ifdef USE_STRINGSTREAM
+	pa_stringstream ost(std::stringstream::out);
+
+	table_to_csv(ost, table, separators, output_column_names);
+
+	r.write_no_lang(*new VString(*new String(pa_strdup(ost.str().c_str()), String::L_CLEAN)));
+#else
+	String sdata;
+
+	table_to_csv(sdata, table, separators, output_column_names);
+
+	r.write_no_lang(*new VString(*new String(sdata.cstr(), String::L_CLEAN)));
+#endif
 }
 
 static void _count(Request& r, MethodParams& params) {
@@ -639,12 +715,12 @@ static void _menu(Request& r, MethodPara
 	Value* delim_maybe_code=params.count()>1?&params[1]:0;
 
 	Table& table=GET_SELF(r, VTable).table();
-	int saved_current=table.current();
-	int size=table.count();
+	size_t saved_current=table.current();
+	size_t size=table.count();
 
 	if(delim_maybe_code) { // delimiter set
 		bool need_delim=false;
-		for(int row=0; row<size; row++) {
+		for(size_t row=0; row<size; row++) {
 			table.set_current(row);
 
 			StringOrValue sv_processed=r.process(body_code);
@@ -664,7 +740,7 @@ static void _menu(Request& r, MethodPara
 				break;
 		}
 	} else {
-		for(int row=0; row<size; row++) {
+		for(size_t row=0; row<size; row++) {
 			table.set_current(row);
  
 			r.process_write(body_code);
@@ -779,7 +855,7 @@ static void _hash(Request& r, MethodPara
 			Table2hash_value_type value_type=C_HASH;
 			int param_index=params.count()-1;
 			if(param_index>0) {
-				if(HashStringValue* options=params.as_no_junction(param_index, PARAM_MUST_NOT_BE_CODE).get_hash()){ // options where specified
+				if(HashStringValue* options=params.as_no_junction(param_index, PARAM_MUST_NOT_BE_CODE).get_hash()){ // can't use .as_has because the 2nd param could be table so .as_hash throws an error
 					--param_index;
 					int valid_options=0;
 					if(Value* vdistinct_code=options->get(sql_distinct_name)) { // $.distinct ?
@@ -815,7 +891,7 @@ static void _hash(Request& r, MethodPara
 			}
 
 			if(param_index==2) // options was specified but not as hash
-				throw Exception(PARSER_RUNTIME, 0, OPTIONS_MUST_BE_HASH);
+				throw Exception(PARSER_RUNTIME, 0, "options must be hash");
 
 			Array<int> value_fields;
 			if(param_index==0){ // list of columns wasn't specified
@@ -924,14 +1000,14 @@ static void _sort(Request& r, MethodPara
 		old_table.set_current(i);
 		// calculate key value
 		seq[i].row=old_table[i];
-		Value& value=r.process_to_value(key_maker).as_expr_result(true/*return string as-is*/);
+		Value& value=r.process_to_value(key_maker);
 		if(i==0) // determining key values type by first one
 			key_values_are_strings=value.is_string();
 
 		if(key_values_are_strings)
 			seq[i].value.c_str=value.as_string().cstr();
 		else
-			seq[i].value.d=value.as_double();
+			seq[i].value.d=value.as_expr_result().as_double();
 	}
 
 	// @todo: handle this elsewhere
@@ -942,8 +1018,7 @@ static void _sort(Request& r, MethodPara
 	}
 
 	// sort keys
-	_qsort(seq, old_count, sizeof(Table_seq_item), 
-		key_values_are_strings?sort_cmp_string:sort_cmp_double);
+	qsort(seq, old_count, sizeof(Table_seq_item), key_values_are_strings?sort_cmp_string:sort_cmp_double);
 
 	// reorder table as they require in 'seq'
 	for(i=0; i<old_count; i++)
@@ -989,7 +1064,7 @@ static bool _locate_name_value(Table& ta
 static void _locate(Request& r, MethodParams& params) {
 	Table& table=GET_SELF(r, VTable).table();
 
-	bool result=params[0].get_junction()?
+	bool result=params[0].get_junction() || (params.count() == 1) ?
 		_locate_expression(table, r, params) :
 		_locate_name_value(table, r, params);
 	r.write_no_lang(VBool::get(result));
@@ -1013,6 +1088,70 @@ static void _flip(Request& r, MethodPara
 	r.write_no_lang(*new VTable(&new_table));
 }
 
+static void _foreach(Request& r, MethodParams& params) {
+	InCycle temp(r);
+
+	const String& rownum_name=params.as_string(0, "rownum-var name must be string");
+	const String& value_name=params.as_string(1, "value-var name must be string");
+
+	Value& body_code=params.as_junction(2, "body must be code");
+	
+	Value* delim_maybe_code=params.count()>3?&params[3]:0;
+
+	Table& table=GET_SELF(r, VTable).table();
+	size_t saved_current=table.current();
+	size_t size=table.count();
+
+	const String* rownum_var_name=rownum_name.is_empty()? 0 : &rownum_name;
+	const String* value_var_name=value_name.is_empty()? 0 : &value_name;
+
+	Value* var_context=r.get_method_frame()->caller();
+
+	if(delim_maybe_code) { // delimiter set
+		bool need_delim=false;
+		for(size_t row=0; row<size; row++) {
+			table.set_current(row);
+
+			if(rownum_var_name)
+				r.put_element(*var_context, *rownum_var_name, new VString(*new String(String::Body::Format(row), String::L_CLEAN)));
+			if(value_var_name)
+				r.put_element(*var_context, *value_var_name, new VTable(&table));
+
+			StringOrValue sv_processed=r.process(body_code);
+			Request::Skip lskip=r.get_skip(); r.set_skip(Request::SKIP_NOTHING);
+
+			const String* s_processed=sv_processed.get_string();
+			if(s_processed && !s_processed->is_empty()) { // we have body
+				if(need_delim) // need delim & iteration produced string?
+					r.write_pass_lang(r.process(*delim_maybe_code));
+				else
+					need_delim=true;
+			}
+
+			r.write_pass_lang(sv_processed);
+
+			if(lskip==Request::SKIP_BREAK)
+				break;
+		}
+	} else {
+		for(size_t row=0; row<size; row++) {
+			table.set_current(row);
+ 
+			if(rownum_var_name)
+				r.put_element(*var_context, *rownum_var_name, new VString(*new String(String::Body::Format(row), String::L_CLEAN)));
+			if(value_var_name)
+				r.put_element(*var_context, *value_var_name, new VTable(&table));
+
+			r.process_write(body_code);
+			Request::Skip lskip=r.get_skip(); r.set_skip(Request::SKIP_NOTHING);
+ 
+			if(lskip==Request::SKIP_BREAK)
+				break;
+		}
+	}
+	table.set_current(saved_current);
+}
+
 static void _append(Request& r, MethodParams& params) {
 	Temp_lang temp_lang(r, String::L_PASS_APPENDED);
 	const String& string=r.process_to_string(params[0]);
@@ -1039,16 +1178,10 @@ static void join_nameless_row(Table& src
 	*dest+=src[src.current()];
 }
 static void _join(Request& r, MethodParams& params) {
-	Table* maybe_src=params.as_no_junction(0, "table ref must not be code").get_table();
-	if(!maybe_src)
-		throw Exception(PARSER_RUNTIME, 
-			0, 
-			"source is not a table");
-	Table& src=*maybe_src;
+	Table& src=*params.as_table(0, "source");
 
 	Table::Action_options o=get_action_options(r, params, 1, src);
-	check_option_param(o.defined, params, 1,
-		"invalid extra parameter");
+	check_option_param(o.defined, params, 1, "invalid extra parameter");
 
 	Table& dest=GET_SELF(r, VTable).table();
 	if(&src == &dest)
@@ -1104,7 +1237,7 @@ public:
 	}
 	bool add_row_cell(SQL_Error& error, const char* str, size_t ) {
 		try {
-			*row+=new String(str, String::L_TAINTED /* no length as 0x00 can be inside */);
+			*row+=str?new String(str, String::L_TAINTED /* no length as 0x00 can be inside */):&String::Empty;
 			return false;
 		} catch(...) {
 			error=SQL_Error("exception occured in Table_sql_event_handlers::add_row_cell");
@@ -1131,7 +1264,7 @@ static void marshal_bind(
 // not static, used elsewhere
 int marshal_binds(HashStringValue& hash, SQL_Driver::Placeholder*& placeholders) {
 	int hash_count=hash.count();
-	placeholders=new(UseGC) SQL_Driver::Placeholder[hash_count];
+	placeholders=new SQL_Driver::Placeholder[hash_count];
 	SQL_Driver::Placeholder* ptr=placeholders;
 	hash.for_each<SQL_Driver::Placeholder**>(marshal_bind, &ptr);
 	return hash_count;
@@ -1158,7 +1291,7 @@ static void _sql(Request& r, MethodParam
 	ulong limit=SQL_NO_LIMIT;
 	ulong offset=0;
 	if(params.count()>1)
-		if(HashStringValue* options=params.as_hash(1)) {
+		if(HashStringValue* options=params.as_hash(1, "sql options")) {
 			int valid_options=0;
 			if(Value* vbind=options->get(sql_bind_name)) {
 				valid_options++;
@@ -1186,11 +1319,7 @@ static void _sql(Request& r, MethodParam
 	const char* statement_cstr=statement_string.untaint_cstr(r.flang, r.connection());
 
 	Table_sql_event_handlers handlers;
-#ifdef RESOURCES_DEBUG
-	struct timeval mt[2];
-	//measure:before
-	gettimeofday(&mt[0],NULL);
-#endif	
+
 	r.connection()->query(
 		statement_cstr, 
 		placeholders_count, placeholders,
@@ -1198,17 +1327,6 @@ static void _sql(Request& r, MethodParam
 		handlers,
 		statement_string);
 	
-#ifdef RESOURCES_DEBUG
-		//measure:after connect
-	gettimeofday(&mt[1],NULL);
-	
-	double t[2];
-	for(int i=0;i<2;i++)
-	    t[i]=mt[i].tv_sec+mt[i].tv_usec/1000000.0;
-	    
-	r.sql_request_time+=t[1]-t[0];
-#endif	    			
-
 	if(bind)
 		unmarshal_bind_updates(*bind, placeholders_count, placeholders);
 
@@ -1268,14 +1386,15 @@ static void _select(Request& r, MethodPa
 				reverse=r.process_to_value(*vreverse).as_bool();
 			}
 			if(valid_options!=options->count())
-				throw Exception(PARSER_RUNTIME,
-					0,
-					"called with invalid option");
+				throw Exception(PARSER_RUNTIME, 0, CALLED_WITH_INVALID_OPTION);
 		}
 
 	Table& result_table=*new Table(source_table.columns());
 
-	if(size_t size=source_table.count() && limit>0){
+	size_t size=source_table.count();
+	if(offset<0)
+		offset+=size;
+	if(size && limit>0 && offset>=0 && (size_t)offset<size){
 		size_t saved_current=source_table.current();
 		size_t appended=0;
 
@@ -1325,6 +1444,11 @@ MTable::MTable(): Methoded("table") {
 
 	// add_native_method("save_old", Method::CT_DYNAMIC, _save_old, 1, 3);
 
+	// ^table.csv-string[]
+	// ^table.csv-string[nameless]
+	// ^table.csv-string[nameless;$.encloser["] $.separator[,]]
+	add_native_method("csv-string", Method::CT_DYNAMIC, _csv_string, 0, 2);
+
 	// ^table.count[]
 	// ^table.count[rows]
 	// ^table.count[columns]
@@ -1357,6 +1481,10 @@ MTable::MTable(): Methoded("table") {
 	// ^table.flip[]
 	add_native_method("flip", Method::CT_DYNAMIC, _flip, 0, 0);
 
+	// ^table.foreach[row-num;value]{code}
+	// ^table.foreach[row-num;value]{code}[delim]
+	add_native_method("foreach", Method::CT_DYNAMIC, _foreach, 3, 4);
+
 	// ^table.append{r{tab}e{tab}c{tab}o{tab}r{tab}d}
 	add_native_method("append", Method::CT_DYNAMIC, _append, 1, 1);