From owner-freebsd-hackers Tue May 9 06:45:02 1995 Return-Path: hackers-owner Received: (from majordom@localhost) by freefall.cdrom.com (8.6.10/8.6.6) id GAA14299 for hackers-outgoing; Tue, 9 May 1995 06:45:02 -0700 Received: from inet-gw-1.pa.dec.com (inet-gw-1.pa.dec.com [16.1.0.22]) by freefall.cdrom.com (8.6.10/8.6.6) with SMTP id GAA14285 for ; Tue, 9 May 1995 06:44:50 -0700 Received: from rks32.pcs.dec.com by inet-gw-1.pa.dec.com (5.65/24Feb95) id AA21455; Tue, 9 May 95 06:36:04 -0700 Received: by rks32.pcs.dec.com (Smail3.1.27.1 #16) id m0s8pOC-0005OqC; Tue, 9 May 95 15:31 MSZ Message-Id: To: rgrimes%gndrsh.aac.dev.com@inet-gw-1.pa.dec.com Cc: hackers%freebsd.org@inet-gw-1.pa.dec.com In-Reply-To: Message from "Rodney W. Grimes" of Tue, 09 May 95 05:57:03 MST. Reply-To: gj@FreeBSD.org Subject: Re: This one looks very very suspecious to me.... Date: Tue, 09 May 95 13:31:52 GMT From: "gj%pcs.dec.com@inet-gw-1.pa.dec.com" Sender: hackers-owner@FreeBSD.org Precedence: bulk > While going through all the -Wcomment errors I have found one that > look very suspecious: > > /usr/src/gnu/usr.bin/gdb/gdb/freebsd-nat.c:489: warning: `/*' within comment > > This is the snip of code around that: > > static CORE_ADDR > kvtophys (fd, addr) > CORE_ADDR addr; > { > CORE_ADDR v; > struct pte pte; > static CORE_ADDR PTD = -1; > CORE_ADDR current_ptd; > > /*if (devmem && kfd > 0) > return (addr); > /* > * If we're looking at the kernel stack, > * munge the address to refer to the user space mapping instead; > * that way we get the requested process's kstack, not the running one. > */ > > > Now, did some one really mean to comment out the if (devmem) or is this > an error???? Gdb experts please step forward and tell me to either > close the comment after the if, or removed the comment start before it. EEEK! Boy, did I f*ck that up ! This should definitely be commented out, or even better, totally removed. This is a left-over from when I ported gdb many moons ago and was trying out various things. Seems to me there may be a few more corpses like this laying around in gdb, but as long as they don't start to stink....:-) Gary J.