--- parser3/src/include/pa_table.h	2020/12/25 22:05:31	1.72
+++ parser3/src/include/pa_table.h	2024/11/16 02:57:05	1.75
@@ -1,19 +1,21 @@
 /** @file
 	Parser: table class decl.
 
-	Copyright (c) 2001-2020 Art. Lebedev Studio (http://www.artlebedev.com)
-	Author: Alexandr Petrosian <paf@design.ru> (http://paf.design.ru)
+	Copyright (c) 2001-2024 Art. Lebedev Studio (http://www.artlebedev.com)
+	Authors: Konstantin Morshnev <moko@design.ru>, Alexandr Petrosian <paf@design.ru>
 */
 
 #ifndef PA_TABLE_H
 #define PA_TABLE_H
 
-#define IDENT_PA_TABLE_H "$Id: pa_table.h,v 1.72 2020/12/25 22:05:31 moko Exp $"
+#define IDENT_PA_TABLE_H "$Id: pa_table.h,v 1.75 2024/11/16 02:57:05 moko Exp $"
 
 #include "pa_types.h"
 #include "pa_hash.h"
 #include "pa_string.h"
 
+class Temp_current;
+
 /** 
 	VTable backend.
 
@@ -38,12 +40,11 @@ public:
 	/// gets column names
 	columns_type columns() { return fcolumns; }
 
-	/// moves @a current pointer
+	/// moves @a current pointer, can be out of range when restoring current in modified table
 	void set_current(size_t acurrent) {
-		assert(acurrent==0 || acurrent<count());
-
-		fcurrent=acurrent;
+		fcurrent=acurrent<count() ? acurrent : count()>0 ? count()-1 : 0;
 	}
+
 	/// @return current pointer
 	size_t current() const { return fcurrent; }
 	void offset(bool absolute, int offset);
@@ -81,7 +82,7 @@ public:
 		if(!o.adjust(count()))
 			return;
 
-		size_t saved_current=current();
+		Temp_current tc(*this);
 		size_t row=o.offset;
 		if(o.reverse) { // reverse
 			for(size_t i=0; i<o.limit; i++) {
@@ -94,7 +95,6 @@ public:
 				func(*this, info);
 			}
 		}
-		set_current(saved_current);
 	}
 
 	template<typename I>
@@ -102,7 +102,7 @@ public:
 		if(!o.adjust(count()))
 			return false;
 
-		size_t saved_current=current();
+		Temp_current tc(*this);
 		size_t row=o.offset;
 		if(o.reverse) { // reverse
 			for(size_t i=0; i<o.limit; i++) {
@@ -119,7 +119,6 @@ public:
 					return true;
 			}
 		}
-		set_current(saved_current);
 
 		return false;
 	}
@@ -144,4 +143,15 @@ private:
 
 };
 
+/// Auto-object that temporarily saves and restores current
+class Temp_current {
+	Table& ftable;
+	size_t fcurrent;
+public:
+	Temp_current(Table& atable) : ftable(atable), fcurrent(atable.current()){}
+	~Temp_current(){
+		ftable.set_current(fcurrent);
+	}
+};
+
 #endif