On Wed, 30 Mar 2011, commits@source.squeak.org wrote:
A new version of Tools was added to project The Inbox: http://source.squeak.org/inbox/Tools-fbs.336.mcz
==================== Summary ====================
Name: Tools-fbs.336 Author: fbs Time: 30 March 2011, 10:10:43.612 pm UUID: 263e2db2-c0e9-9e4c-a644-e95106ae600b Ancestors: Tools-fbs.335
As of this commit Browser has NO indices, and relies entirely on symbols to remember what it's browsing.
Great progress! I checked the code and found that there are references to two undeclared variables. The following snippet shows them:
SystemNavigation default browseMessageList: (Array streamContents: [ :stream | Undeclared associationsDo: [ :each | stream nextPutAll: (SystemNavigation default allCallsOn: each) ] ]) name: 'References to undeclared variables'
Levente
On 2011/03/31 13:01, Levente Uzonyi wrote:
On Wed, 30 Mar 2011, commits@source.squeak.org wrote:
A new version of Tools was added to project The Inbox: http://source.squeak.org/inbox/Tools-fbs.336.mcz
==================== Summary ====================
Name: Tools-fbs.336 Author: fbs Time: 30 March 2011, 10:10:43.612 pm UUID: 263e2db2-c0e9-9e4c-a644-e95106ae600b Ancestors: Tools-fbs.335
As of this commit Browser has NO indices, and relies entirely on symbols to remember what it's browsing.
Great progress! I checked the code and found that there are references to two undeclared variables. The following snippet shows them:
SystemNavigation default browseMessageList: (Array streamContents: [ :stream | Undeclared associationsDo: [ :each | stream nextPutAll: (SystemNavigation default allCallsOn: each) ] ]) name: 'References to undeclared variables'
Fixed. I mistakenly removed growable when I pushed messageListIndex from Browser down to MessageSet.
I plan to replace MessageSet's messageListIndex too, in the future. I kept it in for MessageSet and subclasses to delimit the branch's changes.
frank
squeak-dev@lists.squeakfoundation.org