From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (qmail 5921 invoked by alias); 7 May 2013 14:35:19 -0000 Mailing-List: contact gdb-patches-help@sourceware.org; run by ezmlm Precedence: bulk List-Id: List-Subscribe: List-Archive: List-Post: List-Help: , Sender: gdb-patches-owner@sourceware.org Received: (qmail 5911 invoked by uid 89); 7 May 2013 14:35:18 -0000 X-Spam-SWARE-Status: No, score=-8.4 required=5.0 tests=AWL,BAYES_00,KHOP_THREADED,RCVD_IN_HOSTKARMA_W,RCVD_IN_HOSTKARMA_WL,RP_MATCHES_RCVD,SPF_HELO_PASS,SPF_PASS autolearn=ham version=3.3.1 Received: from mx1.redhat.com (HELO mx1.redhat.com) (209.132.183.28) by sourceware.org (qpsmtpd/0.84/v0.84-167-ge50287c) with ESMTP; Tue, 07 May 2013 14:35:18 +0000 Received: from int-mx12.intmail.prod.int.phx2.redhat.com (int-mx12.intmail.prod.int.phx2.redhat.com [10.5.11.25]) by mx1.redhat.com (8.14.4/8.14.4) with ESMTP id r47EZG9I023080 (version=TLSv1/SSLv3 cipher=DHE-RSA-AES256-SHA bits=256 verify=OK); Tue, 7 May 2013 10:35:16 -0400 Received: from [127.0.0.1] (ovpn01.gateway.prod.ext.ams2.redhat.com [10.39.146.11]) by int-mx12.intmail.prod.int.phx2.redhat.com (8.14.4/8.14.4) with ESMTP id r47EZEUL022560; Tue, 7 May 2013 10:35:15 -0400 Message-ID: <51891122.8030800@redhat.com> Date: Tue, 07 May 2013 14:35:00 -0000 From: Pedro Alves User-Agent: Mozilla/5.0 (X11; Linux x86_64; rv:17.0) Gecko/20130311 Thunderbird/17.0.4 MIME-Version: 1.0 To: Joel Brobecker CC: gdb-patches@sourceware.org Subject: Re: [RFA 2/3] Improve output of "maintenance info sol-threads" command. References: <518284AF.5090802@redhat.com> <1367929532-4849-3-git-send-email-brobecker@adacore.com> In-Reply-To: <1367929532-4849-3-git-send-email-brobecker@adacore.com> Content-Type: text/plain; charset=ISO-8859-1 Content-Transfer-Encoding: 7bit X-SW-Source: 2013-05/txt/msg00229.txt.bz2 On 05/07/2013 01:25 PM, Joel Brobecker wrote: > This patch does the following: > - Puts the startfunc and "Sleep func" info on the same line; > - Renames "Sleep func" into "sleepfunc" to be consistent with > "startfunc" > - Avoids the use of a '-' as a separate before the "sleepfunc" > output, because the '-' looks odd and out of place when the > "startfunc" field is not printed (ti.ti_startfunc is nul). > - Use a '=' instead of ':' and avoids the space between > the name of the value and its value, mostly to help group > the value with its name. I'm fine with what I'm imagining it looks like (or whatever, really), but could you give an example of how things now look after the patch, for the archives/log? An output paste is worth a thousand words. :-) Thanks! -- Pedro Alves