Re: [9] Review Request: 8146313 The java.beans.Statement.invoke() method handles 3-arg Class.forName incorrectly

2016-04-28 Thread Sergey Bylokhov
On 27.04.16 10:12, Semyon Sadetsky wrote: Yes it does not work for primitives, it not treated as some special method(like 1-args), it call 3-args Class.forName(which do not support the primitives) directly like any other methods. What will happen if the 3-args is called with a primitive? Same

RFR 8155021: [TEST] create one more inheritance test for @BeanProperty

2016-04-28 Thread Alexander Stepanov
Hello, Could you please review the following fix http://cr.openjdk.java.net/~avstepan/8155021/webrev.00/ for https://bugs.openjdk.java.net/browse/JDK-8155021 Just a single test added + minor updates for a couple of other tests. Thanks, Alexander

Re: RFR 8155021: [TEST] create one more inheritance test for @BeanProperty

2016-04-28 Thread Sergey Bylokhov
Cannot check that all possible combinations are covered, I assume it is, so looks fine. On 28.04.16 17:21, Alexander Stepanov wrote: Hello, Could you please review the following fix http://cr.openjdk.java.net/~avstepan/8155021/webrev.00/ for https://bugs.openjdk.java.net/browse/JDK-8155021 Jus

Re: RFR 8155021: [TEST] create one more inheritance test for @BeanProperty

2016-04-28 Thread Alexander Stepanov
Thanks! On 4/28/2016 7:25 PM, Sergey Bylokhov wrote: Cannot check that all possible combinations are covered, I assume it is, so looks fine. On 28.04.16 17:21, Alexander Stepanov wrote: Hello, Could you please review the following fix http://cr.openjdk.java.net/~avstepan/8155021/webrev.00/ f