[squeak-dev] name in various FooTheme class>>#createFoo methods

Eliot Miranda eliot.miranda at gmail.com
Fri Oct 14 18:13:52 UTC 2016


Hi Marcel,

    I see the "name is shadowed" warning from methods such as

CommunityTheme class>>#createDark
"self createDark apply."
| name |
name := 'Community (dark)'.
^ (self named: name) in: [:theme |
theme merge: (self named: 'Squeak') overwrite: true.
theme name: name.
"General morph stuff."
theme
...

and I thought to correct them but then thought that they don't need the
name ten var at all, and hence realised they could be auto generated.  Are
they?  Is it safe to edit them to rename name to e.g. themeName, or even to
eliminate the temp var altogether?

_,,,^..^,,,_
best, Eliot
-------------- next part --------------
An HTML attachment was scrubbed...
URL: http://lists.squeakfoundation.org/pipermail/squeak-dev/attachments/20161014/639ccfa2/attachment.htm


More information about the Squeak-dev mailing list