diff options
| author | jmoenig <jens@moenig.org> | 2014-07-24 12:24:19 +0200 |
|---|---|---|
| committer | jmoenig <jens@moenig.org> | 2014-07-24 12:24:19 +0200 |
| commit | 54fec68d787f385f87683df5011bbcec8f0c3f77 (patch) | |
| tree | 8d1105140d29aed53806b1dc73fa5ecdd9464be1 | |
| parent | 75d9adbfe26d0303f1986f3140307d2f196c73ec (diff) | |
| download | snap-54fec68d787f385f87683df5011bbcec8f0c3f77.tar.gz snap-54fec68d787f385f87683df5011bbcec8f0c3f77.zip | |
Display “empty” Contexts as empty rings
(e.g. continuations). Brian was complaining about empty continuations
“sucking” up the watchers and value bubbles.
| -rwxr-xr-x | history.txt | 1 | ||||
| -rw-r--r-- | threads.js | 15 |
2 files changed, 14 insertions, 2 deletions
diff --git a/history.txt b/history.txt index 2fe1a18..9f21f79 100755 --- a/history.txt +++ b/history.txt @@ -2227,3 +2227,4 @@ ______ ------ * Objects: fixed “lost sprites bug” - ensure duplicated sprites keep wearing their current costume through save and re-load * GUI, Objects: improve unique sprite- and costume names +* Threads: Display “empty” Contexts (e.g. continuations) as empty rings @@ -83,7 +83,7 @@ ArgLabelMorph, localize, XML_Element, hex_sha512*/ // Global stuff //////////////////////////////////////////////////////// -modules.threads = '2014-July-22'; +modules.threads = '2014-July-24'; var ThreadManager; var Process; @@ -2854,7 +2854,18 @@ Context.prototype.image = function () { ring.embed(block, this.isContinuation ? [] : this.inputs); return ring.fullImage(); } - return newCanvas(); + + // otherwise show an empty ring + ring.color = SpriteMorph.prototype.blockColor.other; + ring.setSpec('%rc %ringparms'); + + // also show my inputs, unless I'm a continuation + if (!this.isContinuation) { + this.inputs.forEach(function (inp) { + ring.parts()[1].addInput(inp); + }); + } + return ring.fullImage(); }; // Context continuations: |
