From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (qmail 21569 invoked by alias); 18 Oct 2012 09:49:16 -0000 Received: (qmail 21560 invoked by uid 22791); 18 Oct 2012 09:49:15 -0000 X-SWARE-Spam-Status: No, hits=-5.0 required=5.0 tests=AWL,BAYES_00,KHOP_RCVD_UNTRUST,KHOP_THREADED,RCVD_IN_HOSTKARMA_W,RCVD_IN_HOSTKARMA_WL,RP_MATCHES_RCVD X-Spam-Check-By: sourceware.org Received: from mms1.broadcom.com (HELO mms1.broadcom.com) (216.31.210.17) by sourceware.org (qpsmtpd/0.43rc1) with ESMTP; Thu, 18 Oct 2012 09:49:10 +0000 Received: from [10.9.200.133] by mms1.broadcom.com with ESMTP (Broadcom SMTP Relay (Email Firewall v6.5)); Thu, 18 Oct 2012 02:47:53 -0700 X-Server-Uuid: 06151B78-6688-425E-9DE2-57CB27892261 Received: from mail-irva-13.broadcom.com (10.11.16.103) by IRVEXCHHUB02.corp.ad.broadcom.com (10.9.200.133) with Microsoft SMTP Server id 8.2.247.2; Thu, 18 Oct 2012 02:48:26 -0700 Received: from [10.177.72.92] (unknown [10.177.72.92]) by mail-irva-13.broadcom.com (Postfix) with ESMTP id 4407A40FF6; Thu, 18 Oct 2012 02:48:54 -0700 (PDT) Message-ID: <507FD088.4030101@broadcom.com> Date: Thu, 18 Oct 2012 09:49:00 -0000 From: "Andrew Burgess" User-Agent: Mozilla/5.0 (Windows NT 6.1; rv:16.0) Gecko/20121010 Thunderbird/16.0.1 MIME-Version: 1.0 To: "Jan Kratochvil" cc: "Pedro Alves" , "gdb-patches@sourceware.org" Subject: Re: [PATCH] Display full file path in MI style disassembly listing References: <506DB4B8.5030001@broadcom.com> <20121005124353.GA22997@host2.jankratochvil.net> <507EF554.4080404@redhat.com> <20121018064825.GA15668@host2.jankratochvil.net> In-Reply-To: <20121018064825.GA15668@host2.jankratochvil.net> Content-Type: text/plain; charset=iso-8859-1 Content-Transfer-Encoding: 7bit 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: 2012-10/txt/msg00319.txt.bz2 On 18/10/2012 7:48 AM, Jan Kratochvil wrote: > On Wed, 17 Oct 2012 20:13:40 +0200, Pedro Alves wrote: >> On 10/05/2012 01:43 PM, Jan Kratochvil wrote: >>> On Thu, 04 Oct 2012 18:09:28 +0200, Andrew Burgess wrote: >>>> The patch below tries to use the fullname when it can, and falls back to the >>>> shorter name if it can't figure out the full name. >>> >>> FYI this is a variant of not yet checked in more featured pathset: >>> [patch] GDB 7.2: new feature for "backtrace" that cuts path to file (remain filename) >> >> I'm not sure I understand the relation between the patches. I agree that though both patches relate to display of paths they are not directly related. I took a look at the original patch. I it changes one place (in backtrace) where we display symtab->filename. There are several places where we display symtab->filename, including the one I changed. Either, (a) we plan to merge the original patch almost as is, in which case it only changes the backtrace code, leaving my patch free to be applied, or (b) we plan to extend the original patch to cover more/all of the places we display symtab->filename, in which case, we'd have to consider each of those places in turn and make a suitable change. If (a) then my patch could go in, if (b) then it doesn't feel like my patch changes things that much, I certainly don't add any new functions or anything like that, that would make doing task (b) any harder later on. >> I think we should >> output a "fullname" field for MI, like we do for breakpoints. I would be happy to take this approach as a compromise. > Probably, it is true I do not know much how MI frontends display the data. The frontend (eclipse in this case) wants to open the file in order to display the contents. Using just symtab->filename is not always specific enough to identify a unique file. I originally switched to only displaying the full path as this code is only used for MI type output or when we fail to open the file, but I can see that displaying the fullpath in addition might be cleaner. Jan, if I post a patch that adds a fullpath would you be happy with that as a solution? Thanks, Andrew > > > Thanks, > Jan > > >