On Sat, 15 Nov 2003 19:07:58 -0500, Casey West <casey@geeknest.com> wrote: >It was Friday, November 14, 2003 when Stas Bekman took the soap box, saying: >: Can someone with some spare time on they hands help us to introduce the new >: APIs which accept pTHX_ instead of declaring dTHX, and deploy them in the >: core, that would be a very noble conribution. >: >: After all it requires no understanding of perl guts, just browse through >: perl/*[ch] files and: >: >: 1) find a function/macro that does dTHX declaration, >: 2) use #define wrappers to create a new API which accepts pTHX_, as the >: first argument, >: 3) then find the places in the core where those functions are used and >: replace with the new API that explicitly passes pTHX. >: 4) Return to 1 untill no more functions using dTHX and having no parallel >: API using pTHX_ are found. >: >: give a try to one function and post the patch here, so the gurus can >: approve it and the rest should be just a mechanical work. > >In an attempt to be the monkey, I did what seemed correct. All tests >passed, did I do something right (assuming the latter)? If not, I >could use some more clues. Thanks. No, you should *not* change any of the *_nocontext() functions. They are only there for source code compatibility of older extensions. They all already have a corresponding version without the "_nocontext" suffix. The core code should *only* call the latter. A simple way to make sure that the core doesn't call any of the *_nocontext() functions would be to just delete all of them and see if Perl still builds and compiles. Cheers, -JanThread Previous | Thread Next