--- parser3/src/include/pa_table.h	2001/03/12 12:00:05	1.11
+++ parser3/src/include/pa_table.h	2001/03/19 15:29:38	1.14
@@ -1,16 +1,11 @@
-/*
+/** @file
 	Parser
 	Copyright (c) 2001 ArtLebedev Group (http://www.artlebedev.com)
 	Author: Alexander Petrosyan <paf@design.ru> (http://design.ru/paf)
 
-	$Id: pa_table.h,v 1.11 2001/03/12 12:00:05 paf Exp $
+	$Id: pa_table.h,v 1.14 2001/03/19 15:29:38 paf Exp $
 */
 
-/*
-	hence most of tables are "named", no need to uptimize nameless onces
-*/
-
-
 #ifndef PA_TABLE_H
 #define PA_TABLE_H
 
@@ -20,29 +15,43 @@
 #include "pa_string.h"
 #include "pa_request.h"
 
+/// parser \b table class backend
+/**
+	holds 
+	- column names[if any]
+	- data rows
+	- current row pointer
+
+	uses String for column names and data items
+
+	hence most of tables are "named", no need to uptimize nameless onces
+*/
 class Table : public Array {
 public:
 
 	Table(Pool& apool,
-		char *afile, uint aline,
+		const String& asource,
 		Array *acolumns,
 		int initial_rows=CR_INITIAL_ROWS_DEFAULT);
 
-	// the base origin of table's data
-	//const Origin& origin() { return forigin; }
+	/// the string source of table's data
+	//const String& source() { return fsource; }
 
+	/// column names
 	const Array *columns() { return fcolumns; }
 
+	/// moves \a current pointer
 	void set_current(int acurrent) { fcurrent=acurrent; }
+	/// \return current pointer
 	int get_current() { return fcurrent; }
-	void inc_current() { fcurrent++; }
 
+	/// \return item from \a column_name
 	const String *item(const String& column_name);
 
-protected:
+private:
 
 	// the base origin of table's data
-	Origin forigin;
+	const String& fsource;
 
 	// column name->number lookup table
 	Hash name2number;
@@ -53,10 +62,12 @@ protected:
 	// columns
 	Array *fcolumns;
 
+	bool valid(int index) { return index>=0 && index<size();	}
+
 	const Array& at(int index);
 
 	const String *item(int column_index) {
-		return at(fcurrent).get_string(column_index);
+		return valid(fcurrent)?at(fcurrent).get_string(column_index):0;
 	}
 };