On Sun, Mar 11, 2012 at 3:33 PM, Eliot Miranda <[email protected]>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 <[email protected] > > 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 > > -- best, Eliot
