[Python-checkins] python/dist/src/Modules gcmodule.c,2.63,2.64

tim_one@users.sourceforge.net tim_one@users.sourceforge.net
Sat, 05 Apr 2003 17:50:52 -0800


Update of /cvsroot/python/python/dist/src/Modules
In directory sc8-pr-cvs1:/tmp/cvs-serv4669/Modules

Modified Files:
	gcmodule.c 
Log Message:
Switched from METH_VARARGS to METH_NOARGS for the 7 module functions that
take no arguments; cuts generated code size.


Index: gcmodule.c
===================================================================
RCS file: /cvsroot/python/python/dist/src/Modules/gcmodule.c,v
retrieving revision 2.63
retrieving revision 2.64
diff -C2 -d -r2.63 -r2.64
*** gcmodule.c	6 Apr 2003 00:11:39 -0000	2.63
--- gcmodule.c	6 Apr 2003 01:50:50 -0000	2.64
***************
*** 634,638 ****
  	/* Collect statistics on uncollectable objects found and print
  	 * debugging information. */
! 	for (gc = finalizers.gc.gc_next; 
  	     gc != &finalizers;
  	     gc = gc->gc.gc_next) {
--- 634,638 ----
  	/* Collect statistics on uncollectable objects found and print
  	 * debugging information. */
! 	for (gc = finalizers.gc.gc_next;
  	     gc != &finalizers;
  	     gc = gc->gc.gc_next) {
***************
*** 700,711 ****
  
  static PyObject *
! gc_enable(PyObject *self, PyObject *args)
  {
- 
- 	if (!PyArg_ParseTuple(args, ":enable"))	/* check no args */
- 		return NULL;
- 
  	enabled = 1;
- 
  	Py_INCREF(Py_None);
  	return Py_None;
--- 700,706 ----
  
  static PyObject *
! gc_enable(PyObject *self, PyObject *noargs)
  {
  	enabled = 1;
  	Py_INCREF(Py_None);
  	return Py_None;
***************
*** 718,729 ****
  
  static PyObject *
! gc_disable(PyObject *self, PyObject *args)
  {
- 
- 	if (!PyArg_ParseTuple(args, ":disable"))	/* check no args */
- 		return NULL;
- 
  	enabled = 0;
- 
  	Py_INCREF(Py_None);
  	return Py_None;
--- 713,719 ----
  
  static PyObject *
! gc_disable(PyObject *self, PyObject *noargs)
  {
  	enabled = 0;
  	Py_INCREF(Py_None);
  	return Py_None;
***************
*** 736,745 ****
  
  static PyObject *
! gc_isenabled(PyObject *self, PyObject *args)
  {
- 
- 	if (!PyArg_ParseTuple(args, ":isenabled"))	/* check no args */
- 		return NULL;
- 
  	return Py_BuildValue("i", enabled);
  }
--- 726,731 ----
  
  static PyObject *
! gc_isenabled(PyObject *self, PyObject *noargs)
  {
  	return Py_BuildValue("i", enabled);
  }
***************
*** 751,764 ****
  
  static PyObject *
! gc_collect(PyObject *self, PyObject *args)
  {
  	long n;
  
! 	if (!PyArg_ParseTuple(args, ":collect"))	/* check no args */
! 		return NULL;
! 
! 	if (collecting) {
  		n = 0; /* already collecting, don't do anything */
- 	}
  	else {
  		collecting = 1;
--- 737,746 ----
  
  static PyObject *
! gc_collect(PyObject *self, PyObject *noargs)
  {
  	long n;
  
! 	if (collecting)
  		n = 0; /* already collecting, don't do anything */
  	else {
  		collecting = 1;
***************
*** 802,810 ****
  
  static PyObject *
! gc_get_debug(PyObject *self, PyObject *args)
  {
- 	if (!PyArg_ParseTuple(args, ":get_debug"))	/* no args */
- 		return NULL;
- 
  	return Py_BuildValue("i", debug);
  }
--- 784,789 ----
  
  static PyObject *
! gc_get_debug(PyObject *self, PyObject *noargs)
  {
  	return Py_BuildValue("i", debug);
  }
***************
*** 840,848 ****
  
  static PyObject *
! gc_get_thresh(PyObject *self, PyObject *args)
  {
- 	if (!PyArg_ParseTuple(args, ":get_threshold"))	/* no args */
- 		return NULL;
- 
  	return Py_BuildValue("(iii)",
  			     generations[0].threshold,
--- 819,824 ----
  
  static PyObject *
! gc_get_thresh(PyObject *self, PyObject *noargs)
  {
  	return Py_BuildValue("(iii)",
  			     generations[0].threshold,
***************
*** 949,963 ****
  
  static PyObject *
! gc_get_objects(PyObject *self, PyObject *args)
  {
  	int i;
  	PyObject* result;
  
- 	if (!PyArg_ParseTuple(args, ":get_objects")) /* check no args */
- 		return NULL;
  	result = PyList_New(0);
! 	if (result == NULL) {
  		return NULL;
- 	}
  	for (i = 0; i < NUM_GENERATIONS; i++) {
  		if (append_objects(result, GEN_HEAD(i))) {
--- 925,936 ----
  
  static PyObject *
! gc_get_objects(PyObject *self, PyObject *noargs)
  {
  	int i;
  	PyObject* result;
  
  	result = PyList_New(0);
! 	if (result == NULL)
  		return NULL;
  	for (i = 0; i < NUM_GENERATIONS; i++) {
  		if (append_objects(result, GEN_HEAD(i))) {
***************
*** 986,998 ****
  
  static PyMethodDef GcMethods[] = {
! 	{"enable",	   gc_enable,	  METH_VARARGS, gc_enable__doc__},
! 	{"disable",	   gc_disable,	  METH_VARARGS, gc_disable__doc__},
! 	{"isenabled",	   gc_isenabled,  METH_VARARGS, gc_isenabled__doc__},
  	{"set_debug",	   gc_set_debug,  METH_VARARGS, gc_set_debug__doc__},
! 	{"get_debug",	   gc_get_debug,  METH_VARARGS, gc_get_debug__doc__},
  	{"set_threshold",  gc_set_thresh, METH_VARARGS, gc_set_thresh__doc__},
! 	{"get_threshold",  gc_get_thresh, METH_VARARGS, gc_get_thresh__doc__},
! 	{"collect",	   gc_collect,	  METH_VARARGS, gc_collect__doc__},
! 	{"get_objects",    gc_get_objects,METH_VARARGS, gc_get_objects__doc__},
  	{"get_referrers",  gc_get_referrers, METH_VARARGS,
  		gc_get_referrers__doc__},
--- 959,971 ----
  
  static PyMethodDef GcMethods[] = {
! 	{"enable",	   gc_enable,	  METH_NOARGS,  gc_enable__doc__},
! 	{"disable",	   gc_disable,	  METH_NOARGS,  gc_disable__doc__},
! 	{"isenabled",	   gc_isenabled,  METH_NOARGS,  gc_isenabled__doc__},
  	{"set_debug",	   gc_set_debug,  METH_VARARGS, gc_set_debug__doc__},
! 	{"get_debug",	   gc_get_debug,  METH_NOARGS,  gc_get_debug__doc__},
  	{"set_threshold",  gc_set_thresh, METH_VARARGS, gc_set_thresh__doc__},
! 	{"get_threshold",  gc_get_thresh, METH_NOARGS,  gc_get_thresh__doc__},
! 	{"collect",	   gc_collect,	  METH_NOARGS,  gc_collect__doc__},
! 	{"get_objects",    gc_get_objects,METH_NOARGS,  gc_get_objects__doc__},
  	{"get_referrers",  gc_get_referrers, METH_VARARGS,
  		gc_get_referrers__doc__},