Jan Dubois <jand@ActiveState.com> writes: >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. Wrap them in #ifndef PERL_CORE ? > >Cheers, >-JanThread Previous | Thread Next