--- parser3/src/include/pa_array.h 2024/10/07 23:46:24 1.97 +++ parser3/src/include/pa_array.h 2025/08/01 16:14:44 1.105 @@ -1,14 +1,14 @@ /** @file Parser: Array & Array_iterator classes decls. - 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 */ #ifndef PA_ARRAY_H #define PA_ARRAY_H -#define IDENT_PA_ARRAY_H "$Id: pa_array.h,v 1.97 2024/10/07 23:46:24 moko Exp $" +#define IDENT_PA_ARRAY_H "$Id: pa_array.h,v 1.105 2025/08/01 16:14:44 moko Exp $" // includes @@ -19,6 +19,7 @@ // forwards template class Array_iterator; +template class Array_robust_iterator; template class Array_reverse_iterator; // defines @@ -29,6 +30,7 @@ template class Array_reverse template class Array: public PA_Object { friend class Array_iterator; + friend class Array_robust_iterator; friend class Array_reverse_iterator; protected: @@ -44,6 +46,7 @@ protected: public: typedef Array_iterator Iterator; + typedef Array_robust_iterator RobustIterator; typedef Array_reverse_iterator ReverseIterator; struct Action_options { @@ -108,14 +111,11 @@ public: } /// append other Array portion to this one. starting from offset - Array& append(const Array& src, - size_t offset=0, - size_t limit=ARRAY_OPTION_LIMIT_ALL) { //< zero limit means 'nothing' - + void append(const Array& src, size_t offset=0, size_t limit=ARRAY_OPTION_LIMIT_ALL) { //< zero limit means 'nothing' size_t src_count=src.count(); // skip tivials if(!src_count || !limit || offset>=src_count) - return *this; + return; // max(limit) size_t m=src_count-offset; // fix limit @@ -123,13 +123,8 @@ public: limit=m; fit(fsize-1+limit); - - T* from=&src.felements[offset]; - T* to=&felements[fsize]; - for(T* from_end=from+limit; from= fallocated) - resize(max(this->fallocated + this->fallocated/4, index+1)); + resize(max(fallocated+fallocated/4, index+1)); } void resize(size_t asize) { if(fallocated){ felements=(T *)pa_realloc(felements, asize*sizeof(T)); +#ifdef PA_DEBUG_DISABLE_GC + // non-gc realloc doesn't zero; manually zero expanded region + if(asize > fallocated) + memset((void *)(felements+fallocated), 0, (asize-fallocated) * sizeof(T)); +#endif fallocated=asize; } else { fallocated=asize; @@ -299,7 +299,6 @@ template class Array_iterato T *flast; public: - Array_iterator(const Array& aarray): farray(aarray) { fcurrent=farray.felements; flast=farray.felements + farray.fsize; @@ -324,42 +323,53 @@ public: inline size_t index() { return fcurrent - farray.felements; } +}; + +// Slower array iterator for arrays that can be modified during iteration +template class Array_robust_iterator { + + const Array& farray; + size_t findex; + +public: + Array_robust_iterator(const Array& aarray) : farray(aarray), findex(0) {} + + inline operator bool() { + return findex < farray.fsize; + } + + inline void next() { + findex++; + } - inline char *key(){ - return pa_uitoa(index()); + inline T value() { + return farray.felements[findex]; } + inline size_t index() { + return findex; + } }; +// Robust as used for arrays that can be modified during iteration template class Array_reverse_iterator { const Array& farray; - T *fcurrent; + size_t findex; public: + Array_reverse_iterator(const Array& aarray): farray(aarray), findex(aarray.fsize) {} - Array_reverse_iterator(const Array& aarray): farray(aarray) { - fcurrent=farray.felements + farray.fsize; - } - - /// there are still elements inline operator bool () { - return fcurrent > farray.felements; + return (findex > 0) && (findex <= farray.fsize); } - /// returns the current element and advances the iterator inline T prev() { - return *(--fcurrent); + return farray.felements[--findex]; } - // returns the current index of the iterator inline size_t index() { - return fcurrent - farray.felements; + return findex; } - - inline char *key(){ - return pa_uitoa(index()); - } - }; #endif