Add PYSLICE to spell-checker ignore in slice.rs

youknowone · youknowone · commit b62b95e5a719 · 2026-03-04T15:51:40.000+09:00
diff --git a/crates/vm/src/builtins/dict.rs b/crates/vm/src/builtins/dict.rs
@@ -62,14 +62,14 @@ impl fmt::Debug for PyDict {
     }
 }
 
-// spell-checker:ignore MAXFREELIST
-const PYDICT_MAXFREELIST: usize = 80;
-
 thread_local! {
     static DICT_FREELIST: Cell<Vec<*mut PyObject>> = const { Cell::new(Vec::new()) };
 }
 
 impl PyPayload for PyDict {
+    // spell-checker:ignore MAXFREELIST
+    /// PyDict_MAXFREELIST
+    const MAX_FREELIST: usize = 80;
     const HAS_FREELIST: bool = true;
 
     #[inline]
@@ -81,7 +81,7 @@ impl PyPayload for PyDict {
     unsafe fn freelist_push(obj: *mut PyObject) -> bool {
         DICT_FREELIST.with(|fl| {
             let mut list = fl.take();
-            let stored = if list.len() < PYDICT_MAXFREELIST {
+            let stored = if list.len() < Self::MAX_FREELIST {
                 list.push(obj);
                 true
             } else {
diff --git a/crates/vm/src/builtins/float.rs b/crates/vm/src/builtins/float.rs
@@ -34,14 +34,14 @@ impl PyFloat {
     }
 }
 
-// spell-checker:ignore MAXFREELIST
-const PYFLOAT_MAXFREELIST: usize = 100;
-
 thread_local! {
     static FLOAT_FREELIST: Cell<Vec<*mut PyObject>> = const { Cell::new(Vec::new()) };
 }
 
 impl PyPayload for PyFloat {
+    // spell-checker:ignore MAXFREELIST
+    /// PyFloat_MAXFREELIST
+    const MAX_FREELIST: usize = 100;
     const HAS_FREELIST: bool = true;
 
     #[inline]
@@ -53,7 +53,7 @@ impl PyPayload for PyFloat {
     unsafe fn freelist_push(obj: *mut PyObject) -> bool {
         FLOAT_FREELIST.with(|fl| {
             let mut list = fl.take();
-            let stored = if list.len() < PYFLOAT_MAXFREELIST {
+            let stored = if list.len() < Self::MAX_FREELIST {
                 list.push(obj);
                 true
             } else {
diff --git a/crates/vm/src/builtins/list.rs b/crates/vm/src/builtins/list.rs
@@ -74,14 +74,14 @@ unsafe impl Traverse for PyList {
     }
 }
 
-// spell-checker:ignore MAXFREELIST
-const PYLIST_MAXFREELIST: usize = 80;
-
 thread_local! {
     static LIST_FREELIST: Cell<Vec<*mut PyObject>> = const { Cell::new(Vec::new()) };
 }
 
 impl PyPayload for PyList {
+    // spell-checker:ignore MAXFREELIST
+    /// PyList_MAXFREELIST
+    const MAX_FREELIST: usize = 80;
     const HAS_FREELIST: bool = true;
 
     #[inline]
@@ -93,7 +93,7 @@ impl PyPayload for PyList {
     unsafe fn freelist_push(obj: *mut PyObject) -> bool {
         LIST_FREELIST.with(|fl| {
             let mut list = fl.take();
-            let stored = if list.len() < PYLIST_MAXFREELIST {
+            let stored = if list.len() < Self::MAX_FREELIST {
                 list.push(obj);
                 true
             } else {
diff --git a/crates/vm/src/builtins/slice.rs b/crates/vm/src/builtins/slice.rs
@@ -25,15 +25,14 @@ pub struct PySlice {
     pub step: Option<PyObjectRef>,
 }
 
-// spell-checker:ignore MAXFREELIST
-// PySlice_MAXFREELIST = 1
-const PYSLICE_MAXFREELIST: usize = 1;
-
 thread_local! {
     static SLICE_FREELIST: Cell<Vec<*mut PyObject>> = const { Cell::new(Vec::new()) };
 }
 
 impl PyPayload for PySlice {
+    // spell-checker:ignore MAXFREELIST
+    /// PySlice_MAXFREELIST
+    const MAX_FREELIST: usize = 1;
     const HAS_FREELIST: bool = true;
 
     #[inline]
@@ -45,7 +44,7 @@ impl PyPayload for PySlice {
     unsafe fn freelist_push(obj: *mut PyObject) -> bool {
         SLICE_FREELIST.with(|fl| {
             let mut list = fl.take();
-            let stored = if list.len() < PYSLICE_MAXFREELIST {
+            let stored = if list.len() < Self::MAX_FREELIST {
                 list.push(obj);
                 true
             } else {
diff --git a/crates/vm/src/object/payload.rs b/crates/vm/src/object/payload.rs
@@ -52,6 +52,9 @@ pub trait PyPayload: MaybeTraverse + PyThreadingConstraint + Sized + 'static {
     /// race conditions when the object is reused.
     const HAS_FREELIST: bool = false;
 
+    /// Maximum number of objects to keep in the freelist.
+    const MAX_FREELIST: usize = 0;
+
     /// Try to push a dead object onto this type's freelist for reuse.
     /// Returns true if the object was stored (caller must NOT free the memory).
     ///
