8000 clear weakrefs unconditionally in extension modules · python/cpython@9024079 · GitHub
[go: up one dir, main page]
Skip to content

Commit 9024079

Browse files
xuantenghxuantengh
committed
clear weakrefs unconditionally in extension modules
1 parent cf33c4a commit 9024079

20 files changed

+22
-59
lines changed

‎Modules/_collectionsmodule.c

Lines changed: 1 addition & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -1532,9 +1532,7 @@ deque_dealloc(PyObject *self)
15321532
Py_ssize_t i;
15331533

15341534
PyObject_GC_UnTrack(deque);
1535-
if (deque->weakreflist != NULL) {
1536-
PyObject_ClearWeakRefs(self);
1537-
}
1535+
PyObject_ClearWeakRefs(self);
15381536
if (deque->leftblock != NULL) {
15391537
(void)deque_clear(self);
15401538
assert(deque->leftblock != NULL);

‎Modules/_elementtree.c

Lines changed: 1 addition & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -690,8 +690,7 @@ element_dealloc(PyObject *op)
690690
/* bpo-31095: UnTrack is needed before calling any callbacks */
691691
PyObject_GC_UnTrack(self);
692692

693-
if (self->weakreflist != NULL)
694-
PyObject_ClearWeakRefs(op);
693+
PyObject_ClearWeakRefs(op);
695694

696695
/* element_gc_clear clears all references and deallocates extra
697696
*/

‎Modules/_functoolsmodule.c

Lines changed: 2 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -351,9 +351,7 @@ partial_dealloc(PyObject *self)
351351
PyTypeObject *tp = Py_TYPE(self);
352352
/* bpo-31095: UnTrack is needed before calling any callbacks */
353353
PyObject_GC_UnTrack(self);
354-
if (partialobject_CAST(self)->weakreflist != NULL) {
355-
PyObject_ClearWeakRefs(self);
356-
}
354+
PyObject_ClearWeakRefs(self);
357355
(void)partial_clear(self);
358356
tp->tp_free(self);
359357
Py_DECREF(tp);
@@ -1621,9 +1619,7 @@ lru_cache_dealloc(PyObject *op)
16211619
PyTypeObject *tp = Py_TYPE(obj);
16221620
/* bpo-31095: UnTrack is needed before calling any callbacks */
16231621
PyObject_GC_UnTrack(obj);
1624-
if (obj->weakreflist != NULL) {
1625-
PyObject_ClearWeakRefs(op);
1626-
}
1622+
PyObject_ClearWeakRefs(op);
16271623

16281624
(void)lru_cache_tp_clear(op);
16291625
tp->tp_free(obj);

‎Modules/_io/bufferedio.c

Lines changed: 1 addition & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -421,8 +421,7 @@ buffered_dealloc(PyObject *op)
421421
return;
422422
_PyObject_GC_UNTRACK(self);
423423
self->ok = 0;
424-
if (self->weakreflist != NULL)
425-
PyObject_ClearWeakRefs(op);
424+
PyObject_ClearWeakRefs(op);
426425
if (self->buffer) {
427426
PyMem_Free(self->buffer);
428427
self->buffer = NULL;

‎Modules/_io/fileio.c

Lines changed: 1 addition & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -570,9 +570,7 @@ fileio_dealloc(PyObject *op)
570570
PyMem_Free(self->stat_atopen);
571571
self->stat_atopen = NULL;
572572
}
573-
if (self->weakreflist != NULL) {
574-
PyObject_ClearWeakRefs(op);
575-
}
573+
PyObject_ClearWeakRefs(op);
576574
(void)fileio_clear(op);
577575

578576
PyTypeObject *tp = Py_TYPE(op);

‎Modules/_io/iobase.c

Lines changed: 1 addition & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -383,8 +383,7 @@ iobase_dealloc(PyObject *op)
383383
}
384384
PyTypeObject *tp = Py_TYPE(self);
385385
_PyObject_GC_UNTRACK(self);
386-
if (self->weakreflist != NULL)
387-
PyObject_ClearWeakRefs(op);
386+
PyObject_ClearWeakRefs(op);
388387
Py_CLEAR(self->dict);
389388
tp->tp_free(self);
390389
Py_DECREF(tp);

‎Modules/_io/textio.c

Lines changed: 1 addition & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -1469,8 +1469,7 @@ textiowrapper_dealloc(PyObject *op)
14691469
return;
14701470
self->ok = 0;
14711471
_PyObject_GC_UNTRACK(self);
1472-
if (self->weakreflist != NULL)
1473-
PyObject_ClearWeakRefs(op);
1472+
PyObject_ClearWeakRefs(op);
14741473
(void)textiowrapper_clear(op);
14751474
tp->tp_free(self);
14761475
Py_DECREF(tp);

‎Modules/_io/winconsoleio.c

Lines changed: 1 addition & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -518,8 +518,7 @@ winconsoleio_dealloc(PyObject *op)
518518
if (_PyIOBase_finalize(op) < 0)
519519
return;
520520
_PyObject_GC_UNTRACK(self);
521-
if (self->weakreflist != NULL)
522-
PyObject_ClearWeakRefs(op);
521+
PyObject_ClearWeakRefs(op);
523522
Py_CLEAR(self->dict);
524523
tp->tp_free(self);
525524
Py_DECREF(tp);

‎Modules/_queuemodule.c

Lines changed: 1 addition & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -221,9 +221,7 @@ simplequeue_dealloc(PyObject *op)
221221

222222
PyObject_GC_UnTrack(self);
223223
(void)simplequeue_clear(op);
224-
if (self->weakreflist != NULL) {
225-
PyObject_ClearWeakRefs(op);
226-
}
224+
PyObject_ClearWeakRefs(op);
227225
tp->tp_free(self);
228226
Py_DECREF(tp);
229227
}

‎Modules/_sqlite/blob.c

Lines changed: 1 addition & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -56,9 +56,7 @@ blob_dealloc(PyObject *op)
5656

5757
close_blob(self);
5858

59-
if (self->in_weakreflist != NULL) {
60-
PyObject_ClearWeakRefs(op);
61-
}
59+
PyObject_ClearWeakRefs(op);
6260
(void)tp->tp_clear(op);
6361
tp->tp_free(self);
6462
Py_DECREF(tp);

‎Modules/_sqlite/cursor.c

Lines changed: 1 addition & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -185,9 +185,7 @@ cursor_dealloc(PyObject *op)
185185
pysqlite_Cursor *self = _pysqlite_Cursor_CAST(op);
186186
PyTypeObject *tp = Py_TYPE(self);
187187
PyObject_GC_UnTrack(self);
188-
if (self->in_weakreflist != NULL) {
189-
PyObject_ClearWeakRefs(op);
190-
}
188+
PyObject_ClearWeakRefs(op);
191189
(void)tp->tp_clear(op);
192190
tp->tp_free(self);
193191
Py_DECREF(tp);

‎Modules/_sre/sre.c

Lines changed: 1 addition & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -736,10 +736,7 @@ pattern_dealloc(PyObject *self)
736736
{
737737
PyTypeObject *tp = Py_TYPE(self);
738738
PyObject_GC_UnTrack(self);
739-
PatternObject *obj = _PatternObject_CAST(self);
740-
if (obj->weakreflist != NULL) {
741-
PyObject_ClearWeakRefs(self);
742-
}
739+
PyObject_ClearWeakRefs(self);
743740
(void)pattern_clear(self);
744741
tp->tp_free(self);
745742
Py_DECREF(tp);

‎Modules/_struct.c

Lines changed: 1 addition & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -1794,9 +1794,7 @@ s_dealloc(PyObject *op)
17941794
PyStructObject *s = PyStructObject_CAST(op);
17951795
PyTypeObject *tp = Py_TYPE(s);
17961796
PyObject_GC_UnTrack(s);
1797-
if (s->weakreflist != NULL) {
1798-
PyObject_ClearWeakRefs(op);
1799-
}
1797+
PyObject_ClearWeakRefs(op);
18001798
if (s->s_codes != NULL) {
18011799
PyMem_Free(s->s_codes);
18021800
}

‎Modules/_testcapi/heaptype.c

Lines changed: 1 addition & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -1034,8 +1034,7 @@ heapctypewithweakref_dealloc(PyObject *op)
10341034
{
10351035
HeapCTypeWithWeakrefObject *self = (HeapCTypeWithWeakrefObject*)op;
10361036
PyTypeObject *tp = Py_TYPE(self);
1037-
if (self->weakreflist != NULL)
1038-
PyObject_ClearWeakRefs((PyObject *) self);
1037+
PyObject_ClearWeakRefs((PyObject *) self);
10391038
Py_XDECREF(self->weakreflist);
10401039
PyObject_Free(self);
10411040
Py_DECREF(tp);

‎Modules/_testlimitedcapi/heaptype_relative.c

Lines changed: 1 addition & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -297,9 +297,7 @@ heapctypewithrelativeweakref_dealloc(PyObject* self)
297297
{
298298
PyTypeObject *tp = Py_TYPE(self);
299299
HeapCTypeWithWeakrefStruct *data = PyObject_GetTypeData(self, tp);
300-
if (data->weakreflist != NULL) {
301-
PyObject_ClearWeakRefs(self);
302-
}
300+
PyObject_ClearWeakRefs(self);
303301
Py_XDECREF(data->weakreflist);
304302
PyObject_Free(self);
305303
Py_DECREF(tp);

‎Modules/_threadmodule.c

Lines changed: 2 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -1365,9 +1365,7 @@ static void
13651365
localdummy_dealloc(PyObject *op)
13661366
{
13671367
localdummyobject *self = localdummyobject_CAST(op);
1368-
if (self->weakreflist != NULL) {
1369-
PyObject_ClearWeakRefs(op);
1370-
}
1368+
PyObject_ClearWeakRefs(op);
13711369
PyTypeObject *tp = Py_TYPE(self);
13721370
tp->tp_free(self);
13731371
Py_DECREF(tp);
@@ -1537,9 +1535,7 @@ local_dealloc(PyObject *op)
15371535
localobject *self = localobject_CAST(op);
15381536
/* Weakrefs must be invalidated right now, otherwise they can be used
15391537
from code called below, which is very dangerous since Py_REFCNT(self) == 0 */
1540-
if (self->weakreflist != NULL) {
1541-
PyObject_ClearWeakRefs(op);
1542-
}
1538+
PyObject_ClearWeakRefs(op);
15431539
PyObject_GC_UnTrack(self);
15441540
(void)local_clear(op);
15451541
PyTypeObject *tp = Py_TYPE(self);

‎Modules/_zoneinfo.c

Lines changed: 1 addition & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -375,9 +375,7 @@ zoneinfo_dealloc(PyObject *obj_self)
375375
PyTypeObject *tp = Py_TYPE(self);
376376
PyObject_GC_UnTrack(self);
377377

378-
if (self->weakreflist != NULL) {
379-
PyObject_ClearWeakRefs(obj_self);
380-
}
378+
PyObject_ClearWeakRefs(obj_self);
381379

382380
if (self->trans_list_utc != NULL) {
383381
PyMem_Free(self->trans_list_utc);

‎Modules/arraymodule.c

Lines changed: 1 addition & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -728,9 +728,7 @@ array_dealloc(PyObject *op)
728728
PyObject_GC_UnTrack(op);
729729

730730
arrayobject *self = arrayobject_CAST(op);
731-
if (self->weakreflist != NULL) {
732-
PyObject_ClearWeakRefs(op);
733-
}
731+
PyObject_ClearWeakRefs(op);
734732
if (self->ob_item != NULL) {
735733
PyMem_Free(self->ob_item);
736734
}

‎Modules/itertoolsmodule.c

Lines changed: 1 addition & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -1018,8 +1018,7 @@ static int
10181018
tee_clear(PyObject *op)
10191019
{
10201020
teeobject *to = teeobject_CAST(op);
1021-
if (to->weakreflist != NULL)
1022-
PyObject_ClearWeakRefs(op);
1021+
PyObject_ClearWeakRefs(op);
10231022
Py_CLEAR(to->dataobj);
10241023
return 0;
10251024
}

‎Modules/mmapmodule.c

Lines changed: 1 addition & 2 deletions
6FCF
Original file line numberDiff line numberDiff line change
@@ -163,8 +163,7 @@ mmap_object_dealloc(PyObject *op)
163163
Py_END_ALLOW_THREADS
164164
#endif /* UNIX */
165165

166-
if (m_obj->weakreflist != NULL)
167-
PyObject_ClearWeakRefs(op);
166+
PyObject_ClearWeakRefs(op);
168167

169168
tp->tp_free(m_obj);
170169
Py_DECREF(tp);

0 commit comments

Comments
 (0)
0