[Issue 8214] blocking option for TaskPool.finish()

http://d.puremagic.com/issues/show_bug.cgi?id=8214 John Belmonte j...@neggie.net changed: What|Removed |Added Status|NEW |ASSIGNED

[Issue 8214] blocking option for TaskPool.finish()

/01cd4f3eed7809080a5ef0f51387b588f1f55b2e Issue 8214 - blocking option for TaskPool.finish() Add backwards compatible blocking option to finish(). Remove disabled join() method. Add unit tests for finish(). https://github.com/D-Programming-Language/phobos/commit/a1e79c76335aff5cacd2c259265ae3ca4921500a Merge pull request #634

[Issue 8214] blocking option for TaskPool.finish()

http://d.puremagic.com/issues/show_bug.cgi?id=8214 John Belmonte j...@neggie.net changed: What|Removed |Added Status|ASSIGNED|RESOLVED

[Issue 8214] blocking option for TaskPool.finish()

http://d.puremagic.com/issues/show_bug.cgi?id=8214 Brad Roberts bra...@puremagic.com changed: What|Removed |Added Status|RESOLVED|REOPENED

[Issue 8214] blocking option for TaskPool.finish()

http://d.puremagic.com/issues/show_bug.cgi?id=8214 --- Comment #6 from John Belmonte j...@neggie.net 2012-06-17 11:55:52 PDT --- Addressing flaky test: https://github.com/D-Programming-Language/phobos/pull/636 -- Configure issuemail: http://d.puremagic.com/issues/userprefs.cgi?tab=email

[Issue 8214] blocking option for TaskPool.finish()

http://d.puremagic.com/issues/show_bug.cgi?id=8214 --- Comment #7 from John Belmonte j...@neggie.net 2012-06-17 12:00:57 PDT --- @braddr: would you kindly add me to while list for pull auto-tester? -- Configure issuemail: http://d.puremagic.com/issues/userprefs.cgi?tab=email --- You are

[Issue 8214] blocking option for TaskPool.finish()

http://d.puremagic.com/issues/show_bug.cgi?id=8214 --- Comment #8 from Brad Roberts bra...@puremagic.com 2012-06-17 12:03:29 PDT --- You were already. Both this pull and the previous one ran through the auto tester, and show(ed) failures. You're not done fixing things yet based on the

[Issue 8214] blocking option for TaskPool.finish()

http://d.puremagic.com/issues/show_bug.cgi?id=8214 --- Comment #9 from John Belmonte j...@neggie.net 2012-06-17 12:24:28 PDT --- Ack. I've disabled the failing assert as I don't have a good solution offhand. I don't feel too bad about disabling this particular assert since this was a test I

[Issue 8214] blocking option for TaskPool.finish()

http://d.puremagic.com/issues/show_bug.cgi?id=8214 --- Comment #10 from John Belmonte j...@neggie.net 2012-06-17 12:46:52 PDT --- dsimcha has pull request addressing test issue in a similar way https://github.com/D-Programming-Language/phobos/pull/637 -- Configure issuemail:

[Issue 8214] blocking option for TaskPool.finish()

http://d.puremagic.com/issues/show_bug.cgi?id=8214 John Belmonte j...@neggie.net changed: What|Removed |Added CC||dsim...@yahoo.com ---

[Issue 8214] blocking option for TaskPool.finish()

http://d.puremagic.com/issues/show_bug.cgi?id=8214 --- Comment #2 from John Belmonte j...@neggie.net 2012-06-09 20:39:07 PDT --- I do prefer this being an option to finish() rather than a separate join() function. We all understand what finish() does and it's not a stretch at all to accept