--- parser3/src/main/execute.C	2002/04/15 08:13:10	1.227
+++ parser3/src/main/execute.C	2002/04/15 13:39:53	1.231
@@ -4,7 +4,7 @@
 	Copyright (c) 2001, 2002 ArtLebedev Group (http://www.artlebedev.com)
 	Author: Alexandr Petrosian <paf@design.ru> (http://paf.design.ru)
 
-	$Id: execute.C,v 1.227 2002/04/15 08:13:10 paf Exp $
+	$Id: execute.C,v 1.231 2002/04/15 13:39:53 paf Exp $
 */
 
 #include "pa_opcode.h"
@@ -274,11 +274,12 @@ void Request::execute(const Array& ops)
 			}
 		case OP_WRITE_EXPR_RESULT:
 			{
+				value=POP();
+				write_no_lang(*value->as_expr_result());
+
+				// must be after write(result) and 
 				// see OP_PREPARE_TO_EXPRESSION
 				wcontext->set_in_expression(false);
-
-				value=POP();
-				write_expr_result(*value->as_expr_result());
 				break;
 			}
 		case OP_STRING__WRITE:
@@ -332,7 +333,7 @@ void Request::execute(const Array& ops)
 
 				execute(*local_ops);
 
-				value=&wcontext->result();
+				value=&wcontext->result().as_value();
 				flang=static_cast<String::Untaint_lang>(reinterpret_cast<int>(POP()));
 				wcontext=static_cast<WContext *>(POP());
 				PUSH(value);
@@ -342,7 +343,6 @@ void Request::execute(const Array& ops)
 		case OP_STRING_POOL:
 			{
 				const Array *local_ops=reinterpret_cast<const Array *>(i.next());
-				
 
 				PUSH(wcontext);
 				WWrapper local(pool(), 0 /*empty*/);
@@ -494,7 +494,7 @@ void Request::execute(const Array& ops)
 								call_type==Method::CT_STATIC?"statically":"dynamically");
 
 				}
-				value=&wcontext->result();
+				value=&wcontext->result().as_value(); // todo CALL_WRITE & VString removal
 
 				wcontext=static_cast<WContext *>(POP());  
 				rcontext=POP();  
@@ -821,6 +821,7 @@ void Request::execute(const Array& ops)
 	}
 }
 
+/// @test cache|prepare junctions 
 Value *Request::get_element(bool can_call_operator) {
 	const String& name=POP_NAME();
 	Value *ncontext=POP();
@@ -834,9 +835,10 @@ Value *Request::get_element(bool can_cal
 		}
 	if(!value)
 		value=ncontext->get_element(name);
-	if(value)
-		value=&process_to_value(*value, &name); // process possible code-junction
-	else {
+	if(value) {
+		value=&process_to_value(*value); // process possible code-junction
+		value->update_name(name);
+	} else {
 		value=NEW VVoid(pool());
 		value->set_name(name);
 	}
@@ -855,11 +857,8 @@ Value *Request::get_element(bool can_cal
 
     using the fact it's either string_ or value_ result requested to speed up checkes
 */
-void Request::process_internal(
-							   Value& input_value, const String *result_name, 
-							   bool intercept_string,
-							   const String **string_result, Value **value_result,
-							   void (*postexecute)(void *info), void *postexecute_info) {
+StringOrValue Request::process(Value& input_value, bool intercept_string) {
+	StringOrValue result;
 	Junction *junction=input_value.get_junction();
 	if(junction && junction->code) { // is it a code-junction?
 		// process it
@@ -886,41 +885,21 @@ void Request::process_internal(
 			wcontext=&local;
 
 			// execute it
-			recoursion_checked_execute(result_name, *junction->code);
+			recoursion_checked_execute(0/*result_name*/, *junction->code);
 			
-			// maybe-postexecute something
-			if(postexecute)
-				postexecute(postexecute_info);
-		
 			// CodeFrame soul:
 			//   string writes were intercepted
 			//   returning them as the result of getting code-junction
-			if(string_result)
-				*string_result=wcontext->get_string();
-			else {
-				*value_result=NEW VString(*wcontext->get_string());
-				if(result_name)
-					(*value_result)->set_name(*result_name);
-			}
+			result.set_string(*wcontext->get_string());
 		} else {
 			// plain wwrapper
 			WWrapper local(pool(), 0/*empty*/);
 			wcontext=&local;
 		
 			// execute it
-			recoursion_checked_execute(result_name, *junction->code);
-		
-			// maybe-postexecute something
-			if(postexecute)
-				postexecute(postexecute_info);
+			recoursion_checked_execute(0/*result_name*/, *junction->code);
 		
-			if(string_result)
-				*string_result=&wcontext->result().as_string();
-			else {
-				*value_result=&wcontext->result();
-				if(result_name)
-					(*value_result)->set_name(*result_name);
-			}
+			result=wcontext->result();
 		}
 		
 		wcontext=static_cast<WContext *>(POP());  
@@ -932,14 +911,9 @@ void Request::process_internal(
 		debug_printf(pool(), "<-ja returned");
 #endif
 	} else {
-		if(string_result) // they asked for string_result
-			*string_result=&input_value.as_string();
-		else {// they asked for value_result
-			*value_result=&input_value;
-			if(result_name)
-				(*value_result)->set_name(*result_name);
-		}
+		result.set_value(input_value);
 	}
+	return result;
 }
 
 const String *Request::execute_method(Value& aself, const Method& method,