[squeak-dev] A lot of shadowed variable names in Squeak 6.0 needing clearing up

tim Rowledge tim at rowledge.org
Sat Dec 31 00:24:31 UTC 2022



> On 2022-12-25, at 10:33 AM, tim Rowledge <tim at rowledge.org> wrote:
> 
> VariableNode>>variableGetterBlockIn: (index is shadowed)
> ContextVariablesInspector>>streamTemporaryVariablesOn: (context is shadowed)
> ContextVariablesInspector>>fieldAllTempVars (object is shadowed)
> ContextInspector>>streamTemporaryVariablesOn: (context is shadowed)
> ContextInspector>>streamStackVariablesOn: (object is shadowed)
> TableLayoutTest>>testShrinkWrapIssue (container is shadowed)
> TableLayoutTest>>testScrollPaneBarUpdate (container is shadowed)
> BasicInspector>>fieldObjectClass (object is shadowed)
> BasicInspector>>fieldObjectSize (object is shadowed)
> BasicInspector>>streamIndexedVariablesOn: (object is shadowed)
> TTFontFileHandle>>printOn: (filename is shadowed)
> CompiledCodeInspector>>fieldHeader (object is shadowed)
> CompiledCodeInspector>>fieldByteCodes (object is shadowed)
> SqueakTheme class>>createDuller (name is shadowed)
> Inspector>>streamError:on: (object is shadowed)
> Inspector>>getContents (object is shadowed)
> Inspector>>streamIndexedVariablesOn: (object is shadowed)
> Inspector>>streamOn:truncate:collectFields: (object is shadowed)
> Inspector>>valuePane (object is shadowed)
> Inspector>>elementGetterAt: (object is shadowed)
> SHTextStylerST80>>createTextAttributesForPixelHeight: (font is shadowed)
> IndentingListItemMorph>>charactersOccluded (icon is shadowed)
> FileList2 class>>endingSpecs (category is shadowed)
> VersionsBrowser>>fileOutSelection (file is shadowed)


> InspectorField class>>generateExpressionFrom:argumentNames: (name is shadowed)
> FtpUrl>>retrieveContents (password is shadowed)
> ClassBasedHelpTopic>>updateSubtopics (priority is shadowed)
> LazyListMorph>>drawOn: (color is shadowed)
> PluggableTreeMorph>>setSelectedPath: (setSelectedPathSelector is shadowed)
> StringTest>>testFindLiteral (string is shadowed)
> StringTest>>testFindSelector (string is shadowed)
> StringTest>>testFindSymbol (string is shadowed)
> SqueakMessageCategoriesHelp class>>addAllCategoriesTopicTo: (category is shadowed)
> Ascii85ConverterTest>>testEncodeDecode (encoded is shadowed)
> MVCMenuMorph>>invokeAt:in:allowKeyboard: (originalFocusHolder is shadowed)
> HtmlEntityInspector>>fieldHtml (object is shadowed)
> ReleaseBuilder class>>clearCaches (environment is shadowed)
> ReleaseBuilder class>>cleanUpBitstreamVeraSans (name is shadowed)
> ReleaseBuilder class>>checkForNilCategories (category is shadowed)
> RxMatcher>>copy:replacingMatchesWith: (stream is shadowed)
> Random>>next:into: (index is shadowed)

> Random>>nextBytes:into:startingAt: (index is shadowed)
That's not correct so far as I can see?


> Random>>hashSeed: (index is shadowed)
> PluggableMultiColumnListMorph>>selectionIndex: (listMorph is shadowed)
> PluggableMultiColumnListMorph>>hoverRow: (listMorph is shadowed)
> PluggableMultiColumnListMorph>>textColor: (listMorph is shadowed)
> PluggableMultiColumnListMorph>>getFilteredList (fullList is shadowed)
> PluggableMultiColumnListMorph>>rowAtLocation: (listMorph is shadowed)
> PluggableMultiColumnListMorph>>updateColumns (listMorph is shadowed)
> PluggableMultiColumnListMorph>>rowAboutToBecomeSelected: (listMorph is shadowed)
> DockingBarMenuMorph>>popUpAdjacentTo:forHand:from: (owner is shadowed)
> SystemWindow>>paneColorToUseWhenNotActive (color is shadowed)
> CommunityTheme class>>createDark (name is shadowed)

OK, that's the list checked over. Now the problem is that 23 packages get touched. So just post the changeset right now and hope that does for today.

-------------- next part --------------
A non-text attachment was scrubbed...
Name: ShadowedVariablesCleanups.2.cs
Type: application/octet-stream
Size: 53100 bytes
Desc: not available
URL: <http://lists.squeakfoundation.org/pipermail/squeak-dev/attachments/20221230/115d6fa3/attachment-0001.obj>
-------------- next part --------------

Marcel, quite a lot appear to be artefacts from a morph related cleanup blast you did a while back.

tim
--
tim Rowledge; tim at rowledge.org; http://www.rowledge.org/tim
Strange OpCodes: BSO: Branch Sort Of




More information about the Squeak-dev mailing list