On Sun, Mar 11, 2012 at 3:33 PM, Eliot Miranda
<eliot.miranda@gmail.com> wrote:
Hi Pavel,
� � I presume they are found by the following:
SystemNavigation new browseAllSelect:
[:m|
m literals anySatisfy:
[:l|
l isVariableBinding
and: [l key isSymbol "avoid class-side methodClass literals"
and: [(m methodClass bindingOf: l key) isNil]]]]
This could be a "browse unbound", and could be included in release tests via e.g.
SystemNavigation>methodsWithUnboundGlobals
^self allSelect:
[:m|
m literals anySatisfy:
[:l|
l isVariableBinding
and: [l key isSymbol "avoid class-side methodClass literals"
and: [(m methodClass bindingOf: l key) isNil]]]]
(or whereever Pharo puts browsing queries these days)
and I suppose it should exclude Undeclared variables, so this is better:
SystemNavigation new browseAllSelect:
[:m|
m literals anySatisfy:
[:l|
l isVariableBinding
and: [l key isSymbol "avoid class-side methodClass literals"
and: [(m methodClass bindingOf: l key) isNil
and: [(Undeclared includesAssociation: l) not]]]]]
�
On Sun, Mar 11, 2012 at 10:43 AM, Pavel Krivanek
<pavel.krivanek@gmail.com> wrote:
Hi,
the method DefaultSettingStyle>>#load includes several obsolete
classes (RBProgramNode, RBConfigurableFormatter,
RefactoryChangeManager). It is interesting that this inconsistency is
not reported by Undeclared nor obsoleteClasses (so release tests are
green). The literals array contains associations to nil pointed only
from the literal array of this method.
http://code.google.com/p/pharo/issues/detail?id=5463
Cheers,
-- Pavel
--
best,Eliot