--- parser3/src/main/execute.C 2002/04/11 13:33:39 1.225 +++ 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 (http://paf.design.ru) - $Id: execute.C,v 1.225 2002/04/11 13:33:39 paf Exp $ + $Id: execute.C,v 1.231 2002/04/15 13:39:53 paf Exp $ */ #include "pa_opcode.h" @@ -33,8 +33,6 @@ ulong wcontext_result_size=0; -const uint ANTI_ENDLESS_EXECUTE_RECOURSION=500; - #ifdef DEBUG_EXECUTE char *opcode_name[]={ // literals @@ -47,8 +45,8 @@ char *opcode_name[]={ "CONSTRUCT_VALUE", "CONSTRUCT_EXPR", "CURLY_CODE__CONSTRUCT", "WRITE_VALUE", "WRITE_EXPR_RESULT", "STRING__WRITE", "GET_ELEMENT_OR_OPERATOR", "GET_ELEMENT", "GET_ELEMENT__WRITE", - "CREATE_EWPOOL", "REDUCE_EWPOOL", - "CREATE_SWPOOL", "REDUCE_SWPOOL", + "OBJECT_POOL", + "STRING_POOL", "GET_METHOD_FRAME", "STORE_PARAM", "PREPARE_TO_CONSTRUCT_OBJECT", "PREPARE_TO_EXPRESSION", "CALL", @@ -99,6 +97,8 @@ void debug_dump(Pool& pool, int level, c case OP_EXPR_CODE__STORE_PARAM: case OP_CURLY_CODE__CONSTRUCT: case OP_NESTED_CODE: + case OP_OBJECT_POOL: + case OP_STRING_POOL: const Array *local_ops=reinterpret_cast(i.next()); debug_dump(pool, level+1, *local_ops); } @@ -111,18 +111,6 @@ void debug_dump(Pool& pool, int level, c #define POP_NAME() static_cast(stack.pop())->as_string() #define POP_CODE() static_cast(stack.pop()) -void Request::recoursion_checked_execute(const String *name, const Array& ops) { - // anti_endless_execute_recoursion - if(++anti_endless_execute_recoursion==ANTI_ENDLESS_EXECUTE_RECOURSION) { - anti_endless_execute_recoursion=0; // give @exception a chance - throw Exception("parser.runtime", - name, - "call canceled - endless recursion detected"); - } - execute(ops); // execute it - anti_endless_execute_recoursion--; -} - void Request::execute(const Array& ops) { // _asm int 3; #ifdef DEBUG_EXECUTE @@ -286,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,31 +321,35 @@ void Request::execute(const Array& ops) } - case OP_CREATE_EWPOOL: + case OP_OBJECT_POOL: { + const Array *local_ops=reinterpret_cast(i.next()); + PUSH(wcontext); PUSH((void *)flang); flang=String::UL_PASS_APPENDED; - wcontext=NEW WWrapper(pool(), 0 /*empty*/); - break; - } - case OP_REDUCE_EWPOOL: - { - value=&wcontext->result(); + WWrapper local(pool(), 0 /*empty*/); + wcontext=&local; + + execute(*local_ops); + + value=&wcontext->result().as_value(); flang=static_cast(reinterpret_cast(POP())); wcontext=static_cast(POP()); PUSH(value); break; } - case OP_CREATE_SWPOOL: + case OP_STRING_POOL: { + const Array *local_ops=reinterpret_cast(i.next()); + PUSH(wcontext); - wcontext=NEW WWrapper(pool(), 0 /*empty*/); - break; - } - case OP_REDUCE_SWPOOL: - { + WWrapper local(pool(), 0 /*empty*/); + wcontext=&local; + + execute(*local_ops); + // from "$a $b" part of expression taking only string value, // ignoring any other content of wcontext const String *string=wcontext->get_string(); @@ -501,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(POP()); rcontext=POP(); @@ -828,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(); @@ -841,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); } @@ -862,10 +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) { +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 @@ -892,33 +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); + // 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); + 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(POP()); @@ -930,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, @@ -949,7 +925,8 @@ const String *Request::execute_method(Va // initialize contexts root=rcontext=self=&aself; - wcontext=NEW WWrapper(pool(), &aself); + WWrapper local(pool(), &aself); + wcontext=&local; // execute! execute(*method.parser_code);