Date: Tue, 9 May 95 11:30:58 MDT From: terry@cs.weber.edu (Terry Lambert) To: gj@FreeBSD.org Cc: rgrimes%gndrsh.aac.dev.com@inet-gw-1.pa.dec.com, hackers%freebsd.org@inet-gw-1.pa.dec.com Subject: Re: This one looks very very suspecious to me.... Message-ID: <9505091730.AA06995@cs.weber.edu> In-Reply-To: <m0s8pOC-0005OqC@rks32.pcs.dec.com> from "gj%pcs.dec.com@inet-gw-1.pa.dec.com" at May 9, 95 01:31:52 pm
next in thread | previous in thread | raw e-mail | index | archive | help
> > /*if (devmem && kfd > 0) > > return (addr); > > /* > > 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....:-) Standard practice is "#ifdef OMIT" or "#ifdef NOTDEF" or "#ifdef GARYJ". Since most compilers generate warnings but work anyway with tags following #endif, you can also make notes to yourself using warnings (assuming your code is otherwise warning free) by putting tags after "#else" or "#endif" ...for instance: #ifdef OMIT #endif /* OMIT*/ FIX_LATER Terry Lambert terry@cs.weber.edu --- Any opinions in this posting are my own and not those of my present or previous employers.
Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?9505091730.AA06995>