[issue42767] Review usage of atomic variables in signamodule.c

2020-12-28 Thread Ammar Askar
Ammar Askar added the comment: > For me, the most surprising part is Handlers[signum].tripped which is > declared as volatile *and* an atomic variable. I think it's just following this part of the C spec (Some background in bpo-12060): > or refers to any object with static storage duration

[issue42767] Review usage of atomic variables in signamodule.c

2020-12-28 Thread hai shi
Change by hai shi : -- nosy: +shihai1991 ___ Python tracker ___ ___ Python-bugs-list mailing list Unsubscribe:

[issue42767] Review usage of atomic variables in signamodule.c

2020-12-28 Thread Dong-hee Na
Change by Dong-hee Na : -- nosy: +corona10 ___ Python tracker ___ ___ Python-bugs-list mailing list Unsubscribe:

[issue42767] Review usage of atomic variables in signamodule.c

2020-12-28 Thread STINNER Victor
New submission from STINNER Victor : In bpo-41713, I ported the _signal module to the multi-phase initialization API. I tried to move the signal state into a structure to cleanup the code, but I was scared by the following atomic variables: --- static volatile struct {