[FIX] isKindOfbyIsClass-asm ( [cd][er][su][et][sl] Recommend including the modified cs )

tomkoenig at mindspring.com tomkoenig at mindspring.com
Mon Dec 6 04:21:28 CET 2004


Alejandro's original changeset modified 55 classes and 74 methods, if I
counted correctly. 10 months later only about 24 of the changes still
apply.  They are included in the change set.
I deleted changes to CRDictionary, CRDictionaryBrowser,
PluggableCollectionMorph, PluginHTTPRequest since these are no longer in
the image and naturally I could not load the cs until they were removed.
I inspected all of the methods in the cs.  A large number were deleted
from the change set because the changes had already been made
(especially by Ned, e.g. 6097 isStringFixes-nk).  In a number of cases,
subsequent changes had been implemented so I regressed to the prior
verions, and reapplied Alexandro changes.

I then found/created, executed and verified manual tests for all but one
of the remaining methods, with 3.9a#6520. All but two tests that  passed
successfully.  I  backed out one change that should have worked but did
not and reported the bug separately.  I was not able to create a test
for   ImageSegment, so that change  in a separate cs.     
I recommend that we include the (modified) isKindOfbyIsClass-asm
changeset in 3.9a but not the isKindOfForImageSegment cs until we can
come up with a convincing test for ImageSement
Tom
The list of tests conducted follows
Behavior >> literalScannedAs:notifying:  tested using examples in method
plus sunit test
Form literalScannedAs: 14 notifying: nil  
Form literalScannedAs: #OneBitForm notifying: nil   
Form literalScannedAs: ##OneBitForm notifying: nil           
Form literalScannedAs: ##Form notifying: nil                   
Form literalScannedAs: ###Form notifying: nil  
BitBlt >> paintBit pixelAt:put:  
BitBlt benchmark3  [ showed no difference to speak of]    used examples
Browser >>  defineClass:notifying:  	BrowseTest
Class tested using DeepCopier new checkVariables a DeepCopier.
ClassBuilder ClassBuilderTest
ClassDescription ClassDescriptionTest
GenericPropertiesMorph tested using TextPropertiesMorph initialize 
(InfiniteForm with: (Form extent: 40 at 40)) displayOn: Display at: 10 at 10 
  clippingBox: (Rectangle center: 5 at 5 extent:  15 at 15) rule: 1 fillColor:
Color red.
(Inspector inspect: 1) copyName
(Inspector inspect: #('a' 'b')) copyName
PackageInfo new externalCallers
(ParagraphEditor newParagraph: (Paragraph withText: (Text fromString:
'abcdefg'))) explain
PhraseTileMorph new vocabulary: Vocabulary new 
PhraseTileMorph new vocabulary: #S
Player new slotInfoAt: '5'
(PluggableListView on: (#('1' '2' '3') asOrderedCollection) 
  list: #contents selected: #first changeSelected: #at:put:)
verifyContents
PostscriptCanvas new stroke: Color red [this fails but not before
successfully testing changed code]
 PseudoClass new name: 'Object'; exists
PseudoClass new name: 'Display'; exists
(Scanner new scan: (ReadStream on: '#at  1')) nextLiteral
(Scanner new scan: (ReadStream on: '#at  1')) nextLiteral; nextLiteral
SequenceableCollection (plus sunit tests)
#('a' b ) asStringWithCr
#('a' b ) copyReplaceAll: 'a' with: 'x' asTokens: false
'a b'   copyReplaceAll: 'a' with: 'x' asTokens: false
SystemNavigation default browseObsoleteReferences 
(TextURL scanFrom:'www.altavista.digital.com/') actOnClickFor: BlobMorph
new
TheWorldMenu new appearanceMenu
TileMorph new rawVocabulary: Vocabulary new
(UpdatingStringMorph on: #(s ) selector: #first) readFromTarget
-------------- next part --------------
A non-text attachment was scrubbed...
Name: isKindOfForImageSegment.cs.gz
Type: application/octet-stream
Size: 2424 bytes
Desc: not available
Url : http://lists.squeakfoundation.org/pipermail/squeak-harvest/attachments/20041205/8c1924c6/isKindOfForImageSegment.cs.obj
-------------- next part --------------
A non-text attachment was scrubbed...
Name: isKindOfbyIsClass-asm.cs.gz
Type: application/octet-stream
Size: 5764 bytes
Desc: not available
Url : http://lists.squeakfoundation.org/pipermail/squeak-harvest/attachments/20041205/8c1924c6/isKindOfbyIsClass-asm.cs.obj


More information about the Squeak-harvest mailing list