In perl.git, the branch nicholas/feature-code-shrink has been created
<http://perl5.git.perl.org/perl.git/commitdiff/ae0ed4a664771994ed0657565e4930543f524469?hp=0000000000000000000000000000000000000000>
at ae0ed4a664771994ed0657565e4930543f524469 (commit)
- Log -----------------------------------------------------------------
commit ae0ed4a664771994ed0657565e4930543f524469
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 67796dcf7a535e6fb8dcfb1467e3a802d7452076
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 1fd9cf1d698de217573cd523a125c8b4ebd498d7
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 764455d648b5ba6c32ab47ed783865304d65a524
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
