Renamed statement:fetch() to statement:rows(), and statement:row() to statement:fetch(). The API reads better this way.

Tue, 02 Dec 2008 21:11:27 +0000

author
nrich@ii.net
date
Tue, 02 Dec 2008 21:11:27 +0000
changeset 12
014ba3ab3903
parent 11
b3e05e361f46
child 13
10c8c6f0da14

Renamed statement:fetch() to statement:rows(), and statement:row() to statement:fetch(). The API reads better this way.

dbd/mysql/statement.c file | annotate | diff | comparison | revisions
dbd/postgresql/statement.c file | annotate | diff | comparison | revisions
dbd/sqlite3/statement.c file | annotate | diff | comparison | revisions
--- a/dbd/mysql/statement.c	Tue Dec 02 08:10:54 2008 +0000
+++ b/dbd/mysql/statement.c	Tue Dec 02 21:11:27 2008 +0000
@@ -310,10 +310,20 @@
 }
 
 /*
- * iterfunc = statement:fetch(named_indexes)
+ * table = statement:fetch(named_indexes)
+ */
+static int statement_fetch(lua_State *L) {
+    statement_t *statement = (statement_t *)luaL_checkudata(L, 1, DBD_MYSQL_STATEMENT);
+    int named_columns = lua_toboolean(L, 2);
+
+    return statement_fetch_impl(L, statement, named_columns);
+}
+
+/*
+ * iterfunc = statement:rows(named_indexes)
  */
 
-static int statement_fetch(lua_State *L) {
+static int statement_rows(lua_State *L) {
     if (lua_gettop(L) == 1) {			
 	lua_pushvalue(L, 1);
 	lua_pushboolean(L, 0);
@@ -327,16 +337,6 @@
 }
 
 /*
- * table = statement:row(named_indexes)
- */
-static int statement_row(lua_State *L) {
-    statement_t *statement = (statement_t *)luaL_checkudata(L, 1, DBD_MYSQL_STATEMENT);
-    int named_columns = lua_toboolean(L, 2);
-
-    return statement_fetch_impl(L, statement, named_columns);
-}
-
-/*
  * __gc
  */
 static int statement_gc(lua_State *L) {
@@ -381,7 +381,7 @@
 	{"close", statement_close},
 	{"execute", statement_execute},
 	{"fetch", statement_fetch},
-	{"row", statement_row},
+	{"rows", statement_rows},
 	{NULL, NULL}
     };
 
--- a/dbd/postgresql/statement.c	Tue Dec 02 08:10:54 2008 +0000
+++ b/dbd/postgresql/statement.c	Tue Dec 02 21:11:27 2008 +0000
@@ -219,6 +219,7 @@
     int tuple = statement->tuple++;
     int i;
     int num_columns;
+    int d = 1;
 
     if (!statement->result) {
 	luaL_error(L, DBI_ERR_FETCH_INVALID);
@@ -237,7 +238,6 @@
 
     num_columns = PQnfields(statement->result);
     lua_newtable(L);
-    int d = 1;
     for (i = 0; i < num_columns; i++) {
 	const char *name = PQfname(statement->result, i);
 
@@ -314,10 +314,19 @@
 }
 
 /*
- * iterfunc = statement:fetch(named_indexes)
+ * table = statement:fetch(named_indexes)
  */
+static int statement_fetch(lua_State *L) {
+    statement_t *statement = (statement_t *)luaL_checkudata(L, 1, DBD_POSTGRESQL_STATEMENT);
+    int named_columns = lua_toboolean(L, 2);
 
-static int statement_fetch(lua_State *L) {
+    return statement_fetch_impl(L, statement, named_columns);
+}
+
+/*
+ * iterfunc = statement:rows(named_indexes)
+ */
+static int statement_rows(lua_State *L) {
     if (lua_gettop(L) == 1) {
         lua_pushvalue(L, 1);
         lua_pushboolean(L, 0);
@@ -331,16 +340,6 @@
 }
 
 /*
- * table = statement:row(named_indexes)
- */
-static int statement_row(lua_State *L) {
-    statement_t *statement = (statement_t *)luaL_checkudata(L, 1, DBD_POSTGRESQL_STATEMENT);
-    int named_columns = lua_toboolean(L, 2);
-
-    return statement_fetch_impl(L, statement, named_columns);
-}
-
-/*
  * __gc
  */
 static int statement_gc(lua_State *L) {
@@ -407,7 +406,7 @@
 	{"close", statement_close},
 	{"execute", statement_execute},
 	{"fetch", statement_fetch},
-	{"row", statement_row},
+	{"rows", statement_rows},
 	{NULL, NULL}
     };
 
--- a/dbd/sqlite3/statement.c	Tue Dec 02 08:10:54 2008 +0000
+++ b/dbd/sqlite3/statement.c	Tue Dec 02 21:11:27 2008 +0000
@@ -255,10 +255,19 @@
 }
 
 /*
- * iterfunc = statement:fetch(named_indexes)
+ * table = statement:fetch(named_indexes)
  */
+static int statement_fetch(lua_State *L) {
+    statement_t *statement = (statement_t *)luaL_checkudata(L, 1, DBD_SQLITE_STATEMENT);
+    int named_columns = lua_toboolean(L, 2);
 
-static int statement_fetch(lua_State *L) {
+    return statement_fetch_impl(L, statement, named_columns);
+}
+
+/*
+ * iterfunc = statement:rows(named_indexes)
+ */
+static int statement_rows(lua_State *L) {
     if (lua_gettop(L) == 1) {
         lua_pushvalue(L, 1);
         lua_pushboolean(L, 0);
@@ -272,16 +281,6 @@
 }
 
 /*
- * table = statement:row(named_indexes)
- */
-static int statement_row(lua_State *L) {
-    statement_t *statement = (statement_t *)luaL_checkudata(L, 1, DBD_SQLITE_STATEMENT);
-    int named_columns = lua_toboolean(L, 2);
-
-    return statement_fetch_impl(L, statement, named_columns);
-}
-
-/*
  * __gc
  */
 static int statement_gc(lua_State *L) {
@@ -315,7 +314,7 @@
 	{"close", statement_close},
 	{"execute", statement_execute},
 	{"fetch", statement_fetch},
-	{"row", statement_row},
+	{"rows", statement_rows},
 	{NULL, NULL}
     };
 

mercurial