Silence compiler warnings in gc.c (#117422)

gvanrossum · web-flow · commit ddf814db7440 · 2024-04-01T16:13:38.000Z
diff --git a/Python/gc.c b/Python/gc.c
@@ -1317,6 +1317,7 @@ gc_collect_young(PyThreadState *tstate,
             survivor_count++;
         }
     }
+    (void)survivor_count;  // Silence compiler warning
     gc_list_merge(&survivors, visited);
     validate_old(gcstate);
     gcstate->young.count = 0;
@@ -1329,12 +1330,14 @@ gc_collect_young(PyThreadState *tstate,
     add_stats(gcstate, 0, stats);
 }
 
+#ifndef NDEBUG
 static inline int
 IS_IN_VISITED(PyGC_Head *gc, int visited_space)
 {
     assert(visited_space == 0 || flip_old_space(visited_space) == 0);
     return gc_old_space(gc) == visited_space;
 }
+#endif
 
 struct container_and_flag {
     PyGC_Head *container;

Original file line number	Diff line number	Diff line change
`@@ -1317,6 +1317,7 @@ gc_collect_young(PyThreadState *tstate,`
`1317`	`1317`	`survivor_count++;`
`1318`	`1318`	`}`
`1319`	`1319`	`}`
	`1320`	`+ (void)survivor_count; // Silence compiler warning`
`1320`	`1321`	`gc_list_merge(&survivors, visited);`
`1321`	`1322`	`validate_old(gcstate);`
`1322`	`1323`	`gcstate->young.count = 0;`
`@@ -1329,12 +1330,14 @@ gc_collect_young(PyThreadState *tstate,`
`1329`	`1330`	`add_stats(gcstate, 0, stats);`
`1330`	`1331`	`}`
`1331`	`1332`
	`1333`	`+#ifndef NDEBUG`
`1332`	`1334`	`static inline int`
`1333`	`1335`	`IS_IN_VISITED(PyGC_Head *gc, int visited_space)`
`1334`	`1336`	`{`
`1335`	`1337`	`assert(visited_space == 0 \|\| flip_old_space(visited_space) == 0);`
`1336`	`1338`	`return gc_old_space(gc) == visited_space;`
`1337`	`1339`	`}`
	`1340`	`+#endif`
`1338`	`1341`
`1339`	`1342`	`struct container_and_flag {`
`1340`	`1343`	`PyGC_Head *container;`