Thank you, Alan. I will add this change into my fix and push it today.
Thanks!
-Dan
On 01/08/2014 01:24 AM, Alan Bateman wrote:
On 08/01/2014 00:50, Dan Xu wrote:
Hi All,
Thanks for your good review. I have dropped the change in
FileSystemPreferences.java , and created the new webrev
Hi Alan,
Btw, I am wondering whether I need pass a variable to see if the
returned char* is a copy or not, and basing on that to do the release.
For example,
jboolean isCopy;
const char *fname = JNU_GetStringPlatformChars(env, java_fname,
isCopy);
if (isCopy == JNI_TRUE)
On 08/01/2014 18:48, Dan Xu wrote:
Hi Alan,
Btw, I am wondering whether I need pass a variable to see if the
returned char* is a copy or not, and basing on that to do the release.
For example,
jboolean isCopy;
const char *fname = JNU_GetStringPlatformChars(env, java_fname, isCopy);
if
On 6 Jan 2014, at 22:29, Dan Xu dan...@oracle.com wrote:
Hi All,
Please review the simple fix for JNI pending exceptions in
FileSystemPreferences.c. Thanks!
Bug: https://bugs.openjdk.java.net/browse/JDK-8028726
Webrev: http://cr.openjdk.java.net/~dxu/8028726/webrev/
Looks good to me
On 06/01/2014 22:29, Dan Xu wrote:
Hi All,
Please review the simple fix for JNI pending exceptions in
FileSystemPreferences.c. Thanks!
Bug: https://bugs.openjdk.java.net/browse/JDK-8028726
Webrev: http://cr.openjdk.java.net/~dxu/8028726/webrev/
The update to FIleSystemPreferences.c looks
Hi All,
Thanks for your good review. I have dropped the change in
FileSystemPreferences.java , and created the new webrev which only
changes FileSystemPreferences.c. Please help review it. Thanks!
Webrev: http://cr.openjdk.java.net/~dxu/8028726/webrev.01/
When changing
Hi All,
Please review the simple fix for JNI pending exceptions in
FileSystemPreferences.c. Thanks!
Bug: https://bugs.openjdk.java.net/browse/JDK-8028726
Webrev: http://cr.openjdk.java.net/~dxu/8028726/webrev/
-Dan
Dan,
Looks OK, but line 914 which you did not change, notice the comments not sure
if that is common in this code but seemed a bit off to me:
914 //// If at first, you don't succeed...
On Jan 6, 2014, at 5:29 PM, Dan Xu wrote:
Hi All,
Please review the simple fix for JNI
Hi Lance,
Thanks for your review.
My understanding towards the for loop in lockFile() of
FileSystemPreferences.java is that it retries if anything fails. So I
don't touch L914 to keep its old logic un-changed. Please let me know if
you have some good suggestions. Thanks!
-Dan
On