From: Andrew Haley <aph@redhat.com>
To: Jakub Jelinek <jakub@redhat.com>
Cc: Java Patch List <java-patches@gcc.gnu.org>
Subject: Re: Split gnu/javax/swing/text/html/parser/HTML_401F.java (Was [vta] Add chains from referenced VALUEs to DVs that reference them)
Date: Thu, 02 Jul 2009 08:15:00 -0000 [thread overview]
Message-ID: <4A4C6C7A.5000003@redhat.com> (raw)
In-Reply-To: <20090701192115.GJ4462@tyan-ft48-01.lab.bos.redhat.com>
Jakub Jelinek wrote:
> On Wed, Jul 01, 2009 at 01:37:23PM +0100, Andrew Haley wrote:
>> It's not an ABI change. This patch is OK iff accompanied by a comment
>> in the code that explains the problem.
>
> Thanks, here is what I've committed.
> In doing so, I had to --enable-java-maintainer-mode to update the class
> file, which in addition to changing the HTML_401F* files changed a lot of
> headers and a handful of classes as well, see attached second patch.
> But at least the header changes look wrong, see e.g.
> - namespace namespace$
> + namespace namespace
> changes in the headers, I guess related to recent keyword handling changes.
> Dunno if that is because I had a wrong gcjh or if there is some recent issue in the
> tools.
I changed gcjh two days ago, so it is very likely you don't have the new
version.
I can't see any attachments.
Andrew.
next prev parent reply other threads:[~2009-07-02 8:15 UTC|newest]
Thread overview: 11+ messages / expand[flat|nested] mbox.gz Atom feed top
[not found] <20090630124346.GX4462@tyan-ft48-01.lab.bos.redhat.com>
[not found] ` <alpine.LSU.1.99.0906301619520.29901@acrux.dbai.tuwien.ac.at>
[not found] ` <4A4A2623.9010609@redhat.com>
[not found] ` <20090630152911.GA4462@tyan-ft48-01.lab.bos.redhat.com>
2009-07-01 12:37 ` Andrew Haley
2009-07-01 19:21 ` Jakub Jelinek
2009-07-02 8:15 ` Andrew Haley [this message]
2009-07-12 20:27 ` Split gnu/javax/swing/text/html/parser/HTML_401F.java Gerald Pfeifer
2009-07-12 20:36 ` Jakub Jelinek
2009-07-12 21:00 ` Gerald Pfeifer
2009-07-14 16:28 ` [cp-patches] " Audrius Meskauskas
2009-07-27 22:55 ` Andrew John Hughes
2009-07-29 7:06 ` Andrew Haley
2009-08-02 21:44 ` Audrius Meskauskas
2009-07-13 9:31 ` Andrew Haley
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=4A4C6C7A.5000003@redhat.com \
--to=aph@redhat.com \
--cc=jakub@redhat.com \
--cc=java-patches@gcc.gnu.org \
/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;
as well as URLs for read-only IMAP folder(s) and NNTP newsgroup(s).