In perl.git, the branch smoke-me/feature-code-shrink has been created
<http://perl5.git.perl.org/perl.git/commitdiff/3a56084401f5e1a2432dd1e0a7aea0efd2ed28f4?hp=0000000000000000000000000000000000000000>
at 3a56084401f5e1a2432dd1e0a7aea0efd2ed28f4 (commit)
- Log -----------------------------------------------------------------
commit 3a56084401f5e1a2432dd1e0a7aea0efd2ed28f4
Author: Nicholas Clark <[email protected]>
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 9a53ca2e18f447e4e39ade4bdf9eb750a8c1a4a7
Author: Nicholas Clark <[email protected]>
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 b27dd1a8a91ec060119ea499731ff5d43e8e92ce
Author: Nicholas Clark <[email protected]>
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 fe445c4fdd855a69c8c7dc0b6c47958cc5dfb0cb
Author: Nicholas Clark <[email protected]>
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
