Exactly! On Wed, May 27, 2015 at 3:07 PM, Alexander Scherbatiy <[email protected]> wrote: > On 5/25/2015 6:56 PM, Sergey Bylokhov wrote: >> >> Hello. >> Please review the fix for jdk9. >> >> This is similar to the issue described here: >> http://mail.openjdk.java.net/pipermail/beans-dev/2015-May/000138.html >> >> The problem is that PropertyDescriptor.getValue() for "enumerationValues" >> return null, when default enumerationValues is assumed or an empty array was >> provided by the user. >> There are a different ways to fix the issue but I select this, because of >> assumption that @BeanProperty(enumerationValues = {}) should not return >> null, and we have no way to detect: it was set by the user or by default. >> >> Bug: https://bugs.openjdk.java.net/browse/JDK-8079084 >> Webrev can be found at: http://cr.openjdk.java.net/~serb/8079084/webrev.00 > > 128 if (0 <= values.length) { > > It looks like the condition is always true and the 'if' statement can be > omitted. > > Thanks, > Alexandr. >
-- Best regards, Sergey A. Malenkov
