Fix looks good to me.
>
>
>
> On 12/9/2016 8:18 PM, Alexandr Scherbatiy wrote:
>> On 12/6/2016 5:52 PM, Prasanta Sadhukhan wrote:
>>> On 12/6/2016 8:17 PM, Alexandr Scherbatiy wrote:
On 12/1/2016 9:16 AM, Prasanta Sadhukhan wrote:
> On 11/30/2016 2:53 PM, Prasanta Sadhukhan wrote:
>
Hello,
Could you review the updated fix:
http://cr.openjdk.java.net/~alexsch/8133919/webrev.01
The test is converted to the automated.
Thanks,
Alexandr.
On 12/15/2016 1:24 PM, Sergey Bylokhov wrote:
Could you review the fix:
bug: https://bugs.openjdk.java.net/browse/JDK-8133919
webrev:
Bug: https://bugs.openjdk.java.net/browse/JDK-8171363
Webrev: http://cr.openjdk.java.net/~prr/8171363/
this is mostly about jtreg syntax.
A few tests fail due to the updated module system spec.
jtreg needs to allow them to access package private fields.
":open" is added on the tests that do this
> On Dec 16, 2016, at 3:27 PM, Phil Race wrote:
>
> Bug: https://bugs.openjdk.java.net/browse/JDK-8171363
> Webrev: http://cr.openjdk.java.net/~prr/8171363/
+1
Mandy