#BUGFIX by mawalch
class: Tools::NewSystemBrowser
changed: #selectorMenuSaveRemove
Send #do: instead of #contains: as the block never
returns true nor false (Only if the boolean values
true and false would be used as selectors). Also:
Terminating the iteration looks unwanted and the
result is discarded anyway.
--- a/Tools__NewSystemBrowser.st Wed Sep 13 18:35:07 2017 +0200
+++ b/Tools__NewSystemBrowser.st Thu Sep 14 14:21:48 2017 +0200
@@ -49287,19 +49287,19 @@
self withSearchCursorDo:[
"/ search through all of the system
environment allMethodsDo:[:mthd |
- |sent resources newFound any|
+ |sent newFound any|
any := false.
mthd literalsDo:[:eachLiteral |
(selectorsToRemove includes:eachLiteral) ifTrue:[any := true].
"/ could be an array (as in a spec)
eachLiteral isArray ifTrue:[
- selectorsToRemove contains:[:selToRemove |
+ selectorsToRemove do:[:selToRemove |
(eachLiteral refersToLiteral:selToRemove) ifTrue:[
possiblyUsedAsSelector add:mthd.
- ]
- ]
- ]
+ ].
+ ].
+ ].
].
any ifTrue:[
selectorsToRemove do:[:eachSelectorToRemove |
@@ -49388,6 +49388,7 @@
self doRemoveMethodsUnconfirmed:methodsToRemove
"Modified: / 11-05-2012 / 10:00:46 / cg"
+ "Modified (format): / 14-09-2017 / 14:13:13 / mawalch"
!
selectorMenuSelectMethodsWithString