From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (qmail 51233 invoked by alias); 17 Apr 2018 15:18:03 -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 50282 invoked by uid 89); 17 Apr 2018 15:18:03 -0000 Authentication-Results: sourceware.org; auth=none X-Virus-Found: No X-Spam-SWARE-Status: No, score=-3.2 required=5.0 tests=AWL,BAYES_00 autolearn=ham version=3.3.2 spammy= X-HELO: mx1.redhat.com Received: from mx3-rdu2.redhat.com (HELO mx1.redhat.com) (66.187.233.73) by sourceware.org (qpsmtpd/0.93/v0.84-503-g423c35a) with ESMTP; Tue, 17 Apr 2018 15:18:01 +0000 Received: from smtp.corp.redhat.com (int-mx03.intmail.prod.int.rdu2.redhat.com [10.11.54.3]) (using TLSv1.2 with cipher AECDH-AES256-SHA (256/256 bits)) (No client certificate requested) by mx1.redhat.com (Postfix) with ESMTPS id 523454023150; Tue, 17 Apr 2018 15:18:00 +0000 (UTC) Received: from [127.0.0.1] (ovpn04.gateway.prod.ext.ams2.redhat.com [10.39.146.4]) by smtp.corp.redhat.com (Postfix) with ESMTP id D4D271102E28; Tue, 17 Apr 2018 15:17:59 +0000 (UTC) Subject: Re: [PATCH] Don't print symbol definition's line number in rbreak output To: Andreas Arnez References: Cc: gdb-patches@sourceware.org From: Pedro Alves Message-ID: <4eae39d3-c6ca-ef5b-4127-0ad29ee792c9@redhat.com> Date: Tue, 17 Apr 2018 15:18:00 -0000 User-Agent: Mozilla/5.0 (X11; Linux x86_64; rv:52.0) Gecko/20100101 Thunderbird/52.6.0 MIME-Version: 1.0 In-Reply-To: Content-Type: text/plain; charset=utf-8 Content-Transfer-Encoding: 7bit X-SW-Source: 2018-04/txt/msg00337.txt.bz2 On 04/17/2018 01:28 PM, Andreas Arnez wrote: > On Mon, Apr 16 2018, Pedro Alves wrote: > >> On 04/16/2018 07:44 PM, Andreas Arnez wrote: >>> This commit: >>> >>> b744723f57 -- Show line numbers in output for "info var/func/type" >>> >>> added the symbol definition's line number to the output of certain GDB >>> commands. It also changes the `rbreak' command's output, although it >>> shouldn't. This is fixed. >> >> Could you update this to include an example of before/after gdb >> output in the commit log? > > Sure. How about the updated commit message below? > Thanks. >> >> Is this a regression in 8.1? > > No, I just caused the regression myself on Friday with the commit above. > For some reason I had not noticed the impact on the `rbreak' command > before. Ahah, thanks, it's much clearer now. > No declaration line number was shown before. Instead, the function > declaration started at the first column. This old behavior is restored. IMHO it's always better to also paste the output after, so the reader doesn't have to imagine it from the description. > /* Helper function for symtab_symbol_info, this function uses > the data returned from search_symbols() to print information > - regarding the match to gdb_stdout. */ > + regarding the match to gdb_stdout. If LAST is not NULL, > + print file- and line number information for the symbol as > + well. Skip printing the filename if it matches LAST. */ Nit, I don't think use of suspense hyphen in this case is common in English: . I'd just write plain "file". The patch looks fine to me. However, I notice now that this doesn't tweak any existing testcase or add any new one. Was the problem caught by some existing testcase? Thanks, Pedro Alves