develooper Front page | perl.perl5.porters | Postings from November 2003

Re: need your help to do some (simple?) patching (was: Re: [perl #24463] DBI tests fail with perl 5.8.2 and -Duseithreads -Dusemultiplicity -Duse64bitint)

Thread Previous | Thread Next
From:
Jan Dubois
Date:
November 15, 2003 16:22
Subject:
Re: need your help to do some (simple?) patching (was: Re: [perl #24463] DBI tests fail with perl 5.8.2 and -Duseithreads -Dusemultiplicity -Duse64bitint)
Message ID:
m7gdrv4b98hnknpo979l8ac0lltp9aq7o6@4ax.com
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,
-Jan


Thread Previous | Thread Next


nntp.perl.org: Perl Programming lists via nntp and http.
Comments to Ask Bjørn Hansen at ask@perl.org | Group listing | About