--- parser3/src/classes/string.C	2003/07/24 11:31:20	1.126
+++ parser3/src/classes/string.C	2003/11/20 17:46:01	1.131
@@ -5,7 +5,7 @@
 	Author: Alexandr Petrosian <paf@design.ru> (http://paf.design.ru)
 */
 
-static const char* IDENT_STRING_C="$Date: 2003/07/24 11:31:20 $";
+static const char * const IDENT_STRING_C="$Date: 2003/11/20 17:46:01 $";
 
 #include "classes.h"
 #include "pa_vmethod_frame.h"
@@ -111,7 +111,8 @@ static void _right(Request& r, MethodPar
 	size_t n=(size_t)params.as_int(0, "n must be int", r);
 	
 	const String& string=GET_SELF(r, VString).string();
-	r.write_assign_lang(string.mid(string.length()-n, string.length()));
+	size_t length=string.length();
+	r.write_assign_lang(n<length?string.mid(length-n, string.length()):string);
 }
 
 static void _mid(Request& r, MethodParams& params) {
@@ -131,8 +132,7 @@ static void _pos(Request& r, MethodParam
 	r.write_assign_lang(*new VInt((int)string.pos(substr.as_string())));
 }
 
-static void split_list(Request& r, 
-		       MethodParams& params, int paramIndex,
+static void split_list(MethodParams& params, int paramIndex,
 		       const String& string, 
 		       ArrayString& result) {
 	Value& delim_value=params.as_no_junction(paramIndex, "delimiter must not be code");
@@ -157,7 +157,7 @@ static int split_options(const String* o
 		{"r", "R", SPLIT_RIGHT, SPLIT_LEFT},
 		{"h", "H", SPLIT_HORIZONTAL, SPLIT_VERTICAL},
 		{"v", "V", SPLIT_VERTICAL, SPLIT_HORIZONTAL},
-		{0}
+		{0, 0, 0, 0}
     };
 
 	int result=0;
@@ -176,8 +176,7 @@ static int split_options(const String* o
 	return result;
 }
 
-static Table& split_vertical(Request& r, ArrayString& pieces, bool right) {
-
+static Table& split_vertical(ArrayString& pieces, bool right) {
 	Table::columns_type columns(new ArrayString);
 	*columns+=new String("piece");
 
@@ -200,11 +199,11 @@ static Table& split_vertical(Request& r,
 	return table;
 }
 
-static Table& split_horizontal(Request& r, ArrayString& pieces, bool right) {
+static Table& split_horizontal(ArrayString& pieces, bool right) {
 	Table& table=*new Table(Table::columns_type(0) /* nameless */);
 	Table::element_type row(new ArrayString(pieces.count()));
 	if(right) { // right
-		for(size_t i=pieces.count(); --i>=0; )
+		for(int i=pieces.count(); --i>=0; )
 			*row+=pieces[i];
 	} else { // left
 		for(Array_iterator<const String*> i(pieces); i.has_next(); )
@@ -220,7 +219,7 @@ static void split_with_options(Request&
 	const String& string=GET_SELF(r, VString).string();
 
 	ArrayString pieces;
-	split_list(r, params, 0, string, pieces);
+	split_list(params, 0, string, pieces);
 
 	if(!bits) {
 		const String* options=0;
@@ -232,8 +231,8 @@ static void split_with_options(Request&
 
 	bool right=(bits & SPLIT_RIGHT) != 0;
 	bool horizontal=(bits & SPLIT_HORIZONTAL) !=0;
-	Table& table=horizontal?split_horizontal(r, pieces, right)
-		:split_vertical(r, pieces, right);
+	Table& table=horizontal?split_horizontal(pieces, right)
+		:split_vertical(pieces, right);
 
 	r.write_no_lang(*new VTable(&table));
 }
@@ -311,12 +310,13 @@ static void _match(Request& r, MethodPar
 
 		String result;
 		VTable* vtable=new VTable;
-		Replace_action_info info={0};
-		info.request=&r;
-		info.src=&src;
-		info.dest=&result;
-		info.vtable=vtable;
-		info.replacement_code=&replacement_code;
+		Replace_action_info info={
+			&r,
+			&src,
+			&result,
+			vtable,
+			&replacement_code
+		};
 		Temp_value_element temp_match_var(
 			*replacement_code.get_junction()->method_frame, 
 			match_var_name, vtable);
@@ -328,7 +328,7 @@ static void _match(Request& r, MethodPar
 	}
 }
 
-static void change_case(Request& r, MethodParams& params, 
+static void change_case(Request& r, MethodParams&, 
 						String::Change_case_kind kind) {
 	const String& src=GET_SELF(r, VString).string();
 
@@ -356,7 +356,7 @@ public:
 		got_cell(false),
 		result(*new String) {}
 
-	bool add_column(SQL_Error& error, const char* str, size_t /*length*/) {
+	bool add_column(SQL_Error& error, const char* /*str*/, size_t /*length*/) {
 		if(got_column) {
 			error=SQL_Error("parser.runtime",
 				//statement_string,
@@ -401,15 +401,13 @@ const String* sql_result_string(Request&
 	if(params.count()>1) {
 		Value& voptions=params.as_no_junction(1, "options must be hash, not code");
 		if(!voptions.is_string())
-			if(options=voptions.get_hash()) {
+			if((options=voptions.get_hash())) {
 				if(Value* vlimit=options->get(sql_limit_name))
 					limit=(ulong)r.process_to_value(*vlimit).as_double();
 				if(Value* voffset=options->get(sql_offset_name))
 					offset=(ulong)r.process_to_value(*voffset).as_double();
-				if(default_code=options->get(sql_default_name)) {
-					if(Junction* default_junction=default_code->get_junction())
-						;//default_junction->change_context(statement.get_junction());
-					else
+				if((default_code=options->get(sql_default_name))) {
+					if(!default_code->get_junction())
 						throw Exception("parser.runtime",
 							0,
 							"default option must be code");