Hi Eliot,


we already discussed this below my inbox proposal, but I still think that "destiny name of unnamed processed" is more than noise. It is still helpful to compare the identities of two process instances by taking a short look at their print strings ...


Best,

Christoph

<http://www.hpi.de/>
________________________________
Von: Squeak-dev <squeak-dev-bounces at lists.squeakfoundation.org> im Auftrag von Eliot Miranda <eliot.miranda at gmail.com>
Gesendet: Samstag, 13. Februar 2021 17:59:54
An: squeak-dev at lists.squeakfoundation.org
Betreff: Re: [squeak-dev] The Trunk: Kernel-ct.1368.mcz

I disagree with this.  Only certain processes are named.  Including the destiny name of unnamed processes just introduces noise.  By all means include the name of named processes, but please *don’t* add verbosity where it is unhelpful.

_,,,^..^,,,_ (phone)

> On Feb 12, 2021, at 3:34 PM, commits at source.squeak.org wrote:
>
> David T. Lewis uploaded a new version of Kernel to project The Trunk:
> http://source.squeak.org/trunk/Kernel-ct.1368.mcz
>
> ==================== Summary ====================
>
> Name: Kernel-ct.1368
> Author: ct
> Time: 24 January 2021, 3:19:48.003384 pm
> UUID: c62a6062-b240-9149-a2ed-78fdc1079dfd
> Ancestors: Kernel-mt.1364
>
> Proposal: Always include process name into its print string. This facilitates debugging/inspecting of multiprocess scenarios.
>
> =============== Diff against Kernel-mt.1364 ===============
>
> Item was changed:
>  ----- Method: Process>>longPrintOn: (in category 'printing') -----
>  longPrintOn: stream
>
>      | ctxt |
>      super printOn: stream.
> +    stream
> +        nextPut: $(;
> +        nextPutAll: self name;
> +        nextPut: $).
>      stream cr.
>      ctxt := self suspendedContext.
>      [ctxt == nil] whileFalse: [
>          stream space.
>          ctxt printOn: stream.
>          stream cr.
>          ctxt := ctxt sender.
>      ].
>  !
>
> Item was changed:
>  ----- Method: Process>>printOn: (in category 'printing') -----
>  printOn: aStream
>
>      super printOn: aStream.
> +    aStream
> +        nextPut: $(;
> +        nextPutAll: self name;
> +        nextPut: $).
>      aStream nextPutAll: ' in '.
>      suspendedContext printOn: aStream!
>
>

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.squeakfoundation.org/pipermail/squeak-dev/attachments/20210213/0867785d/attachment.html>