Message375355
I have a construct in my code where I wrap a function's logic with `gc.disable()` to prevent GC from triggering some race condition.
Today this construct got a bit more complicated, and another function had to use this "trick". Now there are 2 flows:
foo():
gc.disable()
....
gc.enable()
bar()
....
gc.disable()
...
# bar calls foo, which also messes with the gc
foo()
gc.disable()
...
gc.enable()
...
gc.enable()
...
I'd expected the GC to be truly enabled only on the second call to `enable()`, but apparently it's enabled on the first call :( Both `gc.enable()` and `gc.disable()` just write `1`/`0` to `gcstate->enabled`, respectively.
For the meantime I wrote a simple wrapper class to do this counting (finally calling `enable()` only when necessary).
Another option is for the code to check first "is GC enabled?" before disabling, and before enabling again, remember whether it really disabled it or not.
But I think those manual workarounds are more error prone, and it's easier to forget to use it them in all sites (compared to using the "right" API from the standard module), so I was considering if we can add an API that encapsulate this counting: an enable-disable pair that makes sure GC is only enabled back when the number of "enable" calls matches the number of "disable" calls. |
|
Date |
User |
Action |
Args |
2020-08-13 23:52:22 | Yonatan Goldschmidt | set | recipients:
+ Yonatan Goldschmidt |
2020-08-13 23:52:21 | Yonatan Goldschmidt | set | messageid: <1597362741.87.0.831562855057.issue41545@roundup.psfhosted.org> |
2020-08-13 23:52:21 | Yonatan Goldschmidt | link | issue41545 messages |
2020-08-13 23:52:21 | Yonatan Goldschmidt | create | |
|