Issue 5465 in pharo: finding unbound

Status: Accepted
Owner: [hidden email]
Labels: Difficulty-Easy 1.4

New issue 5465 by [hidden email]: finding unbound
http://code.google.com/p/pharo/issues/detail?id=5465

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]]]]]

Should be adapted to ring API.

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)

On Sun, Mar 11, 2012 at 10:43 AM, Pavel Krivanek <[hidden email]>  
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






_______________________________________________
Pharo-bugtracker mailing list
[hidden email]
http://lists.gforge.inria.fr/cgi-bin/mailman/listinfo/pharo-bugtracker

Updates:
        Status: Duplicate
        Mergedinto: 5463

Comment #1 on issue 5465 by [hidden email]: finding unbound
http://code.google.com/p/pharo/issues/detail?id=5465

(No comment was entered for this change.)


_______________________________________________
Pharo-bugtracker mailing list
[hidden email]
http://lists.gforge.inria.fr/cgi-bin/mailman/listinfo/pharo-bugtracker