develooper Front page | perl.perl5.porters | Postings from June 2014

[perl #122122] [PATCH] PERL_UNUSED_CONTEXT audit

From:
Jarkko Hietaniemi via RT
Date:
June 28, 2014 17:08
Subject:
[perl #122122] [PATCH] PERL_UNUSED_CONTEXT audit
Message ID:
rt-4.0.18-12936-1403975292-552.122122-14-0@perl.org
> So this patch isn't currently suitable for blead.

bulk88, if you take a look at the latest blead, I have been actually also working towards removing unused contexts (or making them conditional on whatever) (because I was turning on as many gcc -Wflags as I could easily).  I haven't looked at your patch in detail, but I think you will find we covered many of the same spots.

See for example:

http://perl5.git.perl.org/perl.git/commit/23491f1decae17401c5c08e7f8a11e0d30f4f0e2
http://perl5.git.perl.org/perl.git/commit/dd791c72c136e634557810ed390db9ed30f1576f
http://perl5.git.perl.org/perl.git/commit/8161153450000b2b806ca000eae18af7e430d3d9

(hugmeir also did some similar work in early June, but I think that was your original inspiration)

Also I removed unused dVAR (which I think under some cases are actually also dTHX):

http://perl5.git.perl.org/perl.git/commit/23491f1decae17401c5c08e7f8a11e0d30f4f0e2
http://perl5.git.perl.org/perl.git/commit/c97ab48957e7f73939e2a77e6312b44dc2408398






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