search

stackless-dev / stackless

The Stackless Python programming language
http://www.stackless.com/
Other
1.03k stars 60 forks source link

inconsistent slp_switch_stack.h #15

Closed ghost closed 7 years ago

ghost commented 10 years ago

Originally reported by: RMTEW FULL NAME (Bitbucket: rmtew, GitHub: rmtew)

(originally reported in Trac by @akruis on 2013-04-07 09:58:27)

Version: branch 2.7-slp

The generated file Stackless/platf/slp_switch_stack.h is not consistent with its source files. This is not a serious problem, because the file isn't used.

We should either remove it or recreate it.

If we want to recreate the file, we need a small patch to the mkswitch_stack.py script, because it is no compatible with recent changes of switch_amd64_unix.h.

I would prefer to remove this file from branch 2.7-slp

ghost commented 10 years ago

Original comment by RMTEW FULL NAME (Bitbucket: rmtew, GitHub: rmtew):

@akruis on 2013-04-24 16:28:33 said:

I removed the files from 2.7-slp. Changeset is [0f46dcc0abf0].

ghost commented 10 years ago

Original comment by RMTEW FULL NAME (Bitbucket: rmtew, GitHub: rmtew):

@akruis on 2013-04-07 12:29:22 said:

Replying to [comment:1 ctismer]:

slp_switch_stack was an attempt to enable optional fast stack switching.

If you think this will never be revived, feel free to remove it. ;-)

It could be an interesting option for 3.x, but not for 2.7.

ghost commented 10 years ago

Original comment by RMTEW FULL NAME (Bitbucket: rmtew, GitHub: rmtew):

@ctismer on 2013-04-07 10:22:43 said:

slp_switch_stack was an attempt to enable optional fast stack switching.

If you think this will never be revived, feel free to remove it. ;-)