commit c33a423c536862c2a1c7632c5afa871a25c46b77
author Jack Jansen <jack.jansen@cwi.nl> Fri Jan 19 23:46:28 2001 +0000
committer Jack Jansen <jack.jansen@cwi.nl> Fri Jan 19 23:46:28 2001 +0000
tree e38da3b103f165c6fecab622c6ddb1e42b172025
parent 726d873887c9c7bc9a8296780780b3b6975d1240

Treat an empty kwdict argument the same as a missing one.

Mac/Modules/Nav.c

diff --git a/Mac/Modules/Nav.c b/Mac/Modules/Nav.c
index d8ef6ef..934dc87 100644
--- a/Mac/Modules/Nav.c
+++ b/Mac/Modules/Nav.c
@@ -437,8 +437,8 @@
 	NavTypeListHandle typeList = NULL;
 	OSErr err;
 
-	if ( kw ) {
-		if (!PyArg_ParseTuple(args, ""))
+	if ( kw && PyObject_IsTrue(kw) ) {
+		if (!PyArg_ParseTuple(args, ";either keyword arguments or dictionary expected"))
 			return NULL;
 		dict = kw;
 	} else if (!PyArg_ParseTuple(args, "O!", &PyDict_Type, &dict))
@@ -474,8 +474,8 @@
 	OSType fileCreator;
 	OSErr err;
 
-	if ( kw ) {
-		if (!PyArg_ParseTuple(args, ""))
+	if ( kw && PyObject_IsTrue(kw) ) {
+		if (!PyArg_ParseTuple(args, ";either keyword arguments or dictionary expected"))
 			return NULL;
 		dict = kw;
 	} else if (!PyArg_ParseTuple(args, "O!", &PyDict_Type, &dict))
@@ -510,7 +510,7 @@
 	NavEventUPP eventProc = NULL;
 	OSErr err;
 
-	if ( kw ) {
+	if ( kw && PyObject_IsTrue(kw) ) {
 		if (!PyArg_ParseTuple(args, "l", &action))
 			return NULL;
 		dict = kw;
@@ -542,8 +542,8 @@
 	NavEventUPP eventProc = NULL;
 	OSErr err;
 
-	if ( kw ) {
-		if (!PyArg_ParseTuple(args, ""))
+	if ( kw && PyObject_IsTrue(kw) ) {
+		if (!PyArg_ParseTuple(args, ";either keyword arguments or dictionary expected"))
 			return NULL;
 		dict = kw;
 	} else if (!PyArg_ParseTuple(args, "O!", &PyDict_Type, &dict))
@@ -574,8 +574,8 @@
 	NavEventUPP eventProc = NULL;
 	OSErr err;
 
-	if ( kw ) {
-		if (!PyArg_ParseTuple(args, ""))
+	if ( kw && PyObject_IsTrue(kw) ) {
+		if (!PyArg_ParseTuple(args, ";either keyword arguments or dictionary expected"))
 			return NULL;
 		dict = kw;
 	} else if (!PyArg_ParseTuple(args, "O!", &PyDict_Type, &dict))
@@ -610,8 +610,8 @@
 	NavTypeListHandle typeList = NULL;
 	OSErr err;
 
-	if ( kw ) {
-		if (!PyArg_ParseTuple(args, ""))
+	if ( kw && PyObject_IsTrue(kw) ) {
+		if (!PyArg_ParseTuple(args, ";either keyword arguments or dictionary expected"))
 			return NULL;
 		dict = kw;
 	} else if (!PyArg_ParseTuple(args, "O!", &PyDict_Type, &dict))
@@ -646,8 +646,8 @@
 	NavObjectFilterUPP filterProc = NULL;
 	OSErr err;
 
-	if ( kw ) {
-		if (!PyArg_ParseTuple(args, ""))
+	if ( kw && PyObject_IsTrue(kw) ) {
+		if (!PyArg_ParseTuple(args, ";either keyword arguments or dictionary expected"))
 			return NULL;
 		dict = kw;
 	} else if (!PyArg_ParseTuple(args, "O!", &PyDict_Type, &dict))
@@ -682,8 +682,8 @@
 	NavObjectFilterUPP filterProc = NULL;
 	OSErr err;
 
-	if ( kw ) {
-		if (!PyArg_ParseTuple(args, ""))
+	if ( kw && PyObject_IsTrue(kw) ) {
+		if (!PyArg_ParseTuple(args, ";either keyword arguments or dictionary expected"))
 			return NULL;
 		dict = kw;
 	} else if (!PyArg_ParseTuple(args, "O!", &PyDict_Type, &dict))
@@ -718,8 +718,8 @@
 	NavObjectFilterUPP filterProc = NULL;
 	OSErr err;
 
-	if ( kw ) {
-		if (!PyArg_ParseTuple(args, ""))
+	if ( kw && PyObject_IsTrue(kw) ) {
+		if (!PyArg_ParseTuple(args, ";either keyword arguments or dictionary expected"))
 			return NULL;
 		dict = kw;
 	} else if (!PyArg_ParseTuple(args, "O!", &PyDict_Type, &dict))
@@ -753,8 +753,8 @@
 	NavEventUPP eventProc = NULL;
 	OSErr err;
 
-	if ( kw ) {
-		if (!PyArg_ParseTuple(args, ""))
+	if ( kw && PyObject_IsTrue(kw) ) {
+		if (!PyArg_ParseTuple(args, ";either keyword arguments or dictionary expected"))
 			return NULL;
 		dict = kw;
 	} else if (!PyArg_ParseTuple(args, "O!", &PyDict_Type, &dict))
@@ -771,7 +771,7 @@
 }
 
 #if 0
-XXXX I don't know what to do with the void * argument
+/* XXXX I don't know what to do with the void * argument */
 static char nav_NavCustomControl__doc__[] =
 ""
 ;