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@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;
stream cr. ctxt := self suspendedContext. [ctxt == nil] whileFalse: [ stream space. ctxt printOn: stream. stream cr. ctxt := ctxt sender. ].nextPut: $).
!
Item was changed: ----- Method: Process>>printOn: (in category 'printing') ----- printOn: aStream
super printOn: aStream.
- aStream
nextPut: $(;
nextPutAll: self name;
aStream nextPutAll: ' in '. suspendedContext printOn: aStream!nextPut: $).