> This is a very basic solution to the OOBE described in the issue. While it 
> would definitely be better to figure out the root cause, understanding the 
> NGNode implementation is above my knowledge level atm.
> 
> The code has been like that for 10 years and no one complained, so I am not 
> sure whether this is a deeper issue or just a small oversight when handling 
> all possible cases.
> 
> The fix should retain the exact original workings of the method while 
> avoiding the exception.

Christopher Schnick has updated the pull request incrementally with one 
additional commit since the last revision:

  Add comment

-------------

Changes:
  - all: https://git.openjdk.org/jfx/pull/2037/files
  - new: https://git.openjdk.org/jfx/pull/2037/files/c55f674b..005d73a5

Webrevs:
 - full: https://webrevs.openjdk.org/?repo=jfx&pr=2037&range=02
 - incr: https://webrevs.openjdk.org/?repo=jfx&pr=2037&range=01-02

  Stats: 2 lines in 1 file changed: 2 ins; 0 del; 0 mod
  Patch: https://git.openjdk.org/jfx/pull/2037.diff
  Fetch: git fetch https://git.openjdk.org/jfx.git pull/2037/head:pull/2037

PR: https://git.openjdk.org/jfx/pull/2037

Reply via email to