diff options
author | Raymond Hettinger <python@rcn.com> | 2017-02-02 08:24:48 -0800 |
---|---|---|
committer | Raymond Hettinger <python@rcn.com> | 2017-02-02 08:24:48 -0800 |
commit | 14593e19e199dd34f268b35987ddb288bf022039 (patch) | |
tree | eb38f9abb608183ed45cbe247bcb2c08b5810a1e | |
parent | 4f4c038c0cd2a312cf9155e75f1be041b5085edd (diff) | |
download | cpython-14593e19e199dd34f268b35987ddb288bf022039.tar.gz |
Remove unnecessary variables.
* so->used never gets changed during a resize
* so->filled only changes when dummies are present and being eliminated
-rw-r--r-- | Objects/setobject.c | 7 |
1 files changed, 2 insertions, 5 deletions
diff --git a/Objects/setobject.c b/Objects/setobject.c index 59ed7955e6..c72c0fae62 100644 --- a/Objects/setobject.c +++ b/Objects/setobject.c @@ -296,8 +296,6 @@ set_table_resize(PySetObject *so, Py_ssize_t minused) { Py_ssize_t newsize; setentry *oldtable, *newtable, *entry; - Py_ssize_t oldfill = so->fill; - Py_ssize_t oldused = so->used; Py_ssize_t oldmask = so->mask; size_t newmask; int is_oldtable_malloced; @@ -352,21 +350,20 @@ set_table_resize(PySetObject *so, Py_ssize_t minused) /* Make the set empty, using the new table. */ assert(newtable != oldtable); memset(newtable, 0, sizeof(setentry) * newsize); - so->fill = oldused; - so->used = oldused; so->mask = newsize - 1; so->table = newtable; /* Copy the data over; this is refcount-neutral for active entries; dummy entries aren't copied over, of course */ newmask = (size_t)so->mask; - if (oldfill == oldused) { + if (so->fill == so->used) { for (entry = oldtable; entry <= oldtable + oldmask; entry++) { if (entry->key != NULL) { set_insert_clean(newtable, newmask, entry->key, entry->hash); } } } else { + so->fill = so->used; for (entry = oldtable; entry <= oldtable + oldmask; entry++) { if (entry->key != NULL && entry->key != dummy) { set_insert_clean(newtable, newmask, entry->key, entry->hash); |