Hi Daniel,
Thank you for your response.
> > +if (sizeof(if2.ifr_name) < sizeof(name)) {
> > +return -1;
> > +}
>
> If I'm not mistaken `sizeof(name)` where name is a const char*
> will always be 8 (on 64 bits architecture) - so this is probably
> not doing what you want.
I'm
Hi Roger,
Looks good to me.
Thanks,
David
On 9/07/2020 6:51 am, Roger Riggs wrote:
Please reveiw a change to increase the size of the Process Reaper stack
for debug builds.
This intermittent issue can be traced to the stack shadow page size
being larger
in debug builds than in release builds.
On 9/07/2020 7:24 am, Daniel D. Daugherty wrote:
Thanks for the fast review.
I was surprised that jcheck -r tip didn't flag that, but... :-)
jcheck doesn't check copyright lines or file headers.
Cheers,
David
Dan
On 7/8/20 5:23 PM, igor.ignat...@oracle.com wrote:
LGTM, thanks for fixing,
Hi Koichi,
If you don't mind I'd prefer to keep the simple approach
that will fix the warning without changing the behavior
of the function.
memset((char *)&if2, 0, sizeof(if2));
- strncpy(if2.ifr_name, name, sizeof(if2.ifr_name) - 1);
+ strncpy(if2.ifr_name, name, sizeof(if2
Hi,
I raised the debugDelta to 16k and so far (12hrs) have not seen any
failures.
I'll push later today with 16k unless I hear any concerns.
Webrev:
http://cr.openjdk.java.net/~rriggs/webrev-process-stackoverflow-8217475-1/
Thanks, Roger
On 7/9/20 4:54 AM, David Holmes wrote:
Hi Roger,
Thank you Naoto! The patch was pushed into jdk/jdk15.
Thanks,
Leo
On 7/9/20 3:53 AM, naoto.s...@oracle.com wrote:
Hi Leo,
Looks good to me.
Naoto
On 7/8/20 10:58 AM, li.ji...@oracle.com wrote:
Hi,
Pls review the l10n resource files update for JDK 15 msg drop 10.
Bug:
https://bugs.openjdk.
Hello Sean, Daniel,
Thank you for review
I’ve added “com.sun.jndi.ldap.tls.cbtype” property into the module-info file
and updated synchronisation using CompletableFuture.
Also, I have added new test cases : successful and unsuccessful TLS handshake,
synchronous and asynchronous TLS handshake.
New
Hi Alexander,
Agreed with your suggestion. Updated review available at [1].
- Alexey
[1] http://cr.openjdk.java.net/~asemenyuk/8247229/webrev.01/webrev.01/
On 7/9/2020 12:21 AM, alexander.matv...@oracle.com wrote:
Hi Alexey,
Looks good. Only suggestion is to move CmdlineExecutor to Executor
looks good.
/Andy
On 7/9/2020 3:49 PM, Alexey Semenyuk wrote:
Hi Alexander,
Agreed with your suggestion. Updated review available at [1].
- Alexey
[1] http://cr.openjdk.java.net/~asemenyuk/8247229/webrev.01/webrev.01/
On 7/9/2020 12:21 AM, alexander.matv...@oracle.com wrote:
Hi Alexey,
Lo
Hi Vipin,
Apologies for the delay.
After looking at the bug, which is over 14 years old, the SCCS history of
Attributes.java, I am reluctant to suggest we move forward with your proposed
change.
The key for an Attributes map entry should be an Attributes.Name object (see
Attributes::put).
Hi,
Please review a patch to remove an illegal access to java.lang by the
SafeThread. Note that InnocuousThread meets all the needs for SafeThread
except that it also revokes permissions that the xml parser requires.
This patch uses the new constructor added to Thread in JDK 9 that do not
inh
Hi Alexey,
Looks good.
Thanks,
Alexander
On 7/9/20 12:49 PM, Alexey Semenyuk wrote:
Hi Alexander,
Agreed with your suggestion. Updated review available at [1].
- Alexey
[1] http://cr.openjdk.java.net/~asemenyuk/8247229/webrev.01/webrev.01/
On 7/9/2020 12:21 AM, alexander.matv...@oracle.com
12 matches
Mail list logo