Re: [Pharo-project] ECompletion/Shout bug

April 23, 2012

      These are not really nested if, but rather a kind of caseOf:
implemented with elseif.
It's not to say that the style is ideal...
I did not read the code, and don't know what a range is, nor if it has
subclass specializations.
If so, maybe some actions could simply be dispatched to range.

Nicolas

Le 23 avril 2012 10:58, Esteban Lorenzano <estebanlm@gmail.com> a Ã©crit :
...
he... is not related with the error, but... did you see this method?
guessTypeOf: aNumber
Â  Â  Â  Â | range |
Â  Â  Â  Â self configureSelectorsOnly.
Â  Â  Â  Â recurseCount > 10 ifTrue: [ ^ nil ].
Â  Â  Â  Â recurseCount := recurseCount + 1.
Â  Â  Â  Â range := ranges at: aNumber.
Â  Â  Â  Â ^ range isSelf
Â  Â  Â  Â  Â  Â  Â  Â ifTrue: [ theClass ]
Â  Â  Â  Â  Â  Â  Â  Â ifFalse:
Â  Â  Â  Â  Â  Â  Â  Â  Â  Â  Â  Â [ range isSuper
Â  Â  Â  Â  Â  Â  Â  Â  Â  Â  Â  Â  Â  Â  Â  Â ifTrue: [ theClass superclass ]
Â  Â  Â  Â  Â  Â  Â  Â  Â  Â  Â  Â  Â  Â  Â  Â ifFalse:
Â  Â  Â  Â  Â  Â  Â  Â  Â  Â  Â  Â  Â  Â  Â  Â  Â  Â  Â  Â [ range isConstant
Â  Â  Â  Â  Â  Â  Â  Â  Â  Â  Â  Â  Â  Â  Â  Â  Â  Â  Â  Â  Â  Â  Â  Â ifTrue: [ range asType ]
Â  Â  Â  Â  Â  Â  Â  Â  Â  Â  Â  Â  Â  Â  Â  Â  Â  Â  Â  Â  Â  Â  Â  Â ifFalse:
Â  Â  Â  Â  Â  Â  Â  Â  Â  Â  Â  Â  Â  Â  Â  Â  Â  Â  Â  Â  Â  Â  Â  Â  Â  Â  Â  Â [ range isArgument
Â  Â  Â  Â  Â  Â  Â  Â  Â  Â  Â  Â  Â  Â  Â  Â  Â  Â  Â  Â  Â  Â  Â  Â  Â  Â  Â  Â  Â  Â  Â  Â ifTrue: [ self guessArgument: range ]
Â  Â  Â  Â  Â  Â  Â  Â  Â  Â  Â  Â  Â  Â  Â  Â  Â  Â  Â  Â  Â  Â  Â  Â  Â  Â  Â  Â  Â  Â  Â  Â ifFalse:
Â  Â  Â  Â  Â  Â  Â  Â  Â  Â  Â  Â  Â  Â  Â  Â  Â  Â  Â  Â  Â  Â  Â  Â  Â  Â  Â  Â  Â  Â  Â  Â  Â  Â  Â  Â [ range isTemporaryVariable
Â  Â  Â  Â  Â  Â  Â  Â  Â  Â  Â  Â  Â  Â  Â  Â  Â  Â  Â  Â  Â  Â  Â  Â  Â  Â  Â  Â  Â  Â  Â  Â  Â  Â  Â  Â  Â  Â  Â  Â ifTrue: [ self guessTempVarClass: range ]
Â  Â  Â  Â  Â  Â  Â  Â  Â  Â  Â  Â  Â  Â  Â  Â  Â  Â  Â  Â  Â  Â  Â  Â  Â  Â  Â  Â  Â  Â  Â  Â  Â  Â  Â  Â  Â  Â  Â  Â ifFalse:
Â  Â  Â  Â  Â  Â  Â  Â  Â  Â  Â  Â  Â  Â  Â  Â  Â  Â  Â  Â  Â  Â  Â  Â  Â  Â  Â  Â  Â  Â  Â  Â  Â  Â  Â  Â  Â  Â  Â  Â  Â  Â  Â  Â [ range isInstanceVariable
Â  Â  Â  Â  Â  Â  Â  Â  Â  Â  Â  Â  Â  Â  Â  Â  Â  Â  Â  Â  Â  Â  Â  Â  Â  Â  Â  Â  Â  Â  Â  Â  Â  Â  Â  Â  Â  Â  Â  Â  Â  Â  Â  Â  Â  Â  Â  Â ifTrue: [ self guessInstVarClass: range ]
Â  Â  Â  Â  Â  Â  Â  Â  Â  Â  Â  Â  Â  Â  Â  Â  Â  Â  Â  Â  Â  Â  Â  Â  Â  Â  Â  Â  Â  Â  Â  Â  Â  Â  Â  Â  Â  Â  Â  Â  Â  Â  Â  Â  Â  Â  Â  Â ifFalse:
Â  Â  Â  Â  Â  Â  Â  Â  Â  Â  Â  Â  Â  Â  Â  Â  Â  Â  Â  Â  Â  Â  Â  Â  Â  Â  Â  Â  Â  Â  Â  Â  Â  Â  Â  Â  Â  Â  Â  Â  Â  Â  Â  Â  Â  Â  Â  Â  Â  Â  Â  Â [ range isClassVariable
Â  Â  Â  Â  Â  Â  Â  Â  Â  Â  Â  Â  Â  Â  Â  Â  Â  Â  Â  Â  Â  Â  Â  Â  Â  Â  Â  Â  Â  Â  Â  Â  Â  Â  Â  Â  Â  Â  Â  Â  Â  Â  Â  Â  Â  Â  Â  Â  Â  Â  Â  Â  Â  Â  Â  Â ifTrue: [ self guessClassVarClass: range ]
Â  Â  Â  Â  Â  Â  Â  Â  Â  Â  Â  Â  Â  Â  Â  Â  Â  Â  Â  Â  Â  Â  Â  Â  Â  Â  Â  Â  Â  Â  Â  Â  Â  Â  Â  Â  Â  Â  Â  Â  Â  Â  Â  Â  Â  Â  Â  Â  Â  Â  Â  Â  Â  Â  Â  Â ifFalse: [ range isGlobal ifTrue: [ self guessGlobal: aNumber ] ] ] ] ] ] ] ]
when I was younger, at informatics school, they told me that "3 nested ifs means bad design, with no doubt"... what about 7?
On Apr 23, 2012, at 10:30 AM, Esteban Lorenzano wrote:
...
yes, I can reproduce this one... taking care of it now :)
On Apr 23, 2012, at 10:11 AM, StÃ©phane Ducasse wrote:
...
Note that the bug shows up if you edit the code in the result of the implementors.
Esteban can you reproduce the bug?
Stef
On Apr 23, 2012, at 9:18 AM, StÃ©phane Ducasse wrote:
...
Hi
I get a bug with OCompletion/Ecompletion
TextEditor>>changeEmphasis
try to type
Â  Â  menuList nextPut:
and you get a bug.
Now in addition I do not understand what I see in the debugger:
As you see model returns a debugger and the debugger title show FlatMessageListBrowser
Stef
<Screen Shot 2012-04-23 at 9.08.45 AM.pdf>

Re: [Pharo-project] ECompletion/Shout bug

Nicolas Cellier