[squeak-dev] The Trunk: Kernel-ct.1368.mcz

David T. Lewis lewis at mail.msen.com
Sat Feb 13 18:04:07 UTC 2021


I'll revert if it is not a good thing to do.

To me it seems both useful and harmless.

Dave

On Sat, Feb 13, 2021 at 08:59:54AM -0800, Eliot Miranda wrote:
> 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!
> > 
> > 
> 


More information about the Squeak-dev mailing list