--- parser3/src/classes/hash.C 2008/05/14 10:19:04 1.83 +++ parser3/src/classes/hash.C 2008/05/16 14:28:23 1.85 @@ -5,7 +5,7 @@ Author: Alexandr Petrosian (http://paf.design.ru) */ -static const char * const IDENT_HASH_C="$Date: 2008/05/14 10:19:04 $"; +static const char * const IDENT_HASH_C="$Date: 2008/05/16 14:28:23 $"; #include "classes.h" #include "pa_vmethod_frame.h" @@ -342,6 +342,7 @@ struct Foreach_info { Value* body_code; Value* delim_maybe_code; + Value* var_context; VString* vkey; bool need_delim; }; @@ -350,10 +351,13 @@ static bool one_foreach_cycle( HashStringValue::key_type akey, HashStringValue::value_type avalue, Foreach_info *info) { - info->vkey->set_string(*new String(akey, String::L_TAINTED)); - Value& ncontext=*info->r->get_method_frame()->caller(); - ncontext.put_element(ncontext, *info->key_var_name, info->vkey, false); - ncontext.put_element(ncontext, *info->value_var_name, avalue, false); + Value& var_context=*info->var_context; + if(info->key_var_name){ + info->vkey->set_string(*new String(akey, String::L_TAINTED)); + var_context.put_element(var_context, *info->key_var_name, info->vkey, false); + } + if(info->value_var_name) + var_context.put_element(var_context, *info->value_var_name, avalue, false); StringOrValue sv_processed=info->r->process(*info->body_code); Request::Skip lskip=info->r->get_skip(); info->r->set_skip(Request::SKIP_NOTHING); @@ -374,12 +378,16 @@ static void _foreach(Request& r, MethodP Temp_hash_value cycle_data_setter(r.classes_conf, cycle_data_name, /*any not null flag*/&r); + const String& key_var_name=params.as_string(0, "key-var name must be string"); + const String& value_var_name=params.as_string(1, "value-var name must be string"); + Foreach_info info={ &r, - ¶ms.as_string(0, "key-var name must be string"), - ¶ms.as_string(1, "value-var name must be string"), + key_var_name.is_empty()? 0 : &key_var_name, + value_var_name.is_empty()? 0 : &value_var_name, ¶ms.as_junction(2, "body must be code"), - params.count()>3?params.get(3):0, + /*delimiter*/params.count()>3?params.get(3):0, + /*var_context*/r.get_method_frame()->caller(), /*vkey=*/new VString, false };