Issue 5463 in pharo: Undeclared in DefaultSettingStyle>>#load

Status: New
Owner: [hidden email]

New issue 5463 by [hidden email]: Undeclared in  
DefaultSettingStyle>>#load
http://code.google.com/p/pharo/issues/detail?id=5463

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.


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

Updates:
        Status: FixReviewNeeded

Comment #1 on issue 5463 by [hidden email]: Undeclared in  
DefaultSettingStyle>>#load
http://code.google.com/p/pharo/issues/detail?id=5463

this fix:
- adds methodsWithUnboundGlobals to SystemNavigation
- adds testMethodsWithUnboundGlobals to ReleaseTest
- fixes DefaultSettingStyle to use RB classes safely and fixes  
resetDefaultSettings usage

Attachments:
        methodsWithUnboundGlobals.2.cs  4.2 KB


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

Updates:
        Labels: Type-Bug Milestone-1.4

Comment #2 on issue 5463 by [hidden email]: Undeclared in  
DefaultSettingStyle>>#load
http://code.google.com/p/pharo/issues/detail?id=5463

(No comment was entered for this change.)


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

Updates:
        Status: Integrated

Comment #3 on issue 5463 by [hidden email]: Undeclared in  
DefaultSettingStyle>>#load
http://code.google.com/p/pharo/issues/detail?id=5463

in 14388


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

Comment #4 on issue 5463 by [hidden email]: Undeclared in  
DefaultSettingStyle>>#load
http://code.google.com/p/pharo/issues/detail?id=5463

Issue 5465 has been merged into this issue.


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