On Fri, Mar 28, 2014 at 5:57 AM, Victor Polischuk vict...@ukr.net wrote:
Ulf,
I think that point leading style is something which can be easily
mistreat. If we complicate the example:
String mySub =
myVeryLongNamedString.substring(.indexOf(C),.indexOf(Q));
to something like:
Hi Miran,
I have skimmed the webrev, and it looks ok to me.
You seem to have some files with changes for 8029237, which is already
integrated into jdk9/dev. Can you remove this, then I will sponsor the
change for you.
-Chris.
On 07/03/14 16:21, Miroslav Kos wrote:
Hi,
there is a bulk
Igor, thanks for improving the tests on method handles!
After an offline conversation w/ VladimirI, I've done several changes:
- placed all classes into one file (but I don't like it)
You can convert TestCase TestFactory to inner classes and move
ThrowMode into TestCase since it's usage is
On 03/28/2014 05:57 AM, Victor Polischuk wrote:
Ulf,
I think that point leading style is something which can be easily mistreat. If
we complicate the example:
String mySub = myVeryLongNamedString.substring(.indexOf(C),.indexOf(Q));
to something like:
String mySub =
Hello,
Could you please review the fix for the following bug:
https://bugs.openjdk.java.net/browse/JDK-8038493
Webrev corresponding:
http://cr.openjdk.java.net/~yan/8038493/webrev.00/
Just a minor cleanup of javadoc to avoid tidy warnings; no other code
affected.
Thanks.
Regards,
Alexander
Looks fine
Lance Andersen| Principal Member of Technical Staff | +1.781.442.2037
Oracle Java Engineering
1 Network Drive
Burlington, MA 01803
lance.ander...@oracle.com
Sent from my iPad
On Mar 28, 2014, at 6:58 AM, alexander stepanov
alexander.v.stepa...@oracle.com wrote:
Hello,
Could
On 28/03/14 11:16, Lance @ Oracle wrote:
Looks fine
Looks good to me too.
-Chris.
Lance Andersen| Principal Member of Technical Staff | +1.781.442.2037
Oracle Java Engineering
1 Network Drive
Burlington, MA 01803
lance.ander...@oracle.com
Sent from my iPad
On Mar 28, 2014, at 6:58 AM,
Vladimir, thanks for review, please my comments inline.
On 03/28/2014 01:29 PM, Vladimir Ivanov wrote:
Igor, thanks for improving the tests on method handles!
After an offline conversation w/ VladimirI, I've done several changes:
- placed all classes into one file (but I don't like it)
You
With the change to stringbuilder I am ok with it
Best
Lance
Lance Andersen| Principal Member of Technical Staff | +1.781.442.2037
Oracle Java Engineering
1 Network Drive
Burlington, MA 01803
lance.ander...@oracle.com
Sent from my iPad
On Mar 26, 2014, at 1:44 PM, Aleksej Efimov
Hi everyone,
could you please review my change for JDK-8037781:
http://cr.openjdk.java.net/~chegar/8037781/webrev.00/
After JDK-8035158 was fixed we freed our hands to remove these 4 classes:
1. sun.misc.RegexpPool,
2. sun.misc.RegexpTarget,
3. sun.misc.Regexp,
Volker,
I think we should check the length of passed filename and
throw an exception if filename is too long.
Otherwise we can end up opening wrong file with possibly not expected
permissions.
-Dmitry
On 2014-03-27 22:08, Volker Simonis wrote:
Hi,
a security audit for the PPC64/AIX port
Hi Dimitry,
sorry, but I already pushed the change.
However I don't think this is a big problem because:
- if the file does not exist, connect will fail with No such file or directory
- if the file exists by chance (but is not a socket) connect will fail
with Connection refused
- if the file
Vladimir,
as we'd arranged, I moved ThrowMode into TestCase and give my word that
I'll
remove TestCase.assertEQ and enhance Asserts.assertEquals for arrays
add the test cases from TestCatchException to 'MANDATORY_TEST_CASES' in
CatchExceptionTest and remove TestCatchException
in JDK9 TF.
Hi Chris,
I regenerated webrev against current hg tip - the copyright changes
should be ok now.
patch file is attached, the new webrev is uploading just now:
http://cr.openjdk.java.net/~mkos/8036030/jaxws.01/
JBS: https://bugs.openjdk.java.net/browse/JDK-8036030
Let me know if it is ok now.
Hi,
I really do not know why some proposals are restored back from the
grave, without answering for questions that was already made:
http://mail.openjdk.java.net/pipermail/jdk8-dev/2011-December/000382.html
2014-03-28 12:37 GMT+01:00 Ulf Zibis ulf.zi...@cosoco.de:
Am 28.03.2014 11:05, schrieb
On 3/24/14 3:11 PM, Anthony Petrov wrote:
How does IS_JNI_TRUE sound?
It looks good. If nobody objects, the final version will be:
#define IS_NULL(obj) ((obj) == NULL)
#define JNU_IsNull(env,obj) ((obj) == NULL)
+#define IS_JNI_TRUE(obj) ((obj) ? JNI_TRUE : JNI_FALSE)
without additional cast.
On Mar 28, 2014, at 10:52 AM, Marek Kozieł develop4l...@gmail.com wrote:
Hi,
I really do not know why some proposals are restored back from the
grave, without answering for questions that was already made:
http://mail.openjdk.java.net/pipermail/jdk8-dev/2011-December/000382.html
Problem
17 matches
Mail list logo