From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (qmail 27180 invoked by alias); 17 Oct 2011 16:29:59 -0000 Received: (qmail 27171 invoked by uid 22791); 17 Oct 2011 16:29:58 -0000 X-SWARE-Spam-Status: No, hits=-1.1 required=5.0 tests=AWL,BAYES_00,RCVD_IN_DNSWL_NONE,SPF_SOFTFAIL X-Spam-Check-By: sourceware.org Received: from mtaout22.012.net.il (HELO mtaout22.012.net.il) (80.179.55.172) by sourceware.org (qpsmtpd/0.43rc1) with ESMTP; Mon, 17 Oct 2011 16:29:43 +0000 Received: from conversion-daemon.a-mtaout22.012.net.il by a-mtaout22.012.net.il (HyperSendmail v2007.08) id <0LT700000XQZCF00@a-mtaout22.012.net.il> for gdb-patches@sourceware.org; Mon, 17 Oct 2011 18:29:42 +0200 (IST) Received: from HOME-C4E4A596F7 ([77.124.212.197]) by a-mtaout22.012.net.il (HyperSendmail v2007.08) with ESMTPA id <0LT70004OXTH9330@a-mtaout22.012.net.il>; Mon, 17 Oct 2011 18:29:42 +0200 (IST) Date: Mon, 17 Oct 2011 16:39:00 -0000 From: Eli Zaretskii Subject: Re: [PATCH] Tracing notes and metadata In-reply-to: <4E9C5112.9070705@mentor.com> To: Stan Shebs Cc: gdb-patches@sourceware.org Reply-to: Eli Zaretskii Message-id: <83d3dvwphm.fsf@gnu.org> References: <4E9C5112.9070705@mentor.com> X-IsSubscribed: yes 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 X-SW-Source: 2011-10/txt/msg00470.txt.bz2 > Date: Mon, 17 Oct 2011 09:00:18 -0700 > From: Stan Shebs > > With this patch, any arguments to tstart and tstop are recorded with the > trace run as free-form text, and reported as part of its status. Also, > there is a trace-user variable (which is free form text which is > typically constant and you can set in your .gdbinit), and set/show > variables for the start and stop notes as well. > > In addition, this patch adds start/stop timestamps and per-tracepoint > hit counts and traceframe usage. > > The patch still needs testsuite and NEWS, but considering the user > interface tinkering, it seemed like a good idea to solicit feedback > before committing to all the details. The documentation part is okay, but please add some overview of the notes, to serve as the glue to the description of the commands. As I read the patch, I was wondering whether there's only one note that can be set by either "tstart" or "tstop" (and then what happens if you give arguments to "tstop" when "tstart" already provided a note?), or two separate and independent notes (in which case why do we have "trace-stop-notes", but no "trace-start-notes"?). Instead of disclosing all this piecemeal and leaving the reader with a reasonable doubt, why not explain it once and for all? Btw, didn't the socializing parties ask you for a way to _add_ to an existing note, rather than overwriting it? The way you designed it, fixing a single typo requires retyping (or copy/pasting) the whole thing, which sounds inconvenient. Thanks.