Hi Phil,
The changes look fine. (Not a "Reviewer").
Thanks for doing the backport,
Roger
On 2/11/2014 6:42 PM, Phil Race wrote:
Here's a JDk8u webrev : -http://cr.openjdk.java.net/~prr/8031737.8u/
-phil.
On 2/11/14 2:28 PM, Phil Race wrote:
So since hg export/import doesn't apply cleanly and the dependency
chain seems, long and in order to have some consistency across the
releases,
I think I should prepare a webrev which essentially backports 8031737
including its small changes to Version.c, if only because otherwise
I'd have to have a new bug ID that would not be forwarded ported
(one source of confusion) or even worse re-use 8031737 but not fully
implement it
Agreed ?
-phil.
On 2/11/2014 2:20 PM, roger riggs wrote:
Hi Phil,
On 2/11/2014 5:09 PM, Phil Race wrote:
Are we talking about the same changesets ?
a09982d91fab/8030993 has no change to the macros
right (I didn't think this was topic of this conversation)
fb89dc4fe8da/8031737 is the one that reimplemented the macros
and is the version I'd want. Its the last 'edit' of those macros in
that file.
yes,
c58c6b0fbe34/8030875 is the original addition of these :-
Yes.
Roger
...
changeset: 9229:fb89dc4fe8da
user: rriggs
date: Mon Feb 03 16:58:02 2014 -0500
summary: 8031737: CHECK_NULL and CHECK_EXCEPTION macros cleanup
changeset: 9051:c58c6b0fbe34
user: rriggs
date: Fri Jan 10 10:45:56 2014 -0500
summary: 8030875: Macros for checking and returning on exceptions
...
-phil.
On 2/11/14 1:48 PM, roger riggs wrote:
Hi Phil,
The later changeset picked up the recommended style of
implementing the macros
but I don't think it was substantive. You can probably do without
it.
Version.c had some changes in a different changeset to address
the omission of checking for exceptions after some JNI calls.
Roger
On 2/11/2014 4:39 PM, Phil Race wrote:
Roger,
That later one seems to be using the macros. I don't see any
update to the macros.
So I'm not sure why I'm need it .. since I'm not using those
calls and neither
are the macros.
-phil.
On 2/11/14 12:28 PM, roger riggs wrote:
Hi Phil,
Yes, it ended up in two change sets in jdk 9, you should take
both to be up to date.
changeset: 9245:a09982d91fab
user: rriggs
date: Wed Feb 05 10:59:53 2014 -0500
files: src/share/native/common/jni_util.c
description:
8030993: Check jdk/src/share/native/common/jni_util.c for JNI
pending exceptions
changeset: 9229:fb89dc4fe8da
date: Mon Feb 03 16:58:02 2014 -0500
files: src/share/native/common/jni_util.h
src/share/native/sun/misc/Version.c
interrupted!
description:
8031737: CHECK_NULL and CHECK_EXCEPTION macros cleanup
Thanks, Roger
On 2/11/2014 2:57 PM, Phil Race wrote:
Roger,
Yes, I can do that.
I see here
http://cr.openjdk.java.net/~rriggs/webrev-check-cleanup-8031737/ that
1) There was a previous version of these macros.
Looks like no need to worry about that I just need the latest
version.
2) There was also a change to Version.c. I can include that if
you think it
appropriate .. or omit it if you think its not essential.
-phil.
On 2/11/2014 11:14 AM, roger riggs wrote:
Hi Phil,
I see your point, there is nothing in the changes unique to 9.
Do you want to take care of the back point?
Roger
On 2/11/2014 2:04 PM, Phil Race wrote:
Roger,
Why not JDK 8u ? I've got a lot of changes that utilise
these that will
backport cleanly to JDK 8u only if 8u includes these macros.
And since
the changes are all over the place I don't fancy copy/pasting
them
everywhere. I suspect I am not the only one who would like
these in 8u ..
-phil.
On 02/03/2014 01:48 PM, roger riggs wrote:
Hi Lance,
The convenience macros are only intended for JDK 9.
Roger
On 2/1/2014 1:58 PM, Lance @ Oracle wrote:
Looks fine
Which releases are you think of including this in if any
besides 9?
<http://oracle.com/us/design/oracle-email-sig-198324.gif>Lance
Andersen| Principal Member of Technical Staff |
+1.781.442.2037 <tel:+1.781.442.2037>
Oracle Java Engineering
1 Network Drive <x-apple-data-detectors://34/0>
Burlington, MA 01803 <x-apple-data-detectors://34/0>
lance.ander...@oracle.com <mailto:lance.ander...@oracle.com>
Sent from my iPad
On Feb 1, 2014, at 1:03 PM, roger riggs
<roger.ri...@oracle.com <mailto:roger.ri...@oracle.com>>
wrote: