Issue 7417 in pharo: When future can't wait (fix potential caveat with Delay)

Status: Accepted
Owner: [hidden email]
Labels: Type-Bug Milestone-2.0

New issue 7417 by [hidden email]: When future can't wait (fix potential  
caveat with Delay)
http://code.google.com/p/pharo/issues/detail?id=7417


As we discussed earlier, reusing Delays is error prone.
This piece of code have potential problems with it,
since single Delay instance is scheduled multiple times in a loop,

which actually could turn into an infinite wait cycle if  
image "unfortunately" will restart when process waits for it.

Attachments:
        Time class.IgorStasenko.cs  408 bytes


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

Updates:
        Status: HumanReviewNeeded

Comment #1 on issue 7417 by [hidden email]: When future can't wait  
(fix potential caveat with Delay)
http://code.google.com/p/pharo/issues/detail?id=7417

(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 #2 on issue 7417 by [hidden email]: When future can't wait  
(fix potential caveat with Delay)
http://code.google.com/p/pharo/issues/detail?id=7417

in 2.0 513


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

Updates:
        Status: FixReviewNeeded

Comment #3 on issue 7417 by [hidden email]: When future can't wait  
(fix potential caveat with Delay)
http://code.google.com/p/pharo/issues/detail?id=7417

No, I think I will do the monkey first...

Name:  
SLICE-Issue-7417-When-future-cant-wait-fix-potential-caveat-with-Delay-MarcusDenker.1
Author: MarcusDenker
Time: 5 February 2013, 10:53:50.849 am
UUID: 5871da5c-ff76-4aad-a666-42e2ecdda034
Ancestors:
Dependencies: Kernel-MarcusDenker.1301

Issue 7417: When future can't wait (fix potential caveat with Delay)


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

Updates:
        Status: MonkeyIsChecking

Comment #4 on issue 7417 by [hidden email]: When future can't wait  
(fix potential caveat with Delay)
http://code.google.com/p/pharo/issues/detail?id=7417#c4

The Monkey is currently checking this issue. Please don't change it!


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

Updates:
        Status: ValidatedByTheMonkey
        Labels: CheckedIn20512

Comment #5 on issue 7417 by [hidden email]: When future can't wait  
(fix potential caveat with Delay)
http://code.google.com/p/pharo/issues/detail?id=7417#c5

This Issue has been checked by Ulysse the Monkey
       
7632 tests passed in 00:02:56s:
===============================
        CollectionsTests-Arrayed (553)
        CollectionsTests-Atomic (12)
        CollectionsTests-Sequenceable (912)
        CollectionsTests-SplitJoin (27)
        CollectionsTests-Stack (16)
        CollectionsTests-Streams (37)
        CollectionsTests-Strings (611)
        CollectionsTests-Support (12)
        CollectionsTests-Unordered (1954)
        CollectionsTests-Weak (739)
        CompilerTests (181)
        KernelTests-Chronology (1186)
        KernelTests-Classes (138)
        KernelTests-Exception (5)
        KernelTests-Methods (359)
        KernelTests-Numbers (552)
        KernelTests-Objects (172)
        KernelTests-Pragmas (6)
        KernelTests-Processes (76)
        SUnit-Core-Extensions (3)
        SUnit-Core-Utilities (3)
        SUnit-Tests-Core (78)

----------------------------------------------------------
Loaded Source:  
SLICE-Issue-7417-When-future-cant-wait-fix-potential-caveat-with-Delay-MarcusDenker.1  
from http://ss3.gemstone.com/ss/PharoInbox"Time+class.IgorStasenko.cs" from  
Comment #0
Tested using Pharo-2.0-20512-a on NBCoInterpreter  
NativeBoost-CogPlugin-IgorStasenko.15 uuid:  
44b6b681-38f1-4a9e-b6ee-8769b499576a Nov 27 2012
NBCogit NativeBoost-CogPlugin-IgorStasenko.15 uuid:  
44b6b681-38f1-4a9e-b6ee-8769b499576a Nov 27 2012
https://git.gitorious.org/cogvm/blessed.git Commit:  
40ac7e7bdec6fef0e934d2c019b86db996053912 Date: 2012-11-19 18:54:49 +0100  
By: Mariano Martinez Peck <[hidden email]> Jenkins build #146


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

Updates:
        Status: FixToInclude

Comment #6 on issue 7417 by [hidden email]: When future can't wait  
(fix potential caveat with Delay)
http://code.google.com/p/pharo/issues/detail?id=7417

(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 #7 on issue 7417 by [hidden email]: When future can't wait  
(fix potential caveat with Delay)
http://code.google.com/p/pharo/issues/detail?id=7417

in 2.0 513


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

Issue 7417: When future can't wait (fix potential caveat with Delay)
http://code.google.com/p/pharo/issues/detail?id=7417

This issue is now blocking issue pharo:7129.
See http://code.google.com/p/pharo/issues/detail?id=7129

--
You received this message because you are listed in the owner
or CC fields of this issue, or because you starred this issue.
You may adjust your issue notification preferences at:
http://code.google.com/hosting/settings

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