[Python-checkins] cpython: Close #19798: replace "maximum" term with "peak" in get_traced_memory()

victor.stinner python-checkins at python.org
Wed Nov 27 21:40:12 CET 2013


http://hg.python.org/cpython/rev/553144bd7bf1
changeset:   87615:553144bd7bf1
user:        Victor Stinner <victor.stinner at gmail.com>
date:        Wed Nov 27 21:39:49 2013 +0100
summary:
  Close #19798: replace "maximum" term with "peak" in get_traced_memory()
documentation. Use also the term "current" for the current size.

files:
  Doc/library/tracemalloc.rst  |   4 ++--
  Lib/test/test_tracemalloc.py |  14 +++++++-------
  Modules/_tracemalloc.c       |  24 ++++++++++++------------
  3 files changed, 21 insertions(+), 21 deletions(-)


diff --git a/Doc/library/tracemalloc.rst b/Doc/library/tracemalloc.rst
--- a/Doc/library/tracemalloc.rst
+++ b/Doc/library/tracemalloc.rst
@@ -276,8 +276,8 @@
 
 .. function:: get_traced_memory()
 
-   Get the current size and maximum size of memory blocks traced by the
-   :mod:`tracemalloc` module as a tuple: ``(size: int, max_size: int)``.
+   Get the current size and peak size of memory blocks traced by the
+   :mod:`tracemalloc` module as a tuple: ``(current: int, peak: int)``.
 
 
 .. function:: get_tracemalloc_memory()
diff --git a/Lib/test/test_tracemalloc.py b/Lib/test/test_tracemalloc.py
--- a/Lib/test/test_tracemalloc.py
+++ b/Lib/test/test_tracemalloc.py
@@ -182,19 +182,19 @@
         obj_size = 1024 * 1024
         tracemalloc.clear_traces()
         obj, obj_traceback = allocate_bytes(obj_size)
-        size, max_size = tracemalloc.get_traced_memory()
+        size, peak_size = tracemalloc.get_traced_memory()
         self.assertGreaterEqual(size, obj_size)
-        self.assertGreaterEqual(max_size, size)
+        self.assertGreaterEqual(peak_size, size)
 
         self.assertLessEqual(size - obj_size, max_error)
-        self.assertLessEqual(max_size - size, max_error)
+        self.assertLessEqual(peak_size - size, max_error)
 
         # destroy the object
         obj = None
-        size2, max_size2 = tracemalloc.get_traced_memory()
+        size2, peak_size2 = tracemalloc.get_traced_memory()
         self.assertLess(size2, size)
         self.assertGreaterEqual(size - size2, obj_size - max_error)
-        self.assertGreaterEqual(max_size2, max_size)
+        self.assertGreaterEqual(peak_size2, peak_size)
 
         # clear_traces() must reset traced memory counters
         tracemalloc.clear_traces()
@@ -202,8 +202,8 @@
 
         # allocate another object
         obj, obj_traceback = allocate_bytes(obj_size)
-        size, max_size = tracemalloc.get_traced_memory()
-        self.assertGreater(size, 0)
+        size, peak_size = tracemalloc.get_traced_memory()
+        self.assertGreaterEqual(size, obj_size)
 
         # stop() also resets traced memory counters
         tracemalloc.stop()
diff --git a/Modules/_tracemalloc.c b/Modules/_tracemalloc.c
--- a/Modules/_tracemalloc.c
+++ b/Modules/_tracemalloc.c
@@ -106,9 +106,9 @@
    Protected by TABLES_LOCK(). */
 static size_t tracemalloc_traced_memory = 0;
 
-/* Maximum size in bytes of traced memory.
+/* Peak size in bytes of traced memory.
    Protected by TABLES_LOCK(). */
-static size_t tracemalloc_max_traced_memory = 0;
+static size_t tracemalloc_peak_traced_memory = 0;
 
 /* Hash table used as a set to to intern filenames:
    PyObject* => PyObject*.
@@ -464,8 +464,8 @@
     if (res == 0) {
         assert(tracemalloc_traced_memory <= PY_SIZE_MAX - size);
         tracemalloc_traced_memory += size;
-        if (tracemalloc_traced_memory > tracemalloc_max_traced_memory)
-            tracemalloc_max_traced_memory = tracemalloc_traced_memory;
+        if (tracemalloc_traced_memory > tracemalloc_peak_traced_memory)
+            tracemalloc_peak_traced_memory = tracemalloc_traced_memory;
     }
     TABLES_UNLOCK();
 
@@ -674,7 +674,7 @@
     TABLES_LOCK();
     _Py_hashtable_clear(tracemalloc_traces);
     tracemalloc_traced_memory = 0;
-    tracemalloc_max_traced_memory = 0;
+    tracemalloc_peak_traced_memory = 0;
     TABLES_UNLOCK();
 
     _Py_hashtable_foreach(tracemalloc_tracebacks, traceback_free_traceback, NULL);
@@ -1266,26 +1266,26 @@
 PyDoc_STRVAR(tracemalloc_get_traced_memory_doc,
     "get_traced_memory() -> (int, int)\n"
     "\n"
-    "Get the current size and maximum size of memory blocks traced\n"
-    "by the tracemalloc module as a tuple: (size: int, max_size: int).");
+    "Get the current size and peak size of memory blocks traced\n"
+    "by the tracemalloc module as a tuple: (current: int, peak: int).");
 
 static PyObject*
 tracemalloc_get_traced_memory(PyObject *self)
 {
-    Py_ssize_t size, max_size;
-    PyObject *size_obj, *max_size_obj;
+    Py_ssize_t size, peak_size;
+    PyObject *size_obj, *peak_size_obj;
 
     if (!tracemalloc_config.tracing)
         return Py_BuildValue("ii", 0, 0);
 
     TABLES_LOCK();
     size = tracemalloc_traced_memory;
-    max_size = tracemalloc_max_traced_memory;
+    peak_size = tracemalloc_peak_traced_memory;
     TABLES_UNLOCK();
 
     size_obj = PyLong_FromSize_t(size);
-    max_size_obj = PyLong_FromSize_t(max_size);
-    return Py_BuildValue("NN", size_obj, max_size_obj);
+    peak_size_obj = PyLong_FromSize_t(peak_size);
+    return Py_BuildValue("NN", size_obj, peak_size_obj);
 }
 
 static PyMethodDef module_methods[] = {

-- 
Repository URL: http://hg.python.org/cpython


More information about the Python-checkins mailing list