[Python-checkins] r63859 - in python/trunk: Lib/test/test_sys.py Objects/bytesobject.c Objects/dictobject.c Objects/listobject.c Objects/longobject.c Objects/typeobject.c Python/sysmodule.c

Sun Jun 1 18:42:16 CEST 2008

Author: georg.brandl
Date: Sun Jun 1 18:42:16 2008
New Revision: 63859
Log:
Some style nits. Also clarify in the docstrings what __sizeof__ does.
Modified:
 python/trunk/Lib/test/test_sys.py
 python/trunk/Objects/bytesobject.c
 python/trunk/Objects/dictobject.c
 python/trunk/Objects/listobject.c
 python/trunk/Objects/longobject.c
 python/trunk/Objects/typeobject.c
 python/trunk/Python/sysmodule.c
Modified: python/trunk/Lib/test/test_sys.py
==============================================================================

--- python/trunk/Lib/test/test_sys.py	(original)
+++ python/trunk/Lib/test/test_sys.py	Sun Jun 1 18:42:16 2008
@@ -419,7 +419,7 @@
 
 def tearDown(self):
 self.file.close()
- os.remove(test.test_support.TESTFN)
+ test.test_support.unlink(test.test_support.TESTFN)
 
 def check_sizeof(self, o, size):
 result = sys.getsizeof(o)
@@ -435,13 +435,13 @@
 return value
 
 def test_align(self):
- self.assertTrue( (self.align(0) % self.p) == 0 )
- self.assertTrue( (self.align(1) % self.p) == 0 )
- self.assertTrue( (self.align(3) % self.p) == 0 )
- self.assertTrue( (self.align(4) % self.p) == 0 )
- self.assertTrue( (self.align(7) % self.p) == 0 )
- self.assertTrue( (self.align(8) % self.p) == 0 )
- self.assertTrue( (self.align(9) % self.p) == 0 )
+ self.assertEqual(self.align(0) % self.p, 0)
+ self.assertEqual(self.align(1) % self.p, 0)
+ self.assertEqual(self.align(3) % self.p, 0)
+ self.assertEqual(self.align(4) % self.p, 0)
+ self.assertEqual(self.align(7) % self.p, 0)
+ self.assertEqual(self.align(8) % self.p, 0)
+ self.assertEqual(self.align(9) % self.p, 0)
 
 def test_standardtypes(self):
 i = self.i
@@ -507,7 +507,7 @@
 self.check_sizeof(abs, h + 3*p)
 # module
 self.check_sizeof(unittest, h + p)
- # xange
+ # xrange
 self.check_sizeof(xrange(1), h + 3*p)
 # slice
 self.check_sizeof(slice(0), h + 3*p)
@@ -520,8 +520,8 @@
 # type (PyTypeObject + PyNumberMethods + PyMappingMethods +
 # PySequenceMethods + PyBufferProcs)
 len_typeobject = p + 2*l + 15*p + l + 4*p + l + 9*p + l + 11*p
- self.check_sizeof(class_newstyle, h + \
- len_typeobject + 42*p + 10*p + 3*p + 6*p)
+ self.check_sizeof(class_newstyle,
+ h + len_typeobject + 42*p + 10*p + 3*p + 6*p)
 
 
 def test_specialtypes(self):
Modified: python/trunk/Objects/bytesobject.c
==============================================================================
--- python/trunk/Objects/bytesobject.c	(original)
+++ python/trunk/Objects/bytesobject.c	Sun Jun 1 18:42:16 2008
@@ -3918,7 +3918,7 @@
 }
 
 PyDoc_STRVAR(sizeof__doc__,
-"S.__sizeof__() -> size of S in bytes");
+"S.__sizeof__() -> size of S in memory, in bytes");
 
 static PyObject *
 string_sizeof(PyBytesObject *v)
Modified: python/trunk/Objects/dictobject.c
==============================================================================
--- python/trunk/Objects/dictobject.c	(original)
+++ python/trunk/Objects/dictobject.c	Sun Jun 1 18:42:16 2008
@@ -2052,7 +2052,7 @@
 PyDoc_STRVAR(getitem__doc__, "x.__getitem__(y) <==> x[y]");
 
 PyDoc_STRVAR(sizeof__doc__,
-"D.__sizeof__() -> size of D in bytes");
+"D.__sizeof__() -> size of D in memory, in bytes");
 
 PyDoc_STRVAR(get__doc__,
 "D.get(k[,d]) -> D[k] if k in D, else d. d defaults to None.");
Modified: python/trunk/Objects/listobject.c
==============================================================================
--- python/trunk/Objects/listobject.c	(original)
+++ python/trunk/Objects/listobject.c	Sun Jun 1 18:42:16 2008
@@ -2437,7 +2437,7 @@
 PyDoc_STRVAR(reversed_doc,
 "L.__reversed__() -- return a reverse iterator over the list");
 PyDoc_STRVAR(sizeof_doc,
-"L.__sizeof__() -- size of L in bytes");
+"L.__sizeof__() -- size of L in memory, in bytes");
 PyDoc_STRVAR(append_doc,
 "L.append(object) -- append object to end");
 PyDoc_STRVAR(extend_doc,
Modified: python/trunk/Objects/longobject.c
==============================================================================
--- python/trunk/Objects/longobject.c	(original)
+++ python/trunk/Objects/longobject.c	Sun Jun 1 18:42:16 2008
@@ -3467,7 +3467,7 @@
 	{"__getnewargs__",	(PyCFunction)long_getnewargs,	METH_NOARGS},
 {"__format__", (PyCFunction)long__format__, METH_VARARGS},
 	{"__sizeof__",	(PyCFunction)long_sizeof, METH_NOARGS,
-	 "Returns size in bytes"},
+	 "Returns size in memory, in bytes"},
 	{NULL,		NULL}		/* sentinel */
 };
 
Modified: python/trunk/Objects/typeobject.c
==============================================================================
--- python/trunk/Objects/typeobject.c	(original)
+++ python/trunk/Objects/typeobject.c	Sun Jun 1 18:42:16 2008
@@ -3421,7 +3421,7 @@
 {"__format__", object_format, METH_VARARGS,
 PyDoc_STR("default object formatter")},
 {"__sizeof__", object_sizeof, METH_NOARGS,
- PyDoc_STR("__sizeof__() -> size of object in bytes")},
+ PyDoc_STR("__sizeof__() -> size of object in memory, in bytes")},
 	{0}
 };
 
Modified: python/trunk/Python/sysmodule.c
==============================================================================
--- python/trunk/Python/sysmodule.c	(original)
+++ python/trunk/Python/sysmodule.c	Sun Jun 1 18:42:16 2008
@@ -664,9 +664,9 @@
 		return PyObject_CallFunctionObjArgs(method, args, NULL);
 	} 
 	/* Instance of old-style classes */
-	else if(PyInstance_Check(args))
+	else if (PyInstance_Check(args))
 		return PyInt_FromSsize_t(PyInstance_Type.tp_basicsize);
-	/* Old-style class */
+	/* Old-style classes */
 	else if (PyClass_Check(args))
 		return PyInt_FromSsize_t(PyClass_Type.tp_basicsize);
 	else
