[
Date Prev][
Date Next][
Thread Prev][
Thread Next][
Date Index][
Thread Index]
[
List Home]
Re: [cdt-dev] Proposed change in thread naming
|
Hi,
Thanks for the feedback.
So we've converged to (I've added all fields that I see in my Debug view):
A-
MyProcess [1530] [cores: 2,3]
Thread [1] MyProcess 18787 [core: 3] (Suspended: Container)
Thread [2] MyProcess 18788 [core: 2] (Running: Container)
Thread [3] Worker 18789 [core: 3] (Suspended: Container)
Thread [4] Heartbeat 18797 [core: 3] (Running: Container)
or
B -
MyProcess [1530] [cores: 2,3]
Thread #1 [MyProcess] 18787 [core: 3] (Suspended: Container)
Thread #2 [MyProcess] 18788 [core: 2] (Running: Container)
Thread #3 [Worker] 18789 [core: 3] (Suspended: Container)
Thread #4 [Heartbeat] 18797 [core: 3] (Running: Container)
I like A because it simply inserts the name of the thread into our existing format.
And I like B because putting the name in brackets makes it more understandable. Also, thread names can have spaces, so it is good to have some kind of grouping symbol.
I tried to have a hybrid of the two:
C-
MyProcess [1530] [cores: 2,3]
Thread [1] <MyProcess> 18787 [core: 3] (Suspended: Container)
Thread [2] <MyProcess> 18788 [core: 2] (Running: Container)
Thread [3] <Worker> 18789 [core: 3] (Suspended: Container)
Thread [4] <Heartbeat> 18797 [core: 3] (Running: Container)
We had an internal discussion here and we found that the different grouping delimiter (<>) helps differentiate between different groups. And although we liked #1, #2 for the id, we thought we would follow the current solution more and keep [1], [2].
Does someone disagree?
If not, this will become the new 'face' of the CDT Debug view for the next release.
Thanks!
Marc
> -----Original Message-----
> From: cdt-dev-bounces@xxxxxxxxxxx [mailto:cdt-dev-bounces@xxxxxxxxxxx]
> On Behalf Of Simon Marchi
> Sent: Wednesday, November 26, 2014 10:54 AM
> To: cdt-dev@xxxxxxxxxxx
> Subject: Re: [cdt-dev] Proposed change in thread naming
>
> On 2014-11-26 10:39 AM, Alena Laskavaia wrote:
> > In this case second set square brackets really don't add anything,so
> > it really becomes #1. Which is fine with me too, since it is what we
> > show now anyway :)
> >
> > Another variant with JDT similar look
> >
> > 5 -
> > MyProcess [1530]
> > Thread #1 [MyProcess] (Suspended: Container)
> > Thread #2 [MyProcess] (Running: Container)
> > Thread #3 [Worker] (Suspended: Container)
> > Thread #4 [Heartbeat] (Running: Container)
>
> This suggestion is the one I prefer so far.
> _______________________________________________
> cdt-dev mailing list
> cdt-dev@xxxxxxxxxxx
> To change your delivery options, retrieve your password, or unsubscribe
> from this list, visit https://dev.eclipse.org/mailman/listinfo/cdt-dev