--- parser3/src/classes/array.C 2024/10/28 00:18:25 1.31 +++ parser3/src/classes/array.C 2025/05/27 15:10:24 1.40 @@ -1,7 +1,7 @@ /** @file Parser: @b array parser class. - Copyright (c) 2001-2023 Art. Lebedev Studio (http://www.artlebedev.com) + Copyright (c) 2001-2024 Art. Lebedev Studio (http://www.artlebedev.com) Authors: Konstantin Morshnev , Alexandr Petrosian */ @@ -17,7 +17,7 @@ #include "pa_vbool.h" #include "pa_vmethod_frame.h" -volatile const char * IDENT_ARRAY_C="$Id: array.C,v 1.31 2024/10/28 00:18:25 moko Exp $"; +volatile const char * IDENT_ARRAY_C="$Id: array.C,v 1.40 2025/05/27 15:10:24 moko Exp $"; // class @@ -53,7 +53,8 @@ static void _copy_or_add(Request& r, Met self_array.put(i.index(), i.value()); } } else { - self_array.append(src->array()); + self_array.copy(src->array()); + return; } } else { HashStringValue* src_hash=vsrc.as_hash("param must be array or"); @@ -656,7 +657,7 @@ static void _remove(Request& r, MethodPa array.invalidate(); } -static void _pop(Request& r, MethodParams& params) { +static void _pop(Request& r, MethodParams&) { ArrayValue& array=GET_SELF(r, VArray).array(); Value *result=array.pop(); if(result){ @@ -690,12 +691,9 @@ static void _for(Request& r, MethodParam if(delim_maybe_code){ // delimiter set bool need_delim=false; - for(ArrayValue::Iterator i(array); i; i.next()){ - if(key_var_name){ - VString* vkey=new VString(*new String(i.key(), String::L_TAINTED)); - r.put_element(caller, *key_var_name, vkey); - } - + for(ArrayValue::RobustIterator i(array); i; i.next()){ + if(key_var_name) + r.put_element(caller, *key_var_name, VString::uitoa(i.index())); if(value_var_name) r.put_element(caller, *value_var_name, i.value() ? i.value() : VVoid::get()); @@ -716,12 +714,9 @@ static void _for(Request& r, MethodParam break; } } else { - for(ArrayValue::Iterator i(array); i; i.next()){ - if(key_var_name){ - VString* vkey=new VString(*new String(i.key(), String::L_TAINTED)); - r.put_element(caller, *key_var_name, vkey); - } - + for(ArrayValue::RobustIterator i(array); i; i.next()){ + if(key_var_name) + r.put_element(caller, *key_var_name, VString::uitoa(i.index())); if(value_var_name) r.put_element(caller, *value_var_name, i.value() ? i.value() : VVoid::get()); @@ -749,13 +744,10 @@ static void _foreach(Request& r, MethodP if(delim_maybe_code){ // delimiter set bool need_delim=false; - for(ArrayValue::Iterator i(array); i; i.next()){ + for(ArrayValue::RobustIterator i(array); i; i.next()){ if(i.value()){ - if(key_var_name){ - VString* vkey=new VString(*new String(i.key(), String::L_TAINTED)); - r.put_element(caller, *key_var_name, vkey); - } - + if(key_var_name) + r.put_element(caller, *key_var_name, VString::uitoa(i.index())); if(value_var_name) r.put_element(caller, *value_var_name, i.value()); @@ -777,13 +769,10 @@ static void _foreach(Request& r, MethodP } } } else { - for(ArrayValue::Iterator i(array); i; i.next()){ + for(ArrayValue::RobustIterator i(array); i; i.next()){ if(i.value()){ - if(key_var_name){ - VString* vkey=new VString(*new String(i.key(), String::L_TAINTED)); - r.put_element(caller, *key_var_name, vkey); - } - + if(key_var_name) + r.put_element(caller, *key_var_name, VString::uitoa(i.index())); if(value_var_name) r.put_element(caller, *value_var_name, i.value()); @@ -841,10 +830,10 @@ static void _sort(Request& r, MethodPara int pos=0; bool key_values_are_strings=true; - for(ArrayValue::Iterator i(array); i; i.next() ){ + for(ArrayValue::RobustIterator i(array); i; i.next() ){ if(i.value()){ if(key_var) - r.put_element(*context, *key_var, new VString(*new String(i.key(), String::L_TAINTED))); + r.put_element(*context, *key_var, VString::uitoa(i.index())); if(value_var) r.put_element(*context, *value_var, i.value()); @@ -915,13 +904,13 @@ static void _at(Request& r, MethodParams if(count == array.count()){ l1: switch(result_type) { case AtResultTypeKey: - r.write(*new VString(*new String(pa_uitoa(pos), String::L_TAINTED))); + r.write(*VString::uitoa(pos)); break; case AtResultTypeValue: r.write(*array.get(pos)); break; case AtResultTypeHash: - r.write(SingleElementHash(pa_uitoa(pos), array.get(pos))); + r.write(SingleElementHash(String::Body::uitoa(pos), array.get(pos))); break; } } else if((size_t)pos == count-1){ @@ -932,13 +921,13 @@ l1: switch(result_type) { if(i.value() && !(pos--)){ switch(result_type) { case AtResultTypeKey: - r.write(*new VString(*new String(i.key(), String::L_TAINTED))); + r.write(*VString::uitoa(i.index())); break; case AtResultTypeValue: r.write(*i.value()); break; case AtResultTypeHash: - r.write(SingleElementHash(i.key(), i.value())); + r.write(SingleElementHash(String::Body::uitoa(i.index()), i.value())); break; } break; @@ -964,7 +953,7 @@ static void _set(Request& r, MethodParam } else { for(ArrayValue::Iterator i(array); i; i.next() ){ if(i.value() && !(pos--)){ - array.put(pos, &r.process(params[1])); + array.put(i.index(), &r.process(params[1])); return; } } @@ -1016,7 +1005,7 @@ static void _select(Request& r, MethodPa for(ArrayValue::ReverseIterator i(source_array); i; ){ if(Value *value=i.prev()){ // here for correct i.key() if(key_var_name) - r.put_element(caller, *key_var_name, new VString(*new String(i.key(), String::L_TAINTED))); + r.put_element(caller, *key_var_name, VString::uitoa(i.index())); if(value_var_name) r.put_element(caller, *value_var_name, value); @@ -1033,10 +1022,10 @@ static void _select(Request& r, MethodPa } } } else { - for(ArrayValue::Iterator i(source_array); i; i.next() ){ + for(ArrayValue::RobustIterator i(source_array); i; i.next() ){ if(Value *value=i.value()){ if(key_var_name) - r.put_element(caller, *key_var_name, new VString(*new String(i.key(), String::L_TAINTED))); + r.put_element(caller, *key_var_name, VString::uitoa(i.index())); if(value_var_name) r.put_element(caller, *value_var_name, value);