Sounds good Joe,
I will push those soon.
Thanks
Alejandro
On 6/19/2015 3:56 PM, huizhe wang wrote:
Hi Alejandro,
Alan's right about the JAXP version-check code. But since this is
urgently needed, I agree you can push the change as is. I'll create a
separate bug to re-examine version related
Hi Alejandro,
Alan's right about the JAXP version-check code. But since this is
urgently needed, I agree you can push the change as is. I'll create a
separate bug to re-examine version related code in jaxp. As Alan pointed
out, it would be some clean-up.
Thanks,
Joe
On 6/19/2015 9:53 AM, Al
There's a whole cottage industry of web sites explaining the do {...} while
(0) thing, e.g.
http://www.bruceblinn.com/linuxinfo/DoWhile.html
On Fri, Jun 19, 2015 at 11:47 AM, Ivan Gerasimov
wrote:
>
>
> On 19.06.2015 15:11, Alan Bateman wrote:
>
>> On 18/06/2015 21:40, Xueming Shen wrote:
>>
>>>
On 19.06.2015 15:11, Alan Bateman wrote:
On 18/06/2015 21:40, Xueming Shen wrote:
Hi,
Please help review the change to resolve the disabled "parentheses"
warnings for libzip
issue: https://bugs.openjdk.java.net/browse/JDK-8074819
webrev: http://cr.openjdk.java.net/~sherman/8074819
Looks ok
Hi Ivan,
Both fixed,
Thanks, Roger
On 6/19/2015 1:33 PM, Ivan Gerasimov wrote:
Thanks! I like it better now :)
A couple of minor nits:
- please use VerQueryValueW() for consistency,
- missing space after if in '507 if(is_workstation'.
Otherwise looks fine to me.
Sincerely yours,
Ivan
Thanks! I like it better now :)
A couple of minor nits:
- please use VerQueryValueW() for consistency,
- missing space after if in '507 if(is_workstation'.
Otherwise looks fine to me.
Sincerely yours,
Ivan
On 19.06.2015 16:38, Roger Riggs wrote:
Hi Ivan,
Thanks for the review and recom
Hi Alan, just to you.
didn't hear back from you, so I'll assume you are fine with
the corrections. We want to get this into further testing
so I'm going to push the changes
cheers
Alejandro
On 6/18/2015 4:56 PM, Alejandro E Murillo wrote:
Thanks Alan,
see below
On 6/18/2015 7:41 AM, Alan Bat
On Jun 19, 2015, at 2:04 PM, Paul Sandoz wrote:
> There are two main aspects to the webrev:
>
> 1) remove the back propagation logic, which simplifies the preparation of the
> pipeline for parallel execution and also cleans up a certain aspect of
> toArray optimisation that i had been meaning g
it appears the "warning-switch-off" change might trigger a timzezone.c
warning-failure on mac ... will take another round ...
On 6/19/15 5:11 AM, Alan Bateman wrote:
On 18/06/2015 21:40, Xueming Shen wrote:
Hi,
Please help review the change to resolve the disabled "parentheses"
warnings for
Please review this update to ProcessHandle to specify the equals method
and that (JDK provided) instances have value-base semantics.
This update to the spec is needed to support pending work to uniquely
identify processes (8078099) using the process start time if available
from the operating syst
Hi Ivan,
Thanks for the review and recommendation.
The webrev is updated in place to use WCHAR.
http://cr.openjdk.java.net/~rriggs/webrev-win-ver-8066504/
On 6/18/2015 2:07 PM, Ivan Gerasimov wrote:
Hi Roger!
On 17.06.2015 17:27, Roger Riggs wrote:
I updated the webrev to remove the Win
On 18/06/2015 21:40, Xueming Shen wrote:
Hi,
Please help review the change to resolve the disabled "parentheses"
warnings for libzip
issue: https://bugs.openjdk.java.net/browse/JDK-8074819
webrev: http://cr.openjdk.java.net/~sherman/8074819
Looks okay to me, although I have to admit that pare
Hi,
http://cr.openjdk.java.net/~psandoz/jdk9/JDK-8129120-flag-back-propagation/webrev/
I am a little red in the face (of the embarrassed kind) about this fix. We
found an issue reported on stack overflow whereby a stream can report incorrect
results. I may be wrong but this might be the first r
13 matches
Mail list logo