The fix looks good for me.

  It may have sense to remove the obsolete commented code from the test.

  Thanks,
  Alexandr.

On 2/19/2014 6:18 PM, sergey malenkov wrote:
Hello,

Could you please review the following fix:
fix: http://cr.openjdk.java.net/~malenkov/8034085.9.0/
bug: https://bugs.openjdk.java.net/browse/JDK-8034085

The regression test checks for all combinations of getters and setters.
It contains several bean with names like Bean1010, where
   first digit is used for getter (1-exist, 0-not)
  second digit is used for setter (1-exist, 0-not)
   third digit is used for indexed getter (1-exist, 0-not)
  fourth digit is used for indexed setter (1-exist, 0-not)
Note, that a type of indexed property does not correspond to a property type and must be ignored.

Thanks,
SAM

Reply via email to