From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (qmail 24665 invoked by alias); 30 Mar 2010 17:05:31 -0000 Received: (qmail 24595 invoked by uid 22791); 30 Mar 2010 17:05:31 -0000 X-SWARE-Spam-Status: No, hits=-1.9 required=5.0 tests=BAYES_00,T_RP_MATCHES_RCVD X-Spam-Check-By: sourceware.org Received: from mail.codesourcery.com (HELO mail.codesourcery.com) (38.113.113.100) by sourceware.org (qpsmtpd/0.43rc1) with ESMTP; Tue, 30 Mar 2010 17:05:27 +0000 Received: (qmail 1649 invoked from network); 30 Mar 2010 17:05:25 -0000 Received: from unknown (HELO macbook-2.local) (stan@127.0.0.2) by mail.codesourcery.com with ESMTPA; 30 Mar 2010 17:05:25 -0000 Message-ID: <4BB22F4A.1030506@codesourcery.com> Date: Tue, 30 Mar 2010 17:05:00 -0000 From: Stan Shebs User-Agent: Thunderbird 2.0.0.24 (Macintosh/20100228) MIME-Version: 1.0 To: Pedro Alves CC: gdb-patches@sourceware.org, Stan Shebs Subject: Re: [PATCH] Whack some dead code References: <4BB144E7.1080800@codesourcery.com> <201003301203.22533.pedro@codesourcery.com> In-Reply-To: <201003301203.22533.pedro@codesourcery.com> Content-Type: text/plain; charset=UTF-8; format=flowed Content-Transfer-Encoding: 7bit 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: 2010-03/txt/msg01067.txt.bz2 Pedro Alves wrote: > On Tuesday 30 March 2010 01:25:11, Stan Shebs wrote: > >> A while back, Vladimir Prus noticed a chunk of dead code in >> trace_find_line_command; basically, if you don't have symbolic info, >> there is no plausible fallback for line numbers, you just have to give >> up. Committed to trunk. >> > > >> * tracepoint.c (trace_find_line_command): Remove dead code. >> > > Errr, what does "dead" mean exactly? It's certainly exercisable, just > like the comment in the code suggests: > > (top-gdb) help tfind line > Select a trace frame by source line. > Argument can be a line number (with optional source file), > a function name, or '*' followed by an address. > Default argument is 'the next source line that was traced'. > A better term is maybe "bogus code", and my explanation is misleading. If you look at the locals start_pc and end_pc, you see that the original code's sal.symtab == 0 case does not set them before they get passed to tfind_1 at the bottom. I suppose one could try to salvage the sal.pc != 0 case and treat it as a start_pc == end_pc situation; we'd would want to think how each case is supposed to be useful. It would be a little annoying if a typo in tfind takes you to an unexpected trace frame, which in turn could disable your display commands, etc. "info line" is not necessarily the right model, since as an info command it has no side effects and can make far-out interpretations. (Incidentally, I notice that "help info line" doesn't mention the raw address option. Didn't it used to??) Stan