[squeak-dev] The Inbox: Kernel-ct.1368.mcz
David T. Lewis
lewis at mail.msen.com
Tue Jan 26 02:19:27 UTC 2021
On Mon, Jan 25, 2021 at 05:05:05PM -0800, Eliot Miranda wrote:
> Hi David,
>
> On Sun, Jan 24, 2021 at 8:10 AM David T. Lewis <lewis at mail.msen.com> wrote:
>
> > I like this proposal. It has no effect on the ProcessBrowser, and is
> > helpful when looking at processes elsewhere.
> >
> > Most processes do not have a name, but all processes respond in a concise
> > and useful way to #name.
> >
> > Process>>name
> > ^name ifNil: [ self hash asString forceTo: 5 paddingStartWith: $ ]
> >
>
> 5 is too short. The identityHash in Spur has 7 digits of significance and
> hash => scaledIdentityHash => identityHash * 256. Also asString is lazy;
> it just calls printString. I wouldn't contract at all:
>
> Process>>name
> ^name ifNil: [self hash printString]
>
>
> > So +1 from me
> >
What I said is that I like Christoph's proposal, that's all.
If we think that we should change the implementation of Process>>name
that's fine too, but let's not it get in the way of merging Christoph's
suggestion into trunk. OK?
Dave
More information about the Squeak-dev
mailing list
|