--- parser3/src/classes/table.C	2015/04/06 22:27:25	1.302
+++ parser3/src/classes/table.C	2015/07/28 14:42:44	1.306
@@ -5,10 +5,6 @@
 	Author: Alexandr Petrosian <paf@design.ru> (http://paf.design.ru)
 */
 
-#if (!defined(NO_STRINGSTREAM) && !defined(FREEBSD4))
-#include <sstream>
-#endif
-
 #include "classes.h"
 #include "pa_vmethod_frame.h"
 
@@ -21,7 +17,12 @@
 #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 $";
+#if (!defined(NO_STRINGSTREAM) && !defined(FREEBSD4))
+#include <sstream>
+#define USE_STRINGSTREAM
+#endif
+
+volatile const char * IDENT_TABLE_C="$Id: table.C,v 1.306 2015/07/28 14:42:44 moko Exp $";
 
 // class
 
@@ -94,14 +95,6 @@ static Table::Action_options get_action_
 
 	return result;
 }
-static void check_option_param(bool options_defined, 
-			  MethodParams& params, size_t next_param_index,
-			  const char *msg) {
-	if(next_param_index+(options_defined?1:0) != params.count())
-		throw Exception(PARSER_RUNTIME,
-			0,
-			"%s", msg);
-}
 
 struct TableSeparators {
 	char column;  const String* scolumn;
@@ -143,8 +136,8 @@ static void _create(Request& r, MethodPa
 	// clone/copy part?
 	if(Table *source=params[0].get_table()) {
 		Table::Action_options o=get_action_options(r, params, 1, *source);
-		check_option_param(o.defined, params, 1, 
-			"too many parameters");
+		if(params.count()>2)
+			throw Exception(PARSER_RUNTIME, 0, "too many parameters");
 		GET_SELF(r, VTable).set_table(*new Table(*source, o));
 		return;
 	}
@@ -396,10 +389,6 @@ static void _load(Request& r, MethodPara
 	GET_SELF(r, VTable).set_table(table);
 }
 
-#if (!defined(NO_STRINGSTREAM) && !defined(FREEBSD4))
-	#define USE_STRINGSTREAM
-#endif
-
 #ifdef USE_STRINGSTREAM
 #include "gc_allocator.h"
 
@@ -1043,10 +1032,9 @@ static bool expression_is_true(Table&, E
 
 static bool _locate_expression(Table& table, Request& r, MethodParams& params) {
 	Value& expression_code=params.as_junction(0, "must be expression");
-	const size_t options_index=1;
-	Table::Action_options o=get_action_options(r, params, options_index, table);
-	check_option_param(o.defined, params, options_index, "locate by expression only has parameters: expression and, maybe, options");
-
+	Table::Action_options o=get_action_options(r, params, 1, table);
+	if(params.count()>2)
+		throw Exception(PARSER_RUNTIME, 0, "locate by expression only has parameters: expression and, maybe, options");
 	Expression_is_true_info info={&r, &expression_code};
 	return table.table_first_that(expression_is_true, &info, o);
 }
@@ -1054,10 +1042,7 @@ static bool _locate_expression(Table& ta
 static bool _locate_name_value(Table& table, Request& r, MethodParams& params) {
 	const String& name=params.as_string(0, "column name must be string");
 	const String& value=params.as_string(1, VALUE_MUST_BE_STRING);
-	const size_t options_index=2;
-	Table::Action_options o=get_action_options(r, params, options_index, table);
-	check_option_param(o.defined, params, options_index, "locate by name has parameters: name, value and, maybe, options");
-
+	Table::Action_options o=get_action_options(r, params, 2, table);
 	return table.locate(name, value, o);
 }
 
@@ -1164,6 +1149,19 @@ static void _append(Request& r, MethodPa
 	GET_SELF(r, VTable).table()+=row;
 }
 
+static void _insert(Request& r, MethodParams& params) {
+	Temp_lang temp_lang(r, String::L_PASS_APPENDED);
+	const String& string=r.process_to_string(params[0]);
+
+	// parse cells
+	Table::element_type row=new ArrayString;
+	size_t pos_after=0;
+	string.split(*row, pos_after, "\t", String::L_AS_IS);
+
+	Table& table=GET_SELF(r, VTable).table();
+	table.insert(table.current(), row);
+}
+
 static void join_named_row(Table& src, Table* dest) {
 	Table::columns_type dest_columns=dest->columns();
 	size_t dest_columns_count=dest_columns->count();
@@ -1181,7 +1179,6 @@ static void _join(Request& r, MethodPara
 	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");
 
 	Table& dest=GET_SELF(r, VTable).table();
 	if(&src == &dest)
@@ -1485,9 +1482,12 @@ MTable::MTable(): Methoded("table") {
 	// ^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}
+	// ^table.append{row{tab}data}
 	add_native_method("append", Method::CT_DYNAMIC, _append, 1, 1);
 
+	// ^table.insert{row{tab}data} before current row
+	add_native_method("insert", Method::CT_DYNAMIC, _insert, 1, 1);
+
 	// ^table.join[table][$.limit(10) $.offset(1) $.offset[cur] ]
 	add_native_method("join", Method::CT_DYNAMIC, _join, 1, 2);