Browse Source

Removing dirty: call from factory method add:

Clients should call dirty themselves (there's only one and it does).
Herbert Vojčík 12 years ago
parent
commit
5b6af1345a
3 changed files with 2 additions and 5 deletions
  1. 0 1
      js/Trapped-Demo.deploy.js
  2. 2 3
      js/Trapped-Demo.js
  3. 0 1
      st/Trapped-Demo.st

+ 0 - 1
js/Trapped-Demo.deploy.js

@@ -39,7 +39,6 @@ selector: "add:",
 fn: function (aSubscription){
 var self=this;
 smalltalk.send(self["@queue"],"_add_",[aSubscription]);
-smalltalk.send(self,"_dirty_",[smalltalk.send(aSubscription,"_isFlagged",[])]);
 return self}
 }),
 smalltalk.TrappedDumbDispatcher);

+ 2 - 3
js/Trapped-Demo.js

@@ -45,11 +45,10 @@ category: 'accessing',
 fn: function (aSubscription){
 var self=this;
 smalltalk.send(self["@queue"],"_add_",[aSubscription]);
-smalltalk.send(self,"_dirty_",[smalltalk.send(aSubscription,"_isFlagged",[])]);
 return self},
 args: ["aSubscription"],
-source: "add: aSubscription\x0a\x09queue add: aSubscription.\x0a    self dirty: aSubscription isFlagged\x0a\x09",
-messageSends: ["add:", "dirty:", "isFlagged"],
+source: "add: aSubscription\x0a\x09queue add: aSubscription.\x0a",
+messageSends: ["add:"],
 referencedClasses: []
 }),
 smalltalk.TrappedDumbDispatcher);

+ 0 - 1
st/Trapped-Demo.st

@@ -26,7 +26,6 @@ TrappedDispatcher subclass: #TrappedDumbDispatcher
 
 add: aSubscription
 	queue add: aSubscription.
-    self dirty: aSubscription isFlagged
 ! !
 
 !TrappedDumbDispatcher methodsFor: 'bookkeeping'!