[issue35466] Use a linked list for the ceval pending calls.

2019-03-01 Thread Eric Snow
Eric Snow added the comment: At this point I'm not terribly interested in this. :) -- resolution: -> wont fix stage: patch review -> resolved status: open -> closed ___ Python tracker

[issue35466] Use a linked list for the ceval pending calls.

2018-12-11 Thread Eric Snow
Eric Snow added the comment: I suppose performance could have been a motivator originally. However, I don't see a benefit now that signals handlers are no longer pending calls. If it's a real problem then we can leverage a free list. As to complexity, the motivator for this change was

[issue35466] Use a linked list for the ceval pending calls.

2018-12-11 Thread Serhiy Storchaka
Serhiy Storchaka added the comment: Was not a circular array used intentionally because this allows to avoid calling malloc()? And the proposed code looks more complicated to me than the current code. -- nosy: +serhiy.storchaka ___ Python tracker

[issue35466] Use a linked list for the ceval pending calls.

2018-12-11 Thread Eric Snow
Change by Eric Snow : -- keywords: +patch pull_requests: +10354 stage: needs patch -> patch review ___ Python tracker ___ ___

[issue35466] Use a linked list for the ceval pending calls.

2018-12-11 Thread Eric Snow
New submission from Eric Snow : Currently the list of pending calls (see Include/internal/pycore_ceval.h) is implemented as a circular buffer. A linked list would be easier to understand and modify. It also allows for removing the restriction on the number of pending calls. --