author | Claus Gittinger <cg@exept.de> |
Fri, 18 Oct 1996 15:20:50 +0200 | |
changeset 1782 | 76bcca2667a8 |
parent 1780 | 1b3a4ddc5b94 |
child 1785 | 4b18823528b8 |
permissions | -rw-r--r-- |
6 | 1 |
" |
2 |
COPYRIGHT (c) 1991 by Claus Gittinger |
|
186 | 3 |
All Rights Reserved |
6 | 4 |
|
5 |
This software is furnished under a license and may be used |
|
6 |
only in accordance with the terms of that license and with the |
|
7 |
inclusion of the above copyright notice. This software may not |
|
8 |
be provided or otherwise made available to, or used by, any |
|
9 |
other person. No title to or ownership of the software is |
|
10 |
hereby transferred. |
|
11 |
" |
|
12 |
||
603 | 13 |
Array subclass:#WeakArray |
1263 | 14 |
instanceVariableNames:'watcher dependents' |
15 |
classVariableNames:'RegistrationFailedSignal AlreadyInitialized' |
|
16 |
poolDictionaries:'' |
|
17 |
category:'Collections-Arrayed' |
|
6 | 18 |
! |
19 |
||
20 |
!WeakArray class methodsFor:'documentation'! |
|
21 |
||
88 | 22 |
copyright |
23 |
" |
|
24 |
COPYRIGHT (c) 1991 by Claus Gittinger |
|
186 | 25 |
All Rights Reserved |
88 | 26 |
|
27 |
This software is furnished under a license and may be used |
|
28 |
only in accordance with the terms of that license and with the |
|
29 |
inclusion of the above copyright notice. This software may not |
|
30 |
be provided or otherwise made available to, or used by, any |
|
31 |
other person. No title to or ownership of the software is |
|
32 |
hereby transferred. |
|
33 |
" |
|
34 |
! |
|
35 |
||
6 | 36 |
documentation |
37 |
" |
|
88 | 38 |
WeakArrays can be used to trace disposal of objects; in contrast to other |
1780
1b3a4ddc5b94
changed WeakArray to set emptied slots to zero
Claus Gittinger <cg@exept.de>
parents:
1688
diff
changeset
|
39 |
objects, references by WeakArrays will NOT keep an object from being |
1b3a4ddc5b94
changed WeakArray to set emptied slots to zero
Claus Gittinger <cg@exept.de>
parents:
1688
diff
changeset
|
40 |
garbage collected. |
1b3a4ddc5b94
changed WeakArray to set emptied slots to zero
Claus Gittinger <cg@exept.de>
parents:
1688
diff
changeset
|
41 |
Instead, whenever an object kept in a WeakArray dies, its entry is cleared, |
1b3a4ddc5b94
changed WeakArray to set emptied slots to zero
Claus Gittinger <cg@exept.de>
parents:
1688
diff
changeset
|
42 |
and the WeakArray is informed by the storage manager. |
290 | 43 |
|
1780
1b3a4ddc5b94
changed WeakArray to set emptied slots to zero
Claus Gittinger <cg@exept.de>
parents:
1688
diff
changeset
|
44 |
WeakArrays are used to track disposal of objects which keep external |
95 | 45 |
world resources. For example, FileStreams must close their underlying |
290 | 46 |
file when disposed (otherwise you could run out of OS filedescriptors). |
95 | 47 |
This can be done by keeping the FileStream objects in a weakArray, and |
1780
1b3a4ddc5b94
changed WeakArray to set emptied slots to zero
Claus Gittinger <cg@exept.de>
parents:
1688
diff
changeset
|
48 |
keep a parallel array of filedescriptors. Whenever a fileStream is |
1b3a4ddc5b94
changed WeakArray to set emptied slots to zero
Claus Gittinger <cg@exept.de>
parents:
1688
diff
changeset
|
49 |
freed, search both arrays for an index where the stream is empty, but the |
95 | 50 |
filedescriptor is non-nil. Then close that file, and nil the filedescriptor |
290 | 51 |
entry. Notice, that there is a class (Registry) which does exactly this in |
52 |
a more programmer friendly way. |
|
95 | 53 |
|
290 | 54 |
Another application is caching of data: keep it in a weakArray, so the |
95 | 55 |
data in that cache will not be unreclaimable due to being cached. |
56 |
(for example, the ResourcePack class uses a WeakArray to cache recently |
|
1780
1b3a4ddc5b94
changed WeakArray to set emptied slots to zero
Claus Gittinger <cg@exept.de>
parents:
1688
diff
changeset
|
57 |
used resource data for a while). |
88 | 58 |
|
186 | 59 |
The way in which weakArrays get informed by the runtime system is via |
60 |
an interrupt (DisposeInterrupt). The reason for not sending messages |
|
216 | 61 |
directly from the VM is to make it possible to run the finalization |
186 | 62 |
code at lower priority or from another class. |
63 |
Also, as a side effect, it is possible to delay finalization by blocking |
|
216 | 64 |
interrupts. |
186 | 65 |
|
290 | 66 |
This interrupt is cought in ObjectMemory and informs the weakArray, |
759
908363ce8a32
interest is written with one 'r' (shame on me)
Claus Gittinger <cg@exept.de>
parents:
681
diff
changeset
|
67 |
which in turn informs its watcher and possibly the dependents. |
290 | 68 |
Having two mechanisms here (i.e. watcher & dependent) is a historic leftover; |
69 |
I dont know, which of the two mechanisms will survive in the long run - |
|
70 |
I started with the watcher, but now switch to dependencies since they seem |
|
1780
1b3a4ddc5b94
changed WeakArray to set emptied slots to zero
Claus Gittinger <cg@exept.de>
parents:
1688
diff
changeset
|
71 |
to offer more flexibility. |
290 | 72 |
Be prepared, that the watcher mechanism may vanish in the future |
73 |
(i.e. use dependents for your applications). |
|
186 | 74 |
|
290 | 75 |
NOTICE: WeakArray handling adds some overhead to the VM (each weakarray |
216 | 76 |
is scanned after each GC). It is uncertain, if the current mechanism works well |
77 |
with (say) ten-thousands of weakArrays. |
|
290 | 78 |
|
1290 | 79 |
[instance variables:] |
290 | 80 |
|
1263 | 81 |
watcher if non-nil, gets informed via #informDispose |
82 |
that the weakArray has lost pointers. |
|
290 | 83 |
|
1263 | 84 |
dependents get informed via #change |
85 |
that the weakArray has lost pointers. |
|
290 | 86 |
|
87 |
||
1290 | 88 |
[class variables:] |
290 | 89 |
|
1263 | 90 |
RegistrationFailedSignal raised if a weakArray cannot be |
91 |
registered by the VM. This only happens, |
|
92 |
if the VM has to resize its shadow tables |
|
93 |
and is running out of malloc-memory. |
|
94 |
||
1290 | 95 |
[author:] |
96 |
Claus Gittinger |
|
97 |
||
1263 | 98 |
[See also:] |
99 |
Array WeakIdentitySet WeakIdentityDictionary |
|
6 | 100 |
" |
101 |
! ! |
|
102 |
||
159 | 103 |
!WeakArray class methodsFor:'initialization'! |
104 |
||
105 |
initialize |
|
106 |
"setup the private signal" |
|
107 |
||
108 |
RegistrationFailedSignal isNil ifTrue:[ |
|
302 | 109 |
RegistrationFailedSignal := ErrorSignal newSignalMayProceed:true. |
186 | 110 |
RegistrationFailedSignal nameClass:self message:#registrationFailedSignal. |
111 |
RegistrationFailedSignal notifierString:'weakArray registration failed'. |
|
159 | 112 |
] |
113 |
! ! |
|
114 |
||
6 | 115 |
!WeakArray class methodsFor:'instance creation'! |
116 |
||
117 |
new:size |
|
95 | 118 |
"return a new weakArray with size slots" |
6 | 119 |
|
95 | 120 |
"This is a kludge: I would like to set WEAK-flag in the classes |
216 | 121 |
initialize method, but (currently) the order in which the class-initialize |
290 | 122 |
methods are called is not defined ... |
216 | 123 |
... therefore it could happen, that a WeakArray is used by other |
124 |
classes initialize method BEFORE this method is evaluated. |
|
125 |
To avoid this, the WEAK bit in the class is set here, when the very first |
|
126 |
WeakArray is created." |
|
95 | 127 |
|
216 | 128 |
AlreadyInitialized isNil ifTrue:[ |
129 |
self flags:(Behavior flagWeakPointers). |
|
130 |
AlreadyInitialized := true |
|
131 |
]. |
|
95 | 132 |
|
159 | 133 |
^ (self basicNew:size) registerAsWeakArray |
134 |
! ! |
|
135 |
||
136 |
!WeakArray methodsFor:'GC registration'! |
|
137 |
||
138 |
registerAsWeakArray |
|
139 |
"register the receiver in the VM - |
|
140 |
i.e. tell the VM to nil disposed entries in the receiver |
|
141 |
and notify the disposeInterruptHandler whenever that happened." |
|
142 |
||
143 |
|ok| |
|
6 | 144 |
%{ |
145 |
OBJ __addShadowObject(); |
|
146 |
||
159 | 147 |
ok = __addShadowObject(self, 0); |
6 | 148 |
if (ok == false) { |
186 | 149 |
/* |
150 |
* this happens when too many shadow objects are |
|
151 |
* already there, collect garbage to get rid of |
|
152 |
* obsolete ones, and try again. |
|
153 |
* since a full collect is expensive, we try |
|
154 |
* a scavenge first, doing a full collect only if |
|
155 |
* that does not help. |
|
290 | 156 |
* |
157 |
* THIS MAY OR MAY NOT BE A GOOD IDEA: although it reduces |
|
158 |
* the number of shadow objects that have to be |
|
159 |
* processed at GC time, it may create a long delay here, |
|
160 |
* at shadow object creation time. |
|
161 |
* Dont know which is better ... |
|
186 | 162 |
*/ |
326 | 163 |
__nonTenuringScavenge(__context); |
186 | 164 |
ok = __addShadowObject(self, 0); |
308 | 165 |
#ifdef OLD |
186 | 166 |
if (ok == false) { |
290 | 167 |
/* |
168 |
* try more ... |
|
169 |
*/ |
|
326 | 170 |
__scavenge(__context); |
186 | 171 |
ok = __addShadowObject(self, 0); |
308 | 172 |
#endif |
186 | 173 |
if (ok == false) { |
290 | 174 |
/* |
175 |
* hard stuff - need full collect |
|
462 | 176 |
* if this is the very first GC, assume that we are in |
177 |
* the startup phase. Then do no GC. |
|
178 |
* Heuristics showed, that this GC does not find much ... |
|
290 | 179 |
*/ |
1127 | 180 |
if ((__garbageCollectCount() != 0) |
181 |
|| (__incrementalGCCount() != 0)) { |
|
462 | 182 |
__markAndSweepIfUseful(__context); |
183 |
ok = __addShadowObject(self, 0); |
|
184 |
} |
|
186 | 185 |
if (ok == false) { |
290 | 186 |
/* |
187 |
* mhmh - it seems that there are really many shadow |
|
188 |
* objects around - force creation |
|
189 |
*/ |
|
186 | 190 |
ok = __addShadowObject(self, 1); |
191 |
if (ok == false) { |
|
192 |
/* no chance - something must be wrong */ |
|
193 |
} |
|
194 |
} |
|
195 |
} |
|
308 | 196 |
#ifdef OLD |
186 | 197 |
} |
308 | 198 |
#endif |
6 | 199 |
} |
159 | 200 |
%}. |
201 |
ok ifFalse:[ |
|
290 | 202 |
" |
203 |
the VM was not able to register the new weakArray |
|
204 |
This can only happen, if the VM has to resize its tables, |
|
205 |
and a malloc request failed. Usually, this smells like big |
|
206 |
trouble being on the way (soon running out of memory in |
|
207 |
other places as well). |
|
208 |
Configure your OS for more swap space. |
|
209 |
" |
|
186 | 210 |
^ RegistrationFailedSignal raiseRequestWith:self |
159 | 211 |
] |
6 | 212 |
! ! |
213 |
||
603 | 214 |
!WeakArray methodsFor:'accessing'! |
215 |
||
216 |
at:index |
|
217 |
"return the indexed instance variable with index, anInteger. |
|
218 |
Reimplemented here for IGC readBarrier. You dont have to understand this." |
|
219 |
||
220 |
^ self basicAt:index |
|
221 |
! |
|
6 | 222 |
|
603 | 223 |
basicAt:index |
224 |
"return the indexed instance variable with index, anInteger. |
|
225 |
Reimplemented here for IGC readBarrier. You dont have to understand this." |
|
226 |
||
227 |
%{ /* NOCONTEXT */ |
|
228 |
||
229 |
REGISTER int indx; |
|
230 |
REGISTER unsigned int nIndex; |
|
231 |
OBJ el; |
|
6 | 232 |
|
603 | 233 |
if (__isSmallInteger(index)) { |
1136 | 234 |
indx = __intVal(index) - 1; |
603 | 235 |
if (indx >= 0) { |
236 |
nIndex = __BYTES2OBJS__(__qSize(self) - OHDR_SIZE); |
|
1136 | 237 |
indx += __intVal(__ClassInstPtr(__qClass(self))->c_ninstvars); |
603 | 238 |
if (indx < nIndex) { |
1136 | 239 |
el = __InstPtr(self)->i_instvars[indx]; |
1780
1b3a4ddc5b94
changed WeakArray to set emptied slots to zero
Claus Gittinger <cg@exept.de>
parents:
1688
diff
changeset
|
240 |
if (__isNonNilObject(el)) el = __WEAK_READ__(self, el); |
1b3a4ddc5b94
changed WeakArray to set emptied slots to zero
Claus Gittinger <cg@exept.de>
parents:
1688
diff
changeset
|
241 |
RETURN (el); |
603 | 242 |
} |
243 |
} |
|
244 |
} |
|
245 |
%}. |
|
246 |
^ super basicAt:index |
|
247 |
! |
|
248 |
||
249 |
dependents |
|
250 |
"return the dependents of the receiver" |
|
251 |
||
252 |
^ dependents |
|
253 |
! |
|
254 |
||
255 |
dependents:aCollection |
|
256 |
"set the dependents of the receiver" |
|
257 |
||
258 |
dependents := aCollection |
|
259 |
! |
|
260 |
||
261 |
watcher |
|
262 |
"return the watcher of the receiver. |
|
263 |
The watcher-stuff is a leftover from an old implementation |
|
264 |
and will vanish soon" |
|
265 |
||
266 |
^ watcher |
|
267 |
! |
|
268 |
||
269 |
watcher:anObject |
|
270 |
"set the watcher of the receiver. |
|
271 |
The watcher-stuff is a leftover from an old implementation |
|
272 |
and will vanish soon" |
|
273 |
||
274 |
watcher := anObject |
|
6 | 275 |
! ! |
276 |
||
159 | 277 |
!WeakArray methodsFor:'copying'! |
278 |
||
279 |
postCopy |
|
280 |
"copying alone does not really help - we have to tell |
|
281 |
the VM, that there is a new WeakArray around ... |
|
282 |
Q: who copies weakArrays ?" |
|
283 |
||
290 | 284 |
dependents := nil. |
159 | 285 |
self registerAsWeakArray. |
286 |
! ! |
|
287 |
||
586
cab695f942a6
weakArray readBarrier for IGC
Claus Gittinger <cg@exept.de>
parents:
530
diff
changeset
|
288 |
!WeakArray methodsFor:'enumeration'! |
cab695f942a6
weakArray readBarrier for IGC
Claus Gittinger <cg@exept.de>
parents:
530
diff
changeset
|
289 |
|
cab695f942a6
weakArray readBarrier for IGC
Claus Gittinger <cg@exept.de>
parents:
530
diff
changeset
|
290 |
do:aBlock |
cab695f942a6
weakArray readBarrier for IGC
Claus Gittinger <cg@exept.de>
parents:
530
diff
changeset
|
291 |
"evaluate the argument, aBlock for each element in the collection. |
cab695f942a6
weakArray readBarrier for IGC
Claus Gittinger <cg@exept.de>
parents:
530
diff
changeset
|
292 |
- reimplemented for IGC readBarrier. You dont have to understand this." |
cab695f942a6
weakArray readBarrier for IGC
Claus Gittinger <cg@exept.de>
parents:
530
diff
changeset
|
293 |
|
981
6da206fcc1c4
keep ref to weak-element somewhere (this is a try)
Claus Gittinger <cg@exept.de>
parents:
829
diff
changeset
|
294 |
|home element| |
586
cab695f942a6
weakArray readBarrier for IGC
Claus Gittinger <cg@exept.de>
parents:
530
diff
changeset
|
295 |
%{ |
cab695f942a6
weakArray readBarrier for IGC
Claus Gittinger <cg@exept.de>
parents:
530
diff
changeset
|
296 |
REGISTER OBJFUNC codeVal; |
cab695f942a6
weakArray readBarrier for IGC
Claus Gittinger <cg@exept.de>
parents:
530
diff
changeset
|
297 |
REGISTER int index; |
cab695f942a6
weakArray readBarrier for IGC
Claus Gittinger <cg@exept.de>
parents:
530
diff
changeset
|
298 |
unsigned int nIndex; |
cab695f942a6
weakArray readBarrier for IGC
Claus Gittinger <cg@exept.de>
parents:
530
diff
changeset
|
299 |
static struct inlineCache val = _ILC1; |
cab695f942a6
weakArray readBarrier for IGC
Claus Gittinger <cg@exept.de>
parents:
530
diff
changeset
|
300 |
REGISTER OBJ rHome; |
cab695f942a6
weakArray readBarrier for IGC
Claus Gittinger <cg@exept.de>
parents:
530
diff
changeset
|
301 |
|
1136 | 302 |
index = __intVal(__ClassInstPtr(__qClass(self))->c_ninstvars); |
586
cab695f942a6
weakArray readBarrier for IGC
Claus Gittinger <cg@exept.de>
parents:
530
diff
changeset
|
303 |
nIndex = __BYTES2OBJS__(__qSize(self) - OHDR_SIZE); |
cab695f942a6
weakArray readBarrier for IGC
Claus Gittinger <cg@exept.de>
parents:
530
diff
changeset
|
304 |
if (__isBlockLike(aBlock) |
1136 | 305 |
&& ((codeVal = __BlockInstPtr(aBlock)->b_code) != (OBJFUNC)nil) |
306 |
&& (__BlockInstPtr(aBlock)->b_nargs == __MKSMALLINT(1))) { |
|
586
cab695f942a6
weakArray readBarrier for IGC
Claus Gittinger <cg@exept.de>
parents:
530
diff
changeset
|
307 |
#ifdef NEW_BLOCK_CALL |
cab695f942a6
weakArray readBarrier for IGC
Claus Gittinger <cg@exept.de>
parents:
530
diff
changeset
|
308 |
for (; index < nIndex; index++) { |
1688
8a42db1eea60
removed all COMMA_CON / CON_COMMA uses
Claus Gittinger <cg@exept.de>
parents:
1290
diff
changeset
|
309 |
if (InterruptPending != nil) __interruptL(@line); |
1136 | 310 |
element = __InstPtr(self)->i_instvars[index]; |
1780
1b3a4ddc5b94
changed WeakArray to set emptied slots to zero
Claus Gittinger <cg@exept.de>
parents:
1688
diff
changeset
|
311 |
if (__isNonNilObject(element)) element = __WEAK_READ__(self, element); |
1688
8a42db1eea60
removed all COMMA_CON / CON_COMMA uses
Claus Gittinger <cg@exept.de>
parents:
1290
diff
changeset
|
312 |
(*codeVal)(aBlock, element); |
586
cab695f942a6
weakArray readBarrier for IGC
Claus Gittinger <cg@exept.de>
parents:
530
diff
changeset
|
313 |
} |
cab695f942a6
weakArray readBarrier for IGC
Claus Gittinger <cg@exept.de>
parents:
530
diff
changeset
|
314 |
#else |
1136 | 315 |
home = __BlockInstPtr(aBlock)->b_home; |
586
cab695f942a6
weakArray readBarrier for IGC
Claus Gittinger <cg@exept.de>
parents:
530
diff
changeset
|
316 |
rHome = home; |
cab695f942a6
weakArray readBarrier for IGC
Claus Gittinger <cg@exept.de>
parents:
530
diff
changeset
|
317 |
if ((rHome == nil) || (__qSpace(rHome) >= STACKSPACE)) { |
cab695f942a6
weakArray readBarrier for IGC
Claus Gittinger <cg@exept.de>
parents:
530
diff
changeset
|
318 |
/* |
cab695f942a6
weakArray readBarrier for IGC
Claus Gittinger <cg@exept.de>
parents:
530
diff
changeset
|
319 |
* home will not move - keep in a fast register |
cab695f942a6
weakArray readBarrier for IGC
Claus Gittinger <cg@exept.de>
parents:
530
diff
changeset
|
320 |
*/ |
cab695f942a6
weakArray readBarrier for IGC
Claus Gittinger <cg@exept.de>
parents:
530
diff
changeset
|
321 |
for (; index < nIndex; index++) { |
1688
8a42db1eea60
removed all COMMA_CON / CON_COMMA uses
Claus Gittinger <cg@exept.de>
parents:
1290
diff
changeset
|
322 |
if (InterruptPending != nil) __interruptL(@line); |
586
cab695f942a6
weakArray readBarrier for IGC
Claus Gittinger <cg@exept.de>
parents:
530
diff
changeset
|
323 |
|
1136 | 324 |
element = __InstPtr(self)->i_instvars[index]; |
1780
1b3a4ddc5b94
changed WeakArray to set emptied slots to zero
Claus Gittinger <cg@exept.de>
parents:
1688
diff
changeset
|
325 |
if (__isNonNilObject(element)) element = __WEAK_READ__(self, element); |
1688
8a42db1eea60
removed all COMMA_CON / CON_COMMA uses
Claus Gittinger <cg@exept.de>
parents:
1290
diff
changeset
|
326 |
(*codeVal)(rHome, element); |
586
cab695f942a6
weakArray readBarrier for IGC
Claus Gittinger <cg@exept.de>
parents:
530
diff
changeset
|
327 |
} |
cab695f942a6
weakArray readBarrier for IGC
Claus Gittinger <cg@exept.de>
parents:
530
diff
changeset
|
328 |
} else { |
cab695f942a6
weakArray readBarrier for IGC
Claus Gittinger <cg@exept.de>
parents:
530
diff
changeset
|
329 |
for (; index < nIndex; index++) { |
1688
8a42db1eea60
removed all COMMA_CON / CON_COMMA uses
Claus Gittinger <cg@exept.de>
parents:
1290
diff
changeset
|
330 |
if (InterruptPending != nil) __interruptL(@line); |
586
cab695f942a6
weakArray readBarrier for IGC
Claus Gittinger <cg@exept.de>
parents:
530
diff
changeset
|
331 |
|
1136 | 332 |
element = __InstPtr(self)->i_instvars[index]; |
1780
1b3a4ddc5b94
changed WeakArray to set emptied slots to zero
Claus Gittinger <cg@exept.de>
parents:
1688
diff
changeset
|
333 |
if (__isNonNilObject(element)) element = __WEAK_READ__(self, element); |
1688
8a42db1eea60
removed all COMMA_CON / CON_COMMA uses
Claus Gittinger <cg@exept.de>
parents:
1290
diff
changeset
|
334 |
(*codeVal)(home, element); |
586
cab695f942a6
weakArray readBarrier for IGC
Claus Gittinger <cg@exept.de>
parents:
530
diff
changeset
|
335 |
} |
cab695f942a6
weakArray readBarrier for IGC
Claus Gittinger <cg@exept.de>
parents:
530
diff
changeset
|
336 |
} |
cab695f942a6
weakArray readBarrier for IGC
Claus Gittinger <cg@exept.de>
parents:
530
diff
changeset
|
337 |
#endif |
cab695f942a6
weakArray readBarrier for IGC
Claus Gittinger <cg@exept.de>
parents:
530
diff
changeset
|
338 |
} else { |
cab695f942a6
weakArray readBarrier for IGC
Claus Gittinger <cg@exept.de>
parents:
530
diff
changeset
|
339 |
for (; index < nIndex; index++) { |
1688
8a42db1eea60
removed all COMMA_CON / CON_COMMA uses
Claus Gittinger <cg@exept.de>
parents:
1290
diff
changeset
|
340 |
if (InterruptPending != nil) __interruptL(@line); |
586
cab695f942a6
weakArray readBarrier for IGC
Claus Gittinger <cg@exept.de>
parents:
530
diff
changeset
|
341 |
|
1136 | 342 |
element = __InstPtr(self)->i_instvars[index]; |
1780
1b3a4ddc5b94
changed WeakArray to set emptied slots to zero
Claus Gittinger <cg@exept.de>
parents:
1688
diff
changeset
|
343 |
if (__isNonNilObject(element)) element = __WEAK_READ__(self, element); |
586
cab695f942a6
weakArray readBarrier for IGC
Claus Gittinger <cg@exept.de>
parents:
530
diff
changeset
|
344 |
(*val.ilc_func)(aBlock, |
cab695f942a6
weakArray readBarrier for IGC
Claus Gittinger <cg@exept.de>
parents:
530
diff
changeset
|
345 |
@symbol(value:), |
1688
8a42db1eea60
removed all COMMA_CON / CON_COMMA uses
Claus Gittinger <cg@exept.de>
parents:
1290
diff
changeset
|
346 |
nil, &val, |
981
6da206fcc1c4
keep ref to weak-element somewhere (this is a try)
Claus Gittinger <cg@exept.de>
parents:
829
diff
changeset
|
347 |
element); |
586
cab695f942a6
weakArray readBarrier for IGC
Claus Gittinger <cg@exept.de>
parents:
530
diff
changeset
|
348 |
} |
cab695f942a6
weakArray readBarrier for IGC
Claus Gittinger <cg@exept.de>
parents:
530
diff
changeset
|
349 |
} |
981
6da206fcc1c4
keep ref to weak-element somewhere (this is a try)
Claus Gittinger <cg@exept.de>
parents:
829
diff
changeset
|
350 |
%}. |
586
cab695f942a6
weakArray readBarrier for IGC
Claus Gittinger <cg@exept.de>
parents:
530
diff
changeset
|
351 |
^ self |
cab695f942a6
weakArray readBarrier for IGC
Claus Gittinger <cg@exept.de>
parents:
530
diff
changeset
|
352 |
! |
cab695f942a6
weakArray readBarrier for IGC
Claus Gittinger <cg@exept.de>
parents:
530
diff
changeset
|
353 |
|
1780
1b3a4ddc5b94
changed WeakArray to set emptied slots to zero
Claus Gittinger <cg@exept.de>
parents:
1688
diff
changeset
|
354 |
forAllDeadIndicesDo:aBlock |
1b3a4ddc5b94
changed WeakArray to set emptied slots to zero
Claus Gittinger <cg@exept.de>
parents:
1688
diff
changeset
|
355 |
"evaluate the argument, aBlock for all indices where elements have been |
1b3a4ddc5b94
changed WeakArray to set emptied slots to zero
Claus Gittinger <cg@exept.de>
parents:
1688
diff
changeset
|
356 |
replaced by zero (due to a collected object)." |
1b3a4ddc5b94
changed WeakArray to set emptied slots to zero
Claus Gittinger <cg@exept.de>
parents:
1688
diff
changeset
|
357 |
|
1b3a4ddc5b94
changed WeakArray to set emptied slots to zero
Claus Gittinger <cg@exept.de>
parents:
1688
diff
changeset
|
358 |
self keysAndValuesDo:[:index :element | |
1b3a4ddc5b94
changed WeakArray to set emptied slots to zero
Claus Gittinger <cg@exept.de>
parents:
1688
diff
changeset
|
359 |
element == 0 ifTrue:[ |
1b3a4ddc5b94
changed WeakArray to set emptied slots to zero
Claus Gittinger <cg@exept.de>
parents:
1688
diff
changeset
|
360 |
aBlock value:index |
1b3a4ddc5b94
changed WeakArray to set emptied slots to zero
Claus Gittinger <cg@exept.de>
parents:
1688
diff
changeset
|
361 |
] |
1b3a4ddc5b94
changed WeakArray to set emptied slots to zero
Claus Gittinger <cg@exept.de>
parents:
1688
diff
changeset
|
362 |
] |
1b3a4ddc5b94
changed WeakArray to set emptied slots to zero
Claus Gittinger <cg@exept.de>
parents:
1688
diff
changeset
|
363 |
! |
1b3a4ddc5b94
changed WeakArray to set emptied slots to zero
Claus Gittinger <cg@exept.de>
parents:
1688
diff
changeset
|
364 |
|
1b3a4ddc5b94
changed WeakArray to set emptied slots to zero
Claus Gittinger <cg@exept.de>
parents:
1688
diff
changeset
|
365 |
forAllDeadIndicesDo:aBlock replacingCorpsesWith:newValue |
1b3a4ddc5b94
changed WeakArray to set emptied slots to zero
Claus Gittinger <cg@exept.de>
parents:
1688
diff
changeset
|
366 |
"evaluate the argument, aBlock for all indices where elements have been |
1b3a4ddc5b94
changed WeakArray to set emptied slots to zero
Claus Gittinger <cg@exept.de>
parents:
1688
diff
changeset
|
367 |
replaced by zero (due to a collected object), and replace the element |
1b3a4ddc5b94
changed WeakArray to set emptied slots to zero
Claus Gittinger <cg@exept.de>
parents:
1688
diff
changeset
|
368 |
with newValue. |
1b3a4ddc5b94
changed WeakArray to set emptied slots to zero
Claus Gittinger <cg@exept.de>
parents:
1688
diff
changeset
|
369 |
In the current implementation, the block sees the newValue (i.e. it is |
1b3a4ddc5b94
changed WeakArray to set emptied slots to zero
Claus Gittinger <cg@exept.de>
parents:
1688
diff
changeset
|
370 |
changed before the block is called); this behavior is not guaranteed |
1b3a4ddc5b94
changed WeakArray to set emptied slots to zero
Claus Gittinger <cg@exept.de>
parents:
1688
diff
changeset
|
371 |
with future versions." |
1b3a4ddc5b94
changed WeakArray to set emptied slots to zero
Claus Gittinger <cg@exept.de>
parents:
1688
diff
changeset
|
372 |
|
1b3a4ddc5b94
changed WeakArray to set emptied slots to zero
Claus Gittinger <cg@exept.de>
parents:
1688
diff
changeset
|
373 |
self keysAndValuesDo:[:index :element | |
1b3a4ddc5b94
changed WeakArray to set emptied slots to zero
Claus Gittinger <cg@exept.de>
parents:
1688
diff
changeset
|
374 |
element == 0 ifTrue:[ |
1b3a4ddc5b94
changed WeakArray to set emptied slots to zero
Claus Gittinger <cg@exept.de>
parents:
1688
diff
changeset
|
375 |
self at:index put:newValue. |
1b3a4ddc5b94
changed WeakArray to set emptied slots to zero
Claus Gittinger <cg@exept.de>
parents:
1688
diff
changeset
|
376 |
aBlock value:index. |
1b3a4ddc5b94
changed WeakArray to set emptied slots to zero
Claus Gittinger <cg@exept.de>
parents:
1688
diff
changeset
|
377 |
] |
1b3a4ddc5b94
changed WeakArray to set emptied slots to zero
Claus Gittinger <cg@exept.de>
parents:
1688
diff
changeset
|
378 |
] |
1b3a4ddc5b94
changed WeakArray to set emptied slots to zero
Claus Gittinger <cg@exept.de>
parents:
1688
diff
changeset
|
379 |
! |
1b3a4ddc5b94
changed WeakArray to set emptied slots to zero
Claus Gittinger <cg@exept.de>
parents:
1688
diff
changeset
|
380 |
|
1b3a4ddc5b94
changed WeakArray to set emptied slots to zero
Claus Gittinger <cg@exept.de>
parents:
1688
diff
changeset
|
381 |
nilAllCorpsesAndDo:aBlock |
1b3a4ddc5b94
changed WeakArray to set emptied slots to zero
Claus Gittinger <cg@exept.de>
parents:
1688
diff
changeset
|
382 |
"evaluate the argument, aBlock for all indices where elements have been |
1b3a4ddc5b94
changed WeakArray to set emptied slots to zero
Claus Gittinger <cg@exept.de>
parents:
1688
diff
changeset
|
383 |
cleared (due to a collected object), nil the entry." |
1b3a4ddc5b94
changed WeakArray to set emptied slots to zero
Claus Gittinger <cg@exept.de>
parents:
1688
diff
changeset
|
384 |
|
1b3a4ddc5b94
changed WeakArray to set emptied slots to zero
Claus Gittinger <cg@exept.de>
parents:
1688
diff
changeset
|
385 |
self forAllDeadIndicesDo:aBlock replacingCorpsesWith:nil |
1b3a4ddc5b94
changed WeakArray to set emptied slots to zero
Claus Gittinger <cg@exept.de>
parents:
1688
diff
changeset
|
386 |
! |
1b3a4ddc5b94
changed WeakArray to set emptied slots to zero
Claus Gittinger <cg@exept.de>
parents:
1688
diff
changeset
|
387 |
|
586
cab695f942a6
weakArray readBarrier for IGC
Claus Gittinger <cg@exept.de>
parents:
530
diff
changeset
|
388 |
from:start to:stop do:aBlock |
cab695f942a6
weakArray readBarrier for IGC
Claus Gittinger <cg@exept.de>
parents:
530
diff
changeset
|
389 |
"evaluate the argument, aBlock for the elements starting at index start |
cab695f942a6
weakArray readBarrier for IGC
Claus Gittinger <cg@exept.de>
parents:
530
diff
changeset
|
390 |
up to (and including) stop in the collection. |
cab695f942a6
weakArray readBarrier for IGC
Claus Gittinger <cg@exept.de>
parents:
530
diff
changeset
|
391 |
- reimplemented for IGC readBarrier. You dont have to understand this." |
cab695f942a6
weakArray readBarrier for IGC
Claus Gittinger <cg@exept.de>
parents:
530
diff
changeset
|
392 |
|
981
6da206fcc1c4
keep ref to weak-element somewhere (this is a try)
Claus Gittinger <cg@exept.de>
parents:
829
diff
changeset
|
393 |
|home element| |
586
cab695f942a6
weakArray readBarrier for IGC
Claus Gittinger <cg@exept.de>
parents:
530
diff
changeset
|
394 |
%{ |
cab695f942a6
weakArray readBarrier for IGC
Claus Gittinger <cg@exept.de>
parents:
530
diff
changeset
|
395 |
REGISTER OBJFUNC codeVal; |
cab695f942a6
weakArray readBarrier for IGC
Claus Gittinger <cg@exept.de>
parents:
530
diff
changeset
|
396 |
REGISTER int index; |
cab695f942a6
weakArray readBarrier for IGC
Claus Gittinger <cg@exept.de>
parents:
530
diff
changeset
|
397 |
REGISTER OBJ rHome; |
cab695f942a6
weakArray readBarrier for IGC
Claus Gittinger <cg@exept.de>
parents:
530
diff
changeset
|
398 |
int nIndex, nInsts; |
cab695f942a6
weakArray readBarrier for IGC
Claus Gittinger <cg@exept.de>
parents:
530
diff
changeset
|
399 |
static struct inlineCache val = _ILC1; |
cab695f942a6
weakArray readBarrier for IGC
Claus Gittinger <cg@exept.de>
parents:
530
diff
changeset
|
400 |
int indexLow, indexHigh; |
cab695f942a6
weakArray readBarrier for IGC
Claus Gittinger <cg@exept.de>
parents:
530
diff
changeset
|
401 |
|
cab695f942a6
weakArray readBarrier for IGC
Claus Gittinger <cg@exept.de>
parents:
530
diff
changeset
|
402 |
if (__bothSmallInteger(start, stop)) { |
1136 | 403 |
indexLow = __intVal(start); |
586
cab695f942a6
weakArray readBarrier for IGC
Claus Gittinger <cg@exept.de>
parents:
530
diff
changeset
|
404 |
if (indexLow > 0) { |
1136 | 405 |
indexHigh = __intVal(stop); |
406 |
nInsts = __intVal(__ClassInstPtr(__qClass(self))->c_ninstvars); |
|
586
cab695f942a6
weakArray readBarrier for IGC
Claus Gittinger <cg@exept.de>
parents:
530
diff
changeset
|
407 |
indexLow += nInsts; |
cab695f942a6
weakArray readBarrier for IGC
Claus Gittinger <cg@exept.de>
parents:
530
diff
changeset
|
408 |
indexHigh += nInsts; |
cab695f942a6
weakArray readBarrier for IGC
Claus Gittinger <cg@exept.de>
parents:
530
diff
changeset
|
409 |
nIndex = __BYTES2OBJS__(__qSize(self) - OHDR_SIZE); |
cab695f942a6
weakArray readBarrier for IGC
Claus Gittinger <cg@exept.de>
parents:
530
diff
changeset
|
410 |
if (indexHigh <= nIndex) { |
cab695f942a6
weakArray readBarrier for IGC
Claus Gittinger <cg@exept.de>
parents:
530
diff
changeset
|
411 |
indexLow--; |
cab695f942a6
weakArray readBarrier for IGC
Claus Gittinger <cg@exept.de>
parents:
530
diff
changeset
|
412 |
indexHigh--; |
cab695f942a6
weakArray readBarrier for IGC
Claus Gittinger <cg@exept.de>
parents:
530
diff
changeset
|
413 |
if (__isBlockLike(aBlock) |
1136 | 414 |
&& ((codeVal = __BlockInstPtr(aBlock)->b_code) != (OBJFUNC)nil) |
415 |
&& (__BlockInstPtr(aBlock)->b_nargs == __MKSMALLINT(1))) { |
|
586
cab695f942a6
weakArray readBarrier for IGC
Claus Gittinger <cg@exept.de>
parents:
530
diff
changeset
|
416 |
#ifdef NEW_BLOCK_CALL |
cab695f942a6
weakArray readBarrier for IGC
Claus Gittinger <cg@exept.de>
parents:
530
diff
changeset
|
417 |
for (index=indexLow; index <= indexHigh; index++) { |
1688
8a42db1eea60
removed all COMMA_CON / CON_COMMA uses
Claus Gittinger <cg@exept.de>
parents:
1290
diff
changeset
|
418 |
if (InterruptPending != nil) __interruptL(@line); |
1136 | 419 |
element = __InstPtr(self)->i_instvars[index]; |
1780
1b3a4ddc5b94
changed WeakArray to set emptied slots to zero
Claus Gittinger <cg@exept.de>
parents:
1688
diff
changeset
|
420 |
if (__isNonNilObject(element)) element = __WEAK_READ__(self, element); |
1688
8a42db1eea60
removed all COMMA_CON / CON_COMMA uses
Claus Gittinger <cg@exept.de>
parents:
1290
diff
changeset
|
421 |
(*codeVal)(aBlock, element); |
586
cab695f942a6
weakArray readBarrier for IGC
Claus Gittinger <cg@exept.de>
parents:
530
diff
changeset
|
422 |
} |
cab695f942a6
weakArray readBarrier for IGC
Claus Gittinger <cg@exept.de>
parents:
530
diff
changeset
|
423 |
#else |
1136 | 424 |
home = __BlockInstPtr(aBlock)->b_home; |
586
cab695f942a6
weakArray readBarrier for IGC
Claus Gittinger <cg@exept.de>
parents:
530
diff
changeset
|
425 |
rHome = home; |
cab695f942a6
weakArray readBarrier for IGC
Claus Gittinger <cg@exept.de>
parents:
530
diff
changeset
|
426 |
if ((rHome == nil) || (__qSpace(rHome) >= STACKSPACE)) { |
cab695f942a6
weakArray readBarrier for IGC
Claus Gittinger <cg@exept.de>
parents:
530
diff
changeset
|
427 |
index = indexLow; |
cab695f942a6
weakArray readBarrier for IGC
Claus Gittinger <cg@exept.de>
parents:
530
diff
changeset
|
428 |
for (; index <= indexHigh; index++) { |
1688
8a42db1eea60
removed all COMMA_CON / CON_COMMA uses
Claus Gittinger <cg@exept.de>
parents:
1290
diff
changeset
|
429 |
if (InterruptPending != nil) __interruptL(@line); |
1136 | 430 |
element = __InstPtr(self)->i_instvars[index]; |
1780
1b3a4ddc5b94
changed WeakArray to set emptied slots to zero
Claus Gittinger <cg@exept.de>
parents:
1688
diff
changeset
|
431 |
if (__isNonNilObject(element)) element = __WEAK_READ__(self, element); |
1688
8a42db1eea60
removed all COMMA_CON / CON_COMMA uses
Claus Gittinger <cg@exept.de>
parents:
1290
diff
changeset
|
432 |
(*codeVal)(rHome, element); |
586
cab695f942a6
weakArray readBarrier for IGC
Claus Gittinger <cg@exept.de>
parents:
530
diff
changeset
|
433 |
} |
cab695f942a6
weakArray readBarrier for IGC
Claus Gittinger <cg@exept.de>
parents:
530
diff
changeset
|
434 |
} else { |
cab695f942a6
weakArray readBarrier for IGC
Claus Gittinger <cg@exept.de>
parents:
530
diff
changeset
|
435 |
for (index=indexLow; index <= indexHigh; index++) { |
1688
8a42db1eea60
removed all COMMA_CON / CON_COMMA uses
Claus Gittinger <cg@exept.de>
parents:
1290
diff
changeset
|
436 |
if (InterruptPending != nil) __interruptL(@line); |
1136 | 437 |
element = __InstPtr(self)->i_instvars[index]; |
1780
1b3a4ddc5b94
changed WeakArray to set emptied slots to zero
Claus Gittinger <cg@exept.de>
parents:
1688
diff
changeset
|
438 |
if (__isNonNilObject(element)) element = __WEAK_READ__(self, element); |
1688
8a42db1eea60
removed all COMMA_CON / CON_COMMA uses
Claus Gittinger <cg@exept.de>
parents:
1290
diff
changeset
|
439 |
(*codeVal)(home, element); |
586
cab695f942a6
weakArray readBarrier for IGC
Claus Gittinger <cg@exept.de>
parents:
530
diff
changeset
|
440 |
} |
cab695f942a6
weakArray readBarrier for IGC
Claus Gittinger <cg@exept.de>
parents:
530
diff
changeset
|
441 |
} |
cab695f942a6
weakArray readBarrier for IGC
Claus Gittinger <cg@exept.de>
parents:
530
diff
changeset
|
442 |
#endif |
cab695f942a6
weakArray readBarrier for IGC
Claus Gittinger <cg@exept.de>
parents:
530
diff
changeset
|
443 |
} else { |
cab695f942a6
weakArray readBarrier for IGC
Claus Gittinger <cg@exept.de>
parents:
530
diff
changeset
|
444 |
for (index=indexLow; index <= indexHigh; index++) { |
1688
8a42db1eea60
removed all COMMA_CON / CON_COMMA uses
Claus Gittinger <cg@exept.de>
parents:
1290
diff
changeset
|
445 |
if (InterruptPending != nil) __interruptL(@line); |
1136 | 446 |
element = __InstPtr(self)->i_instvars[index]; |
1780
1b3a4ddc5b94
changed WeakArray to set emptied slots to zero
Claus Gittinger <cg@exept.de>
parents:
1688
diff
changeset
|
447 |
if (__isNonNilObject(element)) element = __WEAK_READ__(self, element); |
586
cab695f942a6
weakArray readBarrier for IGC
Claus Gittinger <cg@exept.de>
parents:
530
diff
changeset
|
448 |
(*val.ilc_func) (aBlock, |
cab695f942a6
weakArray readBarrier for IGC
Claus Gittinger <cg@exept.de>
parents:
530
diff
changeset
|
449 |
@symbol(value:), |
1688
8a42db1eea60
removed all COMMA_CON / CON_COMMA uses
Claus Gittinger <cg@exept.de>
parents:
1290
diff
changeset
|
450 |
nil, &val, |
981
6da206fcc1c4
keep ref to weak-element somewhere (this is a try)
Claus Gittinger <cg@exept.de>
parents:
829
diff
changeset
|
451 |
element); |
586
cab695f942a6
weakArray readBarrier for IGC
Claus Gittinger <cg@exept.de>
parents:
530
diff
changeset
|
452 |
} |
cab695f942a6
weakArray readBarrier for IGC
Claus Gittinger <cg@exept.de>
parents:
530
diff
changeset
|
453 |
} |
cab695f942a6
weakArray readBarrier for IGC
Claus Gittinger <cg@exept.de>
parents:
530
diff
changeset
|
454 |
} |
cab695f942a6
weakArray readBarrier for IGC
Claus Gittinger <cg@exept.de>
parents:
530
diff
changeset
|
455 |
RETURN ( self ); |
cab695f942a6
weakArray readBarrier for IGC
Claus Gittinger <cg@exept.de>
parents:
530
diff
changeset
|
456 |
} |
cab695f942a6
weakArray readBarrier for IGC
Claus Gittinger <cg@exept.de>
parents:
530
diff
changeset
|
457 |
} |
1780
1b3a4ddc5b94
changed WeakArray to set emptied slots to zero
Claus Gittinger <cg@exept.de>
parents:
1688
diff
changeset
|
458 |
%}. |
586
cab695f942a6
weakArray readBarrier for IGC
Claus Gittinger <cg@exept.de>
parents:
530
diff
changeset
|
459 |
^ super from:start to:stop do:aBlock |
cab695f942a6
weakArray readBarrier for IGC
Claus Gittinger <cg@exept.de>
parents:
530
diff
changeset
|
460 |
! |
cab695f942a6
weakArray readBarrier for IGC
Claus Gittinger <cg@exept.de>
parents:
530
diff
changeset
|
461 |
|
603 | 462 |
keysAndValuesDo:aBlock |
463 |
"evaluate the argument, aBlock for each element in the collection. |
|
464 |
Pass both index and element to the block. |
|
465 |
- reimplemented for speed" |
|
466 |
||
981
6da206fcc1c4
keep ref to weak-element somewhere (this is a try)
Claus Gittinger <cg@exept.de>
parents:
829
diff
changeset
|
467 |
|home element| |
603 | 468 |
%{ |
469 |
REGISTER OBJFUNC codeVal; |
|
470 |
REGISTER int index; |
|
1782
76bcca2667a8
oops - keysAndValuesDo: passed the wrong index
Claus Gittinger <cg@exept.de>
parents:
1780
diff
changeset
|
471 |
unsigned int nIndex, index0; |
603 | 472 |
static struct inlineCache val2 = _ILC2; |
473 |
REGISTER OBJ rHome; |
|
474 |
||
1136 | 475 |
index = __intVal(__ClassInstPtr(__qClass(self))->c_ninstvars); |
1782
76bcca2667a8
oops - keysAndValuesDo: passed the wrong index
Claus Gittinger <cg@exept.de>
parents:
1780
diff
changeset
|
476 |
index0 = index; |
603 | 477 |
nIndex = __BYTES2OBJS__(__qSize(self) - OHDR_SIZE); |
1782
76bcca2667a8
oops - keysAndValuesDo: passed the wrong index
Claus Gittinger <cg@exept.de>
parents:
1780
diff
changeset
|
478 |
|
603 | 479 |
if (__isBlockLike(aBlock) |
1136 | 480 |
&& ((codeVal = __BlockInstPtr(aBlock)->b_code) != (OBJFUNC)nil) |
481 |
&& (__BlockInstPtr(aBlock)->b_nargs == __MKSMALLINT(2))) { |
|
603 | 482 |
#ifdef NEW_BLOCK_CALL |
483 |
for (; index < nIndex; index++) { |
|
1688
8a42db1eea60
removed all COMMA_CON / CON_COMMA uses
Claus Gittinger <cg@exept.de>
parents:
1290
diff
changeset
|
484 |
if (InterruptPending != nil) __interruptL(@line); |
603 | 485 |
|
1136 | 486 |
element = __InstPtr(self)->i_instvars[index]; |
1780
1b3a4ddc5b94
changed WeakArray to set emptied slots to zero
Claus Gittinger <cg@exept.de>
parents:
1688
diff
changeset
|
487 |
if (__isNonNilObject(element)) element = __WEAK_READ__(self, element); |
1782
76bcca2667a8
oops - keysAndValuesDo: passed the wrong index
Claus Gittinger <cg@exept.de>
parents:
1780
diff
changeset
|
488 |
(*codeVal)(aBlock, __MKSMALLINT(index+1-index0), element); |
603 | 489 |
} |
490 |
#else |
|
1136 | 491 |
home = __BlockInstPtr(aBlock)->b_home; |
603 | 492 |
rHome = home; |
493 |
if ((rHome == nil) || (__qSpace(rHome) >= STACKSPACE)) { |
|
494 |
/* |
|
495 |
* home will not move - keep in a fast register |
|
496 |
*/ |
|
497 |
while (index < nIndex) { |
|
1688
8a42db1eea60
removed all COMMA_CON / CON_COMMA uses
Claus Gittinger <cg@exept.de>
parents:
1290
diff
changeset
|
498 |
if (InterruptPending != nil) __interruptL(@line); |
603 | 499 |
|
1136 | 500 |
element = __InstPtr(self)->i_instvars[index]; |
1780
1b3a4ddc5b94
changed WeakArray to set emptied slots to zero
Claus Gittinger <cg@exept.de>
parents:
1688
diff
changeset
|
501 |
if (__isNonNilObject(element)) element = __WEAK_READ__(self, element); |
603 | 502 |
index++; |
1782
76bcca2667a8
oops - keysAndValuesDo: passed the wrong index
Claus Gittinger <cg@exept.de>
parents:
1780
diff
changeset
|
503 |
(*codeVal)(rHome, __MKSMALLINT(index-index0), element); |
603 | 504 |
} |
505 |
} else { |
|
506 |
while (index < nIndex) { |
|
1688
8a42db1eea60
removed all COMMA_CON / CON_COMMA uses
Claus Gittinger <cg@exept.de>
parents:
1290
diff
changeset
|
507 |
if (InterruptPending != nil) __interruptL(@line); |
603 | 508 |
|
1136 | 509 |
element = __InstPtr(self)->i_instvars[index]; |
1780
1b3a4ddc5b94
changed WeakArray to set emptied slots to zero
Claus Gittinger <cg@exept.de>
parents:
1688
diff
changeset
|
510 |
if (__isNonNilObject(element)) element = __WEAK_READ__(self, element); |
603 | 511 |
index++; |
1782
76bcca2667a8
oops - keysAndValuesDo: passed the wrong index
Claus Gittinger <cg@exept.de>
parents:
1780
diff
changeset
|
512 |
(*codeVal)(home, __MKSMALLINT(index-index0), element); |
603 | 513 |
} |
514 |
} |
|
515 |
#endif |
|
516 |
} else { |
|
517 |
while (index < nIndex) { |
|
1688
8a42db1eea60
removed all COMMA_CON / CON_COMMA uses
Claus Gittinger <cg@exept.de>
parents:
1290
diff
changeset
|
518 |
if (InterruptPending != nil) __interruptL(@line); |
603 | 519 |
|
1136 | 520 |
element = __InstPtr(self)->i_instvars[index]; |
1780
1b3a4ddc5b94
changed WeakArray to set emptied slots to zero
Claus Gittinger <cg@exept.de>
parents:
1688
diff
changeset
|
521 |
if (__isNonNilObject(element)) element = __WEAK_READ__(self, element); |
603 | 522 |
index++; |
523 |
(*val2.ilc_func)(aBlock, |
|
524 |
@symbol(value:value:), |
|
1688
8a42db1eea60
removed all COMMA_CON / CON_COMMA uses
Claus Gittinger <cg@exept.de>
parents:
1290
diff
changeset
|
525 |
nil, &val2, |
1782
76bcca2667a8
oops - keysAndValuesDo: passed the wrong index
Claus Gittinger <cg@exept.de>
parents:
1780
diff
changeset
|
526 |
__MKSMALLINT(index-index0), |
981
6da206fcc1c4
keep ref to weak-element somewhere (this is a try)
Claus Gittinger <cg@exept.de>
parents:
829
diff
changeset
|
527 |
element); |
603 | 528 |
} |
529 |
} |
|
530 |
%}. |
|
531 |
^ self |
|
532 |
! |
|
533 |
||
586
cab695f942a6
weakArray readBarrier for IGC
Claus Gittinger <cg@exept.de>
parents:
530
diff
changeset
|
534 |
nonNilElementsDo:aBlock |
cab695f942a6
weakArray readBarrier for IGC
Claus Gittinger <cg@exept.de>
parents:
530
diff
changeset
|
535 |
"evaluate the argument, aBlock for each non-nil element" |
cab695f942a6
weakArray readBarrier for IGC
Claus Gittinger <cg@exept.de>
parents:
530
diff
changeset
|
536 |
|
981
6da206fcc1c4
keep ref to weak-element somewhere (this is a try)
Claus Gittinger <cg@exept.de>
parents:
829
diff
changeset
|
537 |
|home element| |
586
cab695f942a6
weakArray readBarrier for IGC
Claus Gittinger <cg@exept.de>
parents:
530
diff
changeset
|
538 |
%{ |
cab695f942a6
weakArray readBarrier for IGC
Claus Gittinger <cg@exept.de>
parents:
530
diff
changeset
|
539 |
REGISTER OBJFUNC codeVal; |
cab695f942a6
weakArray readBarrier for IGC
Claus Gittinger <cg@exept.de>
parents:
530
diff
changeset
|
540 |
REGISTER int index; |
cab695f942a6
weakArray readBarrier for IGC
Claus Gittinger <cg@exept.de>
parents:
530
diff
changeset
|
541 |
int nIndex; |
cab695f942a6
weakArray readBarrier for IGC
Claus Gittinger <cg@exept.de>
parents:
530
diff
changeset
|
542 |
static struct inlineCache val = _ILC1; |
cab695f942a6
weakArray readBarrier for IGC
Claus Gittinger <cg@exept.de>
parents:
530
diff
changeset
|
543 |
REGISTER OBJ rHome; |
cab695f942a6
weakArray readBarrier for IGC
Claus Gittinger <cg@exept.de>
parents:
530
diff
changeset
|
544 |
|
1136 | 545 |
index = __intVal(__ClassInstPtr(__qClass(self))->c_ninstvars); |
586
cab695f942a6
weakArray readBarrier for IGC
Claus Gittinger <cg@exept.de>
parents:
530
diff
changeset
|
546 |
nIndex = __BYTES2OBJS__(__qSize(self) - OHDR_SIZE); |
cab695f942a6
weakArray readBarrier for IGC
Claus Gittinger <cg@exept.de>
parents:
530
diff
changeset
|
547 |
if (__isBlockLike(aBlock) |
1136 | 548 |
&& ((codeVal = __BlockInstPtr(aBlock)->b_code) != (OBJFUNC)nil) |
549 |
&& (__BlockInstPtr(aBlock)->b_nargs == __MKSMALLINT(1))) { |
|
586
cab695f942a6
weakArray readBarrier for IGC
Claus Gittinger <cg@exept.de>
parents:
530
diff
changeset
|
550 |
#ifdef NEW_BLOCK_CALL |
cab695f942a6
weakArray readBarrier for IGC
Claus Gittinger <cg@exept.de>
parents:
530
diff
changeset
|
551 |
for (; index < nIndex; index++) { |
cab695f942a6
weakArray readBarrier for IGC
Claus Gittinger <cg@exept.de>
parents:
530
diff
changeset
|
552 |
if (InterruptPending != nil) __interruptL(CONARG); |
cab695f942a6
weakArray readBarrier for IGC
Claus Gittinger <cg@exept.de>
parents:
530
diff
changeset
|
553 |
|
1136 | 554 |
element = __InstPtr(self)->i_instvars[index]; |
586
cab695f942a6
weakArray readBarrier for IGC
Claus Gittinger <cg@exept.de>
parents:
530
diff
changeset
|
555 |
if (element != nil) { |
cab695f942a6
weakArray readBarrier for IGC
Claus Gittinger <cg@exept.de>
parents:
530
diff
changeset
|
556 |
element = __WEAK_READ__(self, element); |
759
908363ce8a32
interest is written with one 'r' (shame on me)
Claus Gittinger <cg@exept.de>
parents:
681
diff
changeset
|
557 |
if (element != nil) { |
1688
8a42db1eea60
removed all COMMA_CON / CON_COMMA uses
Claus Gittinger <cg@exept.de>
parents:
1290
diff
changeset
|
558 |
(*codeVal)(aBlock, element); |
759
908363ce8a32
interest is written with one 'r' (shame on me)
Claus Gittinger <cg@exept.de>
parents:
681
diff
changeset
|
559 |
} |
586
cab695f942a6
weakArray readBarrier for IGC
Claus Gittinger <cg@exept.de>
parents:
530
diff
changeset
|
560 |
} |
cab695f942a6
weakArray readBarrier for IGC
Claus Gittinger <cg@exept.de>
parents:
530
diff
changeset
|
561 |
} |
cab695f942a6
weakArray readBarrier for IGC
Claus Gittinger <cg@exept.de>
parents:
530
diff
changeset
|
562 |
#else |
1136 | 563 |
home = __BlockInstPtr(aBlock)->b_home; |
586
cab695f942a6
weakArray readBarrier for IGC
Claus Gittinger <cg@exept.de>
parents:
530
diff
changeset
|
564 |
rHome = home; |
cab695f942a6
weakArray readBarrier for IGC
Claus Gittinger <cg@exept.de>
parents:
530
diff
changeset
|
565 |
if ((rHome == nil) || (__qSpace(rHome) >= STACKSPACE)) { |
cab695f942a6
weakArray readBarrier for IGC
Claus Gittinger <cg@exept.de>
parents:
530
diff
changeset
|
566 |
/* |
cab695f942a6
weakArray readBarrier for IGC
Claus Gittinger <cg@exept.de>
parents:
530
diff
changeset
|
567 |
* home will not move - keep in in a register |
cab695f942a6
weakArray readBarrier for IGC
Claus Gittinger <cg@exept.de>
parents:
530
diff
changeset
|
568 |
*/ |
cab695f942a6
weakArray readBarrier for IGC
Claus Gittinger <cg@exept.de>
parents:
530
diff
changeset
|
569 |
for (; index < nIndex; index++) { |
cab695f942a6
weakArray readBarrier for IGC
Claus Gittinger <cg@exept.de>
parents:
530
diff
changeset
|
570 |
if (InterruptPending != nil) __interruptL(CONARG); |
cab695f942a6
weakArray readBarrier for IGC
Claus Gittinger <cg@exept.de>
parents:
530
diff
changeset
|
571 |
|
1136 | 572 |
element = __InstPtr(self)->i_instvars[index]; |
586
cab695f942a6
weakArray readBarrier for IGC
Claus Gittinger <cg@exept.de>
parents:
530
diff
changeset
|
573 |
if (element != nil) { |
cab695f942a6
weakArray readBarrier for IGC
Claus Gittinger <cg@exept.de>
parents:
530
diff
changeset
|
574 |
element = __WEAK_READ__(self, element); |
cab695f942a6
weakArray readBarrier for IGC
Claus Gittinger <cg@exept.de>
parents:
530
diff
changeset
|
575 |
if (element != nil) { |
1688
8a42db1eea60
removed all COMMA_CON / CON_COMMA uses
Claus Gittinger <cg@exept.de>
parents:
1290
diff
changeset
|
576 |
(*codeVal)(rHome, element); |
586
cab695f942a6
weakArray readBarrier for IGC
Claus Gittinger <cg@exept.de>
parents:
530
diff
changeset
|
577 |
} |
cab695f942a6
weakArray readBarrier for IGC
Claus Gittinger <cg@exept.de>
parents:
530
diff
changeset
|
578 |
} |
cab695f942a6
weakArray readBarrier for IGC
Claus Gittinger <cg@exept.de>
parents:
530
diff
changeset
|
579 |
} |
cab695f942a6
weakArray readBarrier for IGC
Claus Gittinger <cg@exept.de>
parents:
530
diff
changeset
|
580 |
} else { |
cab695f942a6
weakArray readBarrier for IGC
Claus Gittinger <cg@exept.de>
parents:
530
diff
changeset
|
581 |
for (; index < nIndex; index++) { |
cab695f942a6
weakArray readBarrier for IGC
Claus Gittinger <cg@exept.de>
parents:
530
diff
changeset
|
582 |
if (InterruptPending != nil) __interruptL(@line CONARG); |
cab695f942a6
weakArray readBarrier for IGC
Claus Gittinger <cg@exept.de>
parents:
530
diff
changeset
|
583 |
|
1136 | 584 |
element = __InstPtr(self)->i_instvars[index]; |
586
cab695f942a6
weakArray readBarrier for IGC
Claus Gittinger <cg@exept.de>
parents:
530
diff
changeset
|
585 |
if (element != nil) { |
cab695f942a6
weakArray readBarrier for IGC
Claus Gittinger <cg@exept.de>
parents:
530
diff
changeset
|
586 |
element = __WEAK_READ__(self, element); |
cab695f942a6
weakArray readBarrier for IGC
Claus Gittinger <cg@exept.de>
parents:
530
diff
changeset
|
587 |
if (element != nil) { |
1688
8a42db1eea60
removed all COMMA_CON / CON_COMMA uses
Claus Gittinger <cg@exept.de>
parents:
1290
diff
changeset
|
588 |
(*codeVal)(home, element); |
586
cab695f942a6
weakArray readBarrier for IGC
Claus Gittinger <cg@exept.de>
parents:
530
diff
changeset
|
589 |
} |
cab695f942a6
weakArray readBarrier for IGC
Claus Gittinger <cg@exept.de>
parents:
530
diff
changeset
|
590 |
} |
cab695f942a6
weakArray readBarrier for IGC
Claus Gittinger <cg@exept.de>
parents:
530
diff
changeset
|
591 |
} |
cab695f942a6
weakArray readBarrier for IGC
Claus Gittinger <cg@exept.de>
parents:
530
diff
changeset
|
592 |
} |
cab695f942a6
weakArray readBarrier for IGC
Claus Gittinger <cg@exept.de>
parents:
530
diff
changeset
|
593 |
#endif |
cab695f942a6
weakArray readBarrier for IGC
Claus Gittinger <cg@exept.de>
parents:
530
diff
changeset
|
594 |
} else { |
cab695f942a6
weakArray readBarrier for IGC
Claus Gittinger <cg@exept.de>
parents:
530
diff
changeset
|
595 |
for (; index < nIndex; index++) { |
cab695f942a6
weakArray readBarrier for IGC
Claus Gittinger <cg@exept.de>
parents:
530
diff
changeset
|
596 |
if (InterruptPending != nil) __interruptL(@line CONARG); |
cab695f942a6
weakArray readBarrier for IGC
Claus Gittinger <cg@exept.de>
parents:
530
diff
changeset
|
597 |
|
1136 | 598 |
element = __InstPtr(self)->i_instvars[index]; |
586
cab695f942a6
weakArray readBarrier for IGC
Claus Gittinger <cg@exept.de>
parents:
530
diff
changeset
|
599 |
if (element != nil) { |
cab695f942a6
weakArray readBarrier for IGC
Claus Gittinger <cg@exept.de>
parents:
530
diff
changeset
|
600 |
element = __WEAK_READ__(self, element); |
cab695f942a6
weakArray readBarrier for IGC
Claus Gittinger <cg@exept.de>
parents:
530
diff
changeset
|
601 |
if (element != nil) { |
cab695f942a6
weakArray readBarrier for IGC
Claus Gittinger <cg@exept.de>
parents:
530
diff
changeset
|
602 |
(*val.ilc_func)(aBlock, |
cab695f942a6
weakArray readBarrier for IGC
Claus Gittinger <cg@exept.de>
parents:
530
diff
changeset
|
603 |
@symbol(value:), |
1688
8a42db1eea60
removed all COMMA_CON / CON_COMMA uses
Claus Gittinger <cg@exept.de>
parents:
1290
diff
changeset
|
604 |
nil, &val, |
586
cab695f942a6
weakArray readBarrier for IGC
Claus Gittinger <cg@exept.de>
parents:
530
diff
changeset
|
605 |
element); |
cab695f942a6
weakArray readBarrier for IGC
Claus Gittinger <cg@exept.de>
parents:
530
diff
changeset
|
606 |
} |
cab695f942a6
weakArray readBarrier for IGC
Claus Gittinger <cg@exept.de>
parents:
530
diff
changeset
|
607 |
} |
cab695f942a6
weakArray readBarrier for IGC
Claus Gittinger <cg@exept.de>
parents:
530
diff
changeset
|
608 |
} |
cab695f942a6
weakArray readBarrier for IGC
Claus Gittinger <cg@exept.de>
parents:
530
diff
changeset
|
609 |
} |
604 | 610 |
%}. |
586
cab695f942a6
weakArray readBarrier for IGC
Claus Gittinger <cg@exept.de>
parents:
530
diff
changeset
|
611 |
^ self |
cab695f942a6
weakArray readBarrier for IGC
Claus Gittinger <cg@exept.de>
parents:
530
diff
changeset
|
612 |
! ! |
cab695f942a6
weakArray readBarrier for IGC
Claus Gittinger <cg@exept.de>
parents:
530
diff
changeset
|
613 |
|
603 | 614 |
!WeakArray methodsFor:'notification'! |
586
cab695f942a6
weakArray readBarrier for IGC
Claus Gittinger <cg@exept.de>
parents:
530
diff
changeset
|
615 |
|
603 | 616 |
lostPointer |
617 |
"I lost a pointer; tell watcher and dependents. |
|
618 |
This is sent from the finalization in ObjectMemory." |
|
586
cab695f942a6
weakArray readBarrier for IGC
Claus Gittinger <cg@exept.de>
parents:
530
diff
changeset
|
619 |
|
603 | 620 |
dependents notNil ifTrue:[ |
621 |
self changed. |
|
622 |
]. |
|
623 |
watcher notNil ifTrue:[ |
|
624 |
watcher informDispose |
|
625 |
] |
|
626 |
! ! |
|
6 | 627 |
|
628 | 628 |
!WeakArray class methodsFor:'documentation'! |
629 |
||
630 |
version |
|
1782
76bcca2667a8
oops - keysAndValuesDo: passed the wrong index
Claus Gittinger <cg@exept.de>
parents:
1780
diff
changeset
|
631 |
^ '$Header: /cvs/stx/stx/libbasic/Attic/WeakArr.st,v 1.33 1996-10-18 13:20:50 cg Exp $' |
628 | 632 |
! ! |
603 | 633 |
WeakArray initialize! |