David T. Lewis uploaded a new version of ToolBuilder-Kernel to project The Inbox:
http://source.squeak.org/inbox/ToolBuilder-Kernel-dtl.66.mcz ==================== Summary ==================== Name: ToolBuilder-Kernel-dtl.66 Author: dtl Time: 20 March 2015, 4:00:13.461 pm UUID: a396787d-a01a-40b3-8e2d-e1a91f6d3d3c Ancestors: ToolBuilder-Kernel-mt.65 Remove toolBuilder instance variable from UIManager. A UIManager knows what kind of ToolBuilder it should use, so it does not need to hold on to a single instance. Create ToolBuilder instances as required, and do not reuse a ToolBuilder instance after it has done its job. This prevents confusing accumulation of registered widgets in a single ToolBuilder instance =============== Diff against ToolBuilder-Kernel-mt.65 =============== Item was changed: Object subclass: #UIManager + instanceVariableNames: '' - instanceVariableNames: 'toolBuilder' classVariableNames: '' poolDictionaries: '' category: 'ToolBuilder-Kernel'! !UIManager commentStamp: 'dtl 5/2/2010 16:06' prior: 0! UIManager is a dispatcher for various user interface requests, such as menu and dialog interactions. An instance of UIManager is associated with each Project to implement the appropriate functions for Morphic, MVC or other user interfaces.! Item was changed: + ----- Method: UIManager>>toolBuilder (in category 'builder') ----- - ----- Method: UIManager>>toolBuilder (in category 'accessing') ----- toolBuilder + "Answer a ToolBuilder for this kind of UIManager" + ^ self subclassResponsibility! - ^toolBuilder! |
This addresses Eliot's earlier question about how to get rid of garbage that
accumulates in a ToolBuilder. It goes to the inbox for now because I am not entirely sure that it will not break the update stream. I'll move it to trunk this weekend if no one says otherwise. Tested in Morphic and MVC. Dave On Fri, Mar 20, 2015 at 02:08:50PM +0000, [hidden email] wrote: > David T. Lewis uploaded a new version of ToolBuilder-Kernel to project The Inbox: > http://source.squeak.org/inbox/ToolBuilder-Kernel-dtl.66.mcz > > ==================== Summary ==================== > > Name: ToolBuilder-Kernel-dtl.66 > Author: dtl > Time: 20 March 2015, 4:00:13.461 pm > UUID: a396787d-a01a-40b3-8e2d-e1a91f6d3d3c > Ancestors: ToolBuilder-Kernel-mt.65 > > Remove toolBuilder instance variable from UIManager. > > A UIManager knows what kind of ToolBuilder it should use, so it does not need to hold on to a single instance. Create ToolBuilder instances as required, and do not reuse a ToolBuilder instance after it has done its job. This prevents confusing accumulation of registered widgets in a single ToolBuilder instance > > =============== Diff against ToolBuilder-Kernel-mt.65 =============== > > Item was changed: > Object subclass: #UIManager > + instanceVariableNames: '' > - instanceVariableNames: 'toolBuilder' > classVariableNames: '' > poolDictionaries: '' > category: 'ToolBuilder-Kernel'! > > !UIManager commentStamp: 'dtl 5/2/2010 16:06' prior: 0! > UIManager is a dispatcher for various user interface requests, such as menu and dialog interactions. An instance of UIManager is associated with each Project to implement the appropriate functions for Morphic, MVC or other user interfaces.! > > Item was changed: > + ----- Method: UIManager>>toolBuilder (in category 'builder') ----- > - ----- Method: UIManager>>toolBuilder (in category 'accessing') ----- > toolBuilder > + "Answer a ToolBuilder for this kind of UIManager" > + ^ self subclassResponsibility! > - ^toolBuilder! > |
Please test custom builder classes. See my comment at: http://forum.world.st/The-Inbox-ToolBuilder-Morphic-dtl-100-mcz-td4813610.html
Best, Marcel |
Free forum by Nabble | Edit this page |