FogBugz (Case [Issue]11674) Kernel - MNU: receiver of ">" is nil

Previous Topic Next Topic
 
classic Classic list List threaded Threaded
1 message Options
Reply | Threaded
Open this post in threaded view
|

FogBugz (Case [Issue]11674) Kernel - MNU: receiver of ">" is nil

Pharo Issue Tracker
FogBugz Notification
avatar
Bug in Project:  Kernel: 1. Pharo Image  •  You are subscribed to this case
In your case what happens it that the suspendedContext is dead. Now when a suspended context is dead, does that mean the Process is terminated ? I guess so because it cannot be restarted (Now I'm not expert in Process management).

Therefore I think the correct code should be:

isTerminated
self isActiveProcess ifTrue: [^ false].
^ suspendedContext isNil
or: [ suspendedContext isBottomContext
and: [ suspendedContext isDead not
and: [ suspendedContext pc > suspendedContext startpc ] ] ]

Basically it is the same but a little bit clearer to me.

And I do not like:
suspendedContext pc ifNotNil: [suspendedContext pc > suspendedContext startpc]
because if suspendedContext pc is nil, then the resulting executing code would be:
suspendedContext isBottomContext and: [ nil ]
and this will result in other problems one day or the other, because this code returns nil instead of true.
Priority Priority: 3 – Must Fix Status Status: Resolved (Fix Review Needed)
Assigned To Assigned to: Philippe Back Milestone Milestone: Pharo2.0

Go to Case
No longer need updates? Unsubscribe from this case.

Don't want FogBugz notifications anymore? Update your preferences.

FogBugz

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