This is the mail archive of the
gdb-patches@sources.redhat.com
mailing list for the GDB project.
Re: coffread.c extension for DLLs without debugging symbols
On Sun, Jan 05, 2003 at 12:18:26PM -0500, Christopher Faylor wrote:
> On Sun, Jan 05, 2003 at 03:44:44PM +0100, Mark Kettenis wrote:
> >Christopher Faylor <cgf@redhat.com> writes:
> >
> >>>>2) I wonder if this code should be ifdef'ed somehow for Windows since
> >>>>it will add extra code for no gain on every COFF platform. Of course,
> >>>>how many of those are there out there? Maybe this isn't a huge issue
> >>>>after all.
> >>>
> >>>I wondered about that myself. However, wouldn't that suggest putting
> >>>the bulk of the code somewhere like win32-nat.c?
> >>
> >> That would be fine with me (especially since I can approve that part of
> >> gdb). So, you'd need some kind of hooks in coffread.c to handle this.
> >> I guess we should wait for the coffread maintainer to offer an opinion
> >> before you go to this effort, though.
> >
> >Well, I assume this code would be usefull when cross-debugging too.
> >If so, a *-nat.c file would be the wrong place to add it. You could
> >create a win32-tdep.c file though.
>
> Cross debugging to a windows box? Does anyone actually do that? I know
> it is theoretically possible with, *cough* rda *cough* but I wasn't aware
> of anyone actually doing this.
>
> Regardless, win32-tdep.c would be a more logically correct place to put
> it.
In any case, it's nicer to have it in the tdep file; it's a step
towards letting people who make large mechanical changes verify that it
still compiles.
--
Daniel Jacobowitz
MontaVista Software Debian GNU/Linux Developer