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:
Nick Ing-Simmons
Date:
November 17, 2003 00:50
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:
20031117084434.2893.4@llama.elixent.com
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,
>-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