The fix looks good to me.

  Thanks,
  Alexandr.

On 4/13/2015 4:34 PM, Alexey Ivanov wrote:
Hello Swing team,

Could you please review the fix:
    bug: https://bugs.openjdk.java.net/browse/JDK-8074956
    webrev: http://cr.openjdk.java.net/~aivanov/8074956/jdk9/webrev.00/

Description:
Implementation of ContentModel.first(Element) uses cache for elements. It is implied all the possible elements are already created before you call first(Element) method for the first time. If you create a new Element instance and call first(Element) method with it, you'll get ArrayIndexOutOfBoundsException because the implication is now false.

The fix is to re-create arrays used for cache when an "unknown" Element is used as parameter to first(Element).

Thanks,
Alexey

Reply via email to