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 fcbb484..377002a 100644
--- a/Modules/mmapmodule.c
+++ b/Modules/mmapmodule.c
@@ -75,6 +75,8 @@
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 @@
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 @@
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 @@
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 @@
static PyObject *
mmap_read_method (mmap_object * self,
- PyObject * args)
+ PyObject * args)
{
long num_bytes;
PyObject *result;
@@ -225,7 +231,7 @@
static PyObject *
mmap_write_byte_method (mmap_object * self,
- PyObject * args)
+ PyObject * args)
{
char value;
@@ -241,9 +247,11 @@
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 @@
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 @@
/* 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 @@
#endif /* MS_WIN32 */
}
+