From: Benjamin Peterson <benjamin@python.org>
Date: Fri, 25 May 2012 07:24:42 +0000 (-0700)
Subject: use PyDict_Contains
X-Git-Tag: v3.3.0a4~76
X-Git-Url: https://granicus.if.org/sourcecode?a=commitdiff_plain;h=18eac4a1d67b3b4a6f51608f0c3c82fa662100b0;p=python

use PyDict_Contains
---

diff --git a/Modules/zipimport.c b/Modules/zipimport.c
index 5ff39bb75b..70a57c91ce 100644
--- a/Modules/zipimport.c
+++ b/Modules/zipimport.c
@@ -268,7 +268,7 @@ static int
 check_is_directory(ZipImporter *self, PyObject* prefix, PyObject *path)
 {
     PyObject *dirpath;
-    PyObject *item;
+    int res;
 
     /* See if this is a "directory". If so, it's eligible to be part
        of a namespace package. We test by seeing if the name, with an
@@ -277,9 +277,9 @@ check_is_directory(ZipImporter *self, PyObject* prefix, PyObject *path)
     if (dirpath == NULL)
         return -1;
     /* If dirpath is present in self->files, we have a directory. */
-    item = PyDict_GetItem(self->files, dirpath);
+    res = PyDict_Contains(self->files, dirpath);
     Py_DECREF(dirpath);
-    return item != NULL;
+    return res;
 }
 
 /* Return some information about a module. */