In perl.git, the branch smoke-me/feature-code-shrink has been created
<http://perl5.git.perl.org/perl.git/commitdiff/a1a16836a55d44d3db38c2c78b8ee95ec59d8eed?hp=0000000000000000000000000000000000000000>
at a1a16836a55d44d3db38c2c78b8ee95ec59d8eed (commit)
- Log -----------------------------------------------------------------
commit a1a16836a55d44d3db38c2c78b8ee95ec59d8eed
Author: Nicholas Clark <nick@ccl4.org>
Date: Tue Feb 28 23:30:30 2012 +0100
In B::Deparse::_features_from_bundle(), don't call feature::current_bundle()
Instead, directly access feature's package variables, as B::Deparse already
does in 14 other places. (It also has its tentacles firmly into strict
and warning's package variables - it's not fussy)
feature::current_bundle() was not part of the documented API of feature
either, so B::Deparse wasn't clean previously.
M dist/B-Deparse/Deparse.pm
commit 29230a29cbb645b541eb01233fcb315b6de4d9fd
Author: Nicholas Clark <nick@ccl4.org>
Date: Tue Feb 28 11:11:02 2012 +0100
In Deparse, use $feature::hint_mask directly, instead of copying its value.
Also, require feature unconditionally.
Deparse already directly uses data from feature, switch and warnings, so
this isn't a new trend in encapsulation breakage. Previously Deparse copied
the value of $feature::hint_mask, and lazily loaded require in 4 places.
M dist/B-Deparse/Deparse.pm
commit 67c2a9595509031c2546da285244881d50062c3d
Author: Nicholas Clark <nick@ccl4.org>
Date: Tue Feb 28 23:09:02 2012 +0100
In B::Deparse, refactor common code into _features_from_bundle().
M dist/B-Deparse/Deparse.pm
commit c9f42dba7831eb3d30ffeb486a1516920ccb133d
Author: Nicholas Clark <nick@ccl4.org>
Date: Tue Feb 28 22:57:31 2012 +0100
In B::Deparse, refactor the two places that feature::current_bundle()
Converge the code, so that it's easy to extract out into a subroutine.
M dist/B-Deparse/Deparse.pm
-----------------------------------------------------------------------
--
Perl5 Master Repository