From: Elena Zannoni <ezannoni@redhat.com>
To: Daniel Jacobowitz <drow@mvista.com>
Cc: gdb-patches@sources.redhat.com
Subject: Re: [PATCH] remove hp-[p]symtab-read.c & hpread.c
Date: Mon, 21 Jan 2002 15:13:00 -0000 [thread overview]
Message-ID: <15436.38303.470579.895742@localhost.cygnus.com> (raw)
In-Reply-To: <20020121180140.A15709@nevyn.them.org>
Daniel Jacobowitz writes:
> On Mon, Jan 21, 2002 at 05:04:34PM -0500, Elena Zannoni wrote:
> >
> > This is something that I have wanted to do for a long time.
> > I finally got around to it.
> >
> > This patch merges hp-psymtab-read.c, hp-symtab-read.c and hpread.h
> > into hpread.c.
> >
> > hpread.c hasn't been in use since the above 3 files were introduced
> > with the HP merge.
> >
> > This is just a mechanical merge of the 3 files into one. I haven't
> > changed anything. Warnings fixes and minor cleanups will come in a
> > second patch.
> >
> > I am not going to post the diff for hpread.c, since it would be longer
> > than the file itself. I include the whole file instead. The file is
> > huge, but I see that the final size of this message is smaller than
> > Andrew's documentaion patch, so I don't feel too guilty. :-)
>
> As I read this, the previous contents of hpread.c are being scrapped in
> favor of this new merged beast. Do I understand correctly?
>
yes, hpread.c is overwritten by the merge of the 3 other files.
> And thanks - one less symbol reader to grep through every time I do something
> that affects them!
>
Yes, I see people fixing hpread.c and I feel awful.
I'll wait a couple of days and commit it.
Elena
> --
> Daniel Jacobowitz Carnegie Mellon University
> MontaVista Software Debian GNU/Linux Developer
next prev parent reply other threads:[~2002-01-21 23:13 UTC|newest]
Thread overview: 4+ messages / expand[flat|nested] mbox.gz Atom feed top
2002-01-21 14:52 Elena Zannoni
2002-01-21 15:01 ` Daniel Jacobowitz
2002-01-21 15:13 ` Elena Zannoni [this message]
2002-01-23 19:44 ` Elena Zannoni
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=15436.38303.470579.895742@localhost.cygnus.com \
--to=ezannoni@redhat.com \
--cc=drow@mvista.com \
--cc=gdb-patches@sources.redhat.com \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox