This issue tracker has been migrated to GitHub, and is currently read-only.
For more information, see the GitHub FAQs in the Python's Developer Guide.

Author Yonatan Goldschmidt
Recipients Yonatan Goldschmidt
Date 2020-08-13.23:52:21
SpamBayes Score -1.0
Marked as misclassified Yes
Message-id <1597362741.87.0.831562855057.issue41545@roundup.psfhosted.org>
In-reply-to
Content
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.
History
Date User Action Args
2020-08-13 23:52:22Yonatan Goldschmidtsetrecipients: + Yonatan Goldschmidt
2020-08-13 23:52:21Yonatan Goldschmidtsetmessageid: <1597362741.87.0.831562855057.issue41545@roundup.psfhosted.org>
2020-08-13 23:52:21Yonatan Goldschmidtlinkissue41545 messages
2020-08-13 23:52:21Yonatan Goldschmidtcreate