Add missing PyArg_NoArgs() calls to methods that didn't take arguments

   (Pointed out by Moshe Zadka)

diff --git a/Modules/mmapmodule.c b/Modules/mmapmodule.c
index fcbb4845fc961d621603d283778c1acca8e5b587..377002a269d65aa5e6bfbe521f53cf0a1257315c 100644 (file)
--- a/Modules/mmapmodule.c
+++ b/Modules/mmapmodule.c
@@ -75,6 +75,8 @@ mmap_object_dealloc(mmap_object * m_obj)
 static PyObject *
 mmap_close_method (mmap_object * self, PyObject * args)
 {
+        if (!PyArg_NoArgs(args))
+               return NULL;
 #ifdef MS_WIN32
        UnmapViewOfFile (self->data);
        CloseHandle (self->map_handle);
@@ -118,6 +120,8 @@ mmap_read_byte_method (mmap_object * self,
        char value;
        char * where;
        CHECK_VALID(NULL);
+        if (!PyArg_NoArgs(args))
+               return NULL;
        if (self->pos >= 0 && self->pos < self->size) {
                where = self->data + self->pos;
                value = (char) *(where);
@@ -131,7 +135,7 @@ mmap_read_byte_method (mmap_object * self,
 
 static PyObject *
 mmap_read_line_method (mmap_object * self,
-                          PyObject * args)
+                      PyObject * args)
 {
        char * start = self->data+self->pos;
        char * eof = self->data+self->size;
@@ -139,6 +143,8 @@ mmap_read_line_method (mmap_object * self,
        PyObject * result;
 
        CHECK_VALID(NULL);
+        if (!PyArg_NoArgs(args))
+               return NULL;
 
        eol = memchr(start, '\n', self->size - self->pos);
        if (!eol)
@@ -153,7 +159,7 @@ mmap_read_line_method (mmap_object * self,
 
 static PyObject *
 mmap_read_method (mmap_object * self,
-                     PyObject * args)
+                 PyObject * args)
 {
        long num_bytes;
        PyObject *result;
@@ -225,7 +231,7 @@ mmap_write_method (mmap_object * self,
 
 static PyObject *
 mmap_write_byte_method (mmap_object * self,
-                           PyObject * args)
+                       PyObject * args)
 {
        char value;
 
@@ -241,9 +247,11 @@ mmap_write_byte_method (mmap_object * self,
 
 static PyObject *
 mmap_size_method (mmap_object * self,
-                     PyObject * args)
+                 PyObject * args)
 {
        CHECK_VALID(NULL);
+        if (!PyArg_NoArgs(args))
+               return NULL;
 
 #ifdef MS_WIN32
        if (self->file_handle != (HFILE) 0xFFFFFFFF) {
@@ -346,6 +354,8 @@ static PyObject *
 mmap_tell_method (mmap_object * self, PyObject * args)
 {
        CHECK_VALID(NULL);
+        if (!PyArg_NoArgs(args))
+               return NULL;
        return (Py_BuildValue ("l", self->pos) );
 }
 
@@ -462,10 +472,7 @@ static struct PyMethodDef mmap_object_methods[] = {
 /* Functions for treating an mmap'ed file as a buffer */
 
 static int
-mmap_buffer_getreadbuf(self, index, ptr)
-       mmap_object *self;
-int index;
-const void **ptr;
+mmap_buffer_getreadbuf(mmap_object *self, int index, const void **ptr)
 {
        CHECK_VALID(-1);
        if ( index != 0 ) {
@@ -867,3 +874,4 @@ initmmap(void)
 #endif /* MS_WIN32 */
 
 }
+