WeakIdentitySet.st
author ca
Sat, 20 Apr 1996 11:28:14 +0200
changeset 1232 25b756de5c51
parent 1054 62ce964f1d2e
child 1263 92c1db6b0776
permissions -rw-r--r--
oops - disposal left tally with wrong value (if deletedObjects are present)
Ignore whitespace changes - Everywhere: Within whitespace: At end of lines:
94
649321db4b9a Initial revision
claus
parents:
diff changeset
     1
"
649321db4b9a Initial revision
claus
parents:
diff changeset
     2
 COPYRIGHT (c) 1994 by Claus Gittinger
187
a5dc8e2fa93f size 0 bug
claus
parents: 159
diff changeset
     3
	      All Rights Reserved
94
649321db4b9a Initial revision
claus
parents:
diff changeset
     4
649321db4b9a Initial revision
claus
parents:
diff changeset
     5
 This software is furnished under a license and may be used
649321db4b9a Initial revision
claus
parents:
diff changeset
     6
 only in accordance with the terms of that license and with the
649321db4b9a Initial revision
claus
parents:
diff changeset
     7
 inclusion of the above copyright notice.   This software may not
649321db4b9a Initial revision
claus
parents:
diff changeset
     8
 be provided or otherwise made available to, or used by, any
649321db4b9a Initial revision
claus
parents:
diff changeset
     9
 other person.  No title to or ownership of the software is
649321db4b9a Initial revision
claus
parents:
diff changeset
    10
 hereby transferred.
649321db4b9a Initial revision
claus
parents:
diff changeset
    11
"
649321db4b9a Initial revision
claus
parents:
diff changeset
    12
649321db4b9a Initial revision
claus
parents:
diff changeset
    13
IdentitySet subclass:#WeakIdentitySet
1054
62ce964f1d2e commentary
Claus Gittinger <cg@exept.de>
parents: 924
diff changeset
    14
	instanceVariableNames:''
62ce964f1d2e commentary
Claus Gittinger <cg@exept.de>
parents: 924
diff changeset
    15
	classVariableNames:''
62ce964f1d2e commentary
Claus Gittinger <cg@exept.de>
parents: 924
diff changeset
    16
	poolDictionaries:''
62ce964f1d2e commentary
Claus Gittinger <cg@exept.de>
parents: 924
diff changeset
    17
	category:'Collections-Unordered'
94
649321db4b9a Initial revision
claus
parents:
diff changeset
    18
!
649321db4b9a Initial revision
claus
parents:
diff changeset
    19
649321db4b9a Initial revision
claus
parents:
diff changeset
    20
!WeakIdentitySet class methodsFor:'documentation'!
649321db4b9a Initial revision
claus
parents:
diff changeset
    21
649321db4b9a Initial revision
claus
parents:
diff changeset
    22
copyright
649321db4b9a Initial revision
claus
parents:
diff changeset
    23
"
649321db4b9a Initial revision
claus
parents:
diff changeset
    24
 COPYRIGHT (c) 1994 by Claus Gittinger
187
a5dc8e2fa93f size 0 bug
claus
parents: 159
diff changeset
    25
	      All Rights Reserved
94
649321db4b9a Initial revision
claus
parents:
diff changeset
    26
649321db4b9a Initial revision
claus
parents:
diff changeset
    27
 This software is furnished under a license and may be used
649321db4b9a Initial revision
claus
parents:
diff changeset
    28
 only in accordance with the terms of that license and with the
649321db4b9a Initial revision
claus
parents:
diff changeset
    29
 inclusion of the above copyright notice.   This software may not
649321db4b9a Initial revision
claus
parents:
diff changeset
    30
 be provided or otherwise made available to, or used by, any
649321db4b9a Initial revision
claus
parents:
diff changeset
    31
 other person.  No title to or ownership of the software is
649321db4b9a Initial revision
claus
parents:
diff changeset
    32
 hereby transferred.
649321db4b9a Initial revision
claus
parents:
diff changeset
    33
"
649321db4b9a Initial revision
claus
parents:
diff changeset
    34
!
649321db4b9a Initial revision
claus
parents:
diff changeset
    35
649321db4b9a Initial revision
claus
parents:
diff changeset
    36
documentation
649321db4b9a Initial revision
claus
parents:
diff changeset
    37
"
649321db4b9a Initial revision
claus
parents:
diff changeset
    38
    this is a special class to support dependencies which do not
649321db4b9a Initial revision
claus
parents:
diff changeset
    39
    prevent objects from dying.
649321db4b9a Initial revision
claus
parents:
diff changeset
    40
"
649321db4b9a Initial revision
claus
parents:
diff changeset
    41
! !
649321db4b9a Initial revision
claus
parents:
diff changeset
    42
111
9cdd0ed47f38 default size is smaller
claus
parents: 94
diff changeset
    43
!WeakIdentitySet class methodsFor:'instance creation'!
9cdd0ed47f38 default size is smaller
claus
parents: 94
diff changeset
    44
9cdd0ed47f38 default size is smaller
claus
parents: 94
diff changeset
    45
new
9cdd0ed47f38 default size is smaller
claus
parents: 94
diff changeset
    46
    "return a new empty Set"
9cdd0ed47f38 default size is smaller
claus
parents: 94
diff changeset
    47
9cdd0ed47f38 default size is smaller
claus
parents: 94
diff changeset
    48
    ^ self new:1
9cdd0ed47f38 default size is smaller
claus
parents: 94
diff changeset
    49
! !
9cdd0ed47f38 default size is smaller
claus
parents: 94
diff changeset
    50
609
12be97f6d5a7 checkin from browser
Claus Gittinger <cg@exept.de>
parents: 530
diff changeset
    51
!WeakIdentitySet class methodsFor:'queries'!
12be97f6d5a7 checkin from browser
Claus Gittinger <cg@exept.de>
parents: 530
diff changeset
    52
12be97f6d5a7 checkin from browser
Claus Gittinger <cg@exept.de>
parents: 530
diff changeset
    53
goodSizeFrom:arg
12be97f6d5a7 checkin from browser
Claus Gittinger <cg@exept.de>
parents: 530
diff changeset
    54
    "return a good array size for the given argument.
12be97f6d5a7 checkin from browser
Claus Gittinger <cg@exept.de>
parents: 530
diff changeset
    55
     Since WeakIdentitySets are mostly used for dependency management, we typically
12be97f6d5a7 checkin from browser
Claus Gittinger <cg@exept.de>
parents: 530
diff changeset
    56
     have only one element in the set. Therefore use exact size for small sets
12be97f6d5a7 checkin from browser
Claus Gittinger <cg@exept.de>
parents: 530
diff changeset
    57
     (instead of rounding up to the prime 11)."
12be97f6d5a7 checkin from browser
Claus Gittinger <cg@exept.de>
parents: 530
diff changeset
    58
12be97f6d5a7 checkin from browser
Claus Gittinger <cg@exept.de>
parents: 530
diff changeset
    59
    arg <= 10 ifTrue:[
12be97f6d5a7 checkin from browser
Claus Gittinger <cg@exept.de>
parents: 530
diff changeset
    60
	arg < 1 ifTrue:[^ 1].
12be97f6d5a7 checkin from browser
Claus Gittinger <cg@exept.de>
parents: 530
diff changeset
    61
	^ arg.
12be97f6d5a7 checkin from browser
Claus Gittinger <cg@exept.de>
parents: 530
diff changeset
    62
    ].
12be97f6d5a7 checkin from browser
Claus Gittinger <cg@exept.de>
parents: 530
diff changeset
    63
    ^ super goodSizeFrom:arg
12be97f6d5a7 checkin from browser
Claus Gittinger <cg@exept.de>
parents: 530
diff changeset
    64
! !
12be97f6d5a7 checkin from browser
Claus Gittinger <cg@exept.de>
parents: 530
diff changeset
    65
359
claus
parents: 252
diff changeset
    66
!WeakIdentitySet methodsFor:'adding & removing'!
claus
parents: 252
diff changeset
    67
claus
parents: 252
diff changeset
    68
add:newElement 
claus
parents: 252
diff changeset
    69
    "redefined to avoid synchronization promlems, in case
claus
parents: 252
diff changeset
    70
     of interrupts 
claus
parents: 252
diff changeset
    71
     (otherwise, there could be some other operation on the receiver
claus
parents: 252
diff changeset
    72
       done by another process, which garbles my contents)"
claus
parents: 252
diff changeset
    73
claus
parents: 252
diff changeset
    74
    |wasBlocked|
claus
parents: 252
diff changeset
    75
claus
parents: 252
diff changeset
    76
    wasBlocked := OperatingSystem blockInterrupts.
claus
parents: 252
diff changeset
    77
    super add:newElement.
claus
parents: 252
diff changeset
    78
    wasBlocked ifFalse:[OperatingSystem unblockInterrupts]
claus
parents: 252
diff changeset
    79
!
claus
parents: 252
diff changeset
    80
claus
parents: 252
diff changeset
    81
remove:anObject ifAbsent:exceptionBlock
1054
62ce964f1d2e commentary
Claus Gittinger <cg@exept.de>
parents: 924
diff changeset
    82
    "redefined to avoid synchronization problems, in case
359
claus
parents: 252
diff changeset
    83
     of interrupts 
claus
parents: 252
diff changeset
    84
     (otherwise, there could be some other operation on the receiver
claus
parents: 252
diff changeset
    85
       done by another process, which garbles my contents)"
claus
parents: 252
diff changeset
    86
claus
parents: 252
diff changeset
    87
    |wasBlocked|
claus
parents: 252
diff changeset
    88
claus
parents: 252
diff changeset
    89
    wasBlocked := OperatingSystem blockInterrupts.
claus
parents: 252
diff changeset
    90
    [
1054
62ce964f1d2e commentary
Claus Gittinger <cg@exept.de>
parents: 924
diff changeset
    91
        super remove:anObject ifAbsent:exceptionBlock
359
claus
parents: 252
diff changeset
    92
    ] valueNowOrOnUnwindDo:[
1054
62ce964f1d2e commentary
Claus Gittinger <cg@exept.de>
parents: 924
diff changeset
    93
        wasBlocked ifFalse:[OperatingSystem unblockInterrupts]
359
claus
parents: 252
diff changeset
    94
    ]
1054
62ce964f1d2e commentary
Claus Gittinger <cg@exept.de>
parents: 924
diff changeset
    95
62ce964f1d2e commentary
Claus Gittinger <cg@exept.de>
parents: 924
diff changeset
    96
    "Modified: 1.3.1996 / 21:16:10 / cg"
359
claus
parents: 252
diff changeset
    97
! !
claus
parents: 252
diff changeset
    98
609
12be97f6d5a7 checkin from browser
Claus Gittinger <cg@exept.de>
parents: 530
diff changeset
    99
!WeakIdentitySet methodsFor:'element disposal'!
94
649321db4b9a Initial revision
claus
parents:
diff changeset
   100
609
12be97f6d5a7 checkin from browser
Claus Gittinger <cg@exept.de>
parents: 530
diff changeset
   101
informDispose
12be97f6d5a7 checkin from browser
Claus Gittinger <cg@exept.de>
parents: 530
diff changeset
   102
    "an element died - must rehash"
12be97f6d5a7 checkin from browser
Claus Gittinger <cg@exept.de>
parents: 530
diff changeset
   103
1232
25b756de5c51 oops - disposal left tally with wrong value (if deletedObjects are present)
ca
parents: 1054
diff changeset
   104
    |wasBlocked cnt deleted|
25b756de5c51 oops - disposal left tally with wrong value (if deletedObjects are present)
ca
parents: 1054
diff changeset
   105
25b756de5c51 oops - disposal left tally with wrong value (if deletedObjects are present)
ca
parents: 1054
diff changeset
   106
    deleted := DeletedEntry.
111
9cdd0ed47f38 default size is smaller
claus
parents: 94
diff changeset
   107
609
12be97f6d5a7 checkin from browser
Claus Gittinger <cg@exept.de>
parents: 530
diff changeset
   108
    "
12be97f6d5a7 checkin from browser
Claus Gittinger <cg@exept.de>
parents: 530
diff changeset
   109
     must block interrupts here - finalization
12be97f6d5a7 checkin from browser
Claus Gittinger <cg@exept.de>
parents: 530
diff changeset
   110
     may be done at low prio in the background
12be97f6d5a7 checkin from browser
Claus Gittinger <cg@exept.de>
parents: 530
diff changeset
   111
     finalizer, we do not want to be interrupted
12be97f6d5a7 checkin from browser
Claus Gittinger <cg@exept.de>
parents: 530
diff changeset
   112
     while rehashing
12be97f6d5a7 checkin from browser
Claus Gittinger <cg@exept.de>
parents: 530
diff changeset
   113
    "
12be97f6d5a7 checkin from browser
Claus Gittinger <cg@exept.de>
parents: 530
diff changeset
   114
    wasBlocked := OperatingSystem blockInterrupts.
924
5745a1132511 oops - size was not updated when elements were disposed (update tally)
Claus Gittinger <cg@exept.de>
parents: 636
diff changeset
   115
5745a1132511 oops - size was not updated when elements were disposed (update tally)
Claus Gittinger <cg@exept.de>
parents: 636
diff changeset
   116
    "/
5745a1132511 oops - size was not updated when elements were disposed (update tally)
Claus Gittinger <cg@exept.de>
parents: 636
diff changeset
   117
    "/ recompute my tally
5745a1132511 oops - size was not updated when elements were disposed (update tally)
Claus Gittinger <cg@exept.de>
parents: 636
diff changeset
   118
    "/
5745a1132511 oops - size was not updated when elements were disposed (update tally)
Claus Gittinger <cg@exept.de>
parents: 636
diff changeset
   119
    cnt := 0.
1232
25b756de5c51 oops - disposal left tally with wrong value (if deletedObjects are present)
ca
parents: 1054
diff changeset
   120
    keyArray nonNilElementsDo:[:el | el ~~ deleted ifTrue:[cnt := cnt + 1]].
924
5745a1132511 oops - size was not updated when elements were disposed (update tally)
Claus Gittinger <cg@exept.de>
parents: 636
diff changeset
   121
    tally := cnt.
5745a1132511 oops - size was not updated when elements were disposed (update tally)
Claus Gittinger <cg@exept.de>
parents: 636
diff changeset
   122
609
12be97f6d5a7 checkin from browser
Claus Gittinger <cg@exept.de>
parents: 530
diff changeset
   123
    self rehash.
12be97f6d5a7 checkin from browser
Claus Gittinger <cg@exept.de>
parents: 530
diff changeset
   124
    wasBlocked ifFalse:[OperatingSystem unblockInterrupts].
94
649321db4b9a Initial revision
claus
parents:
diff changeset
   125
! !
649321db4b9a Initial revision
claus
parents:
diff changeset
   126
216
a8abff749575 *** empty log message ***
claus
parents: 187
diff changeset
   127
!WeakIdentitySet methodsFor:'enumerating'!
a8abff749575 *** empty log message ***
claus
parents: 187
diff changeset
   128
a8abff749575 *** empty log message ***
claus
parents: 187
diff changeset
   129
do:aBlock
a8abff749575 *** empty log message ***
claus
parents: 187
diff changeset
   130
    "perform the block for all members in the collection.
a8abff749575 *** empty log message ***
claus
parents: 187
diff changeset
   131
     This method has been rewritten to be more robust about
a8abff749575 *** empty log message ***
claus
parents: 187
diff changeset
   132
     changed contents while enumerating elements (which might
a8abff749575 *** empty log message ***
claus
parents: 187
diff changeset
   133
     happen during change&update processing, if dependents
a8abff749575 *** empty log message ***
claus
parents: 187
diff changeset
   134
     are added or removed within the update."
a8abff749575 *** empty log message ***
claus
parents: 187
diff changeset
   135
a8abff749575 *** empty log message ***
claus
parents: 187
diff changeset
   136
    |index "{ Class: SmallInteger }"
a8abff749575 *** empty log message ***
claus
parents: 187
diff changeset
   137
     element|
a8abff749575 *** empty log message ***
claus
parents: 187
diff changeset
   138
a8abff749575 *** empty log message ***
claus
parents: 187
diff changeset
   139
    index := 1.
a8abff749575 *** empty log message ***
claus
parents: 187
diff changeset
   140
    [index <= keyArray size] whileTrue:[
a8abff749575 *** empty log message ***
claus
parents: 187
diff changeset
   141
	element := keyArray at:index.
a8abff749575 *** empty log message ***
claus
parents: 187
diff changeset
   142
	(element notNil and:[element ~~ DeletedEntry]) ifTrue:[
a8abff749575 *** empty log message ***
claus
parents: 187
diff changeset
   143
	    aBlock value:element
a8abff749575 *** empty log message ***
claus
parents: 187
diff changeset
   144
	].
a8abff749575 *** empty log message ***
claus
parents: 187
diff changeset
   145
	index := index + 1
a8abff749575 *** empty log message ***
claus
parents: 187
diff changeset
   146
    ]
a8abff749575 *** empty log message ***
claus
parents: 187
diff changeset
   147
! !
a8abff749575 *** empty log message ***
claus
parents: 187
diff changeset
   148
609
12be97f6d5a7 checkin from browser
Claus Gittinger <cg@exept.de>
parents: 530
diff changeset
   149
!WeakIdentitySet methodsFor:'private'!
94
649321db4b9a Initial revision
claus
parents:
diff changeset
   150
609
12be97f6d5a7 checkin from browser
Claus Gittinger <cg@exept.de>
parents: 530
diff changeset
   151
keyContainerOfSize:n
12be97f6d5a7 checkin from browser
Claus Gittinger <cg@exept.de>
parents: 530
diff changeset
   152
    "return a container for keys and values of size n.
12be97f6d5a7 checkin from browser
Claus Gittinger <cg@exept.de>
parents: 530
diff changeset
   153
     use WeakArrays here."
359
claus
parents: 252
diff changeset
   154
609
12be97f6d5a7 checkin from browser
Claus Gittinger <cg@exept.de>
parents: 530
diff changeset
   155
    |w|
12be97f6d5a7 checkin from browser
Claus Gittinger <cg@exept.de>
parents: 530
diff changeset
   156
12be97f6d5a7 checkin from browser
Claus Gittinger <cg@exept.de>
parents: 530
diff changeset
   157
    w := WeakArray new:n.
12be97f6d5a7 checkin from browser
Claus Gittinger <cg@exept.de>
parents: 530
diff changeset
   158
    w watcher:self.
12be97f6d5a7 checkin from browser
Claus Gittinger <cg@exept.de>
parents: 530
diff changeset
   159
    ^ w
94
649321db4b9a Initial revision
claus
parents:
diff changeset
   160
! !
609
12be97f6d5a7 checkin from browser
Claus Gittinger <cg@exept.de>
parents: 530
diff changeset
   161
636
295267cdb5d5 version at the end
Claus Gittinger <cg@exept.de>
parents: 609
diff changeset
   162
!WeakIdentitySet class methodsFor:'documentation'!
295267cdb5d5 version at the end
Claus Gittinger <cg@exept.de>
parents: 609
diff changeset
   163
295267cdb5d5 version at the end
Claus Gittinger <cg@exept.de>
parents: 609
diff changeset
   164
version
1232
25b756de5c51 oops - disposal left tally with wrong value (if deletedObjects are present)
ca
parents: 1054
diff changeset
   165
    ^ '$Header: /cvs/stx/stx/libbasic/WeakIdentitySet.st,v 1.15 1996-04-20 09:28:14 ca Exp $'
636
295267cdb5d5 version at the end
Claus Gittinger <cg@exept.de>
parents: 609
diff changeset
   166
! !