> Fixed as described in the issue + added tests to check for this scenario with 
> all Transform sub classes. Since the test scenario slightly exceeding the 
> regular parametrized testing of `TransformOperationsTest` I decided to 
> implement the tests in a separate class.
> 
> I didn't manage to reproduce the issue with other Transform sub classes, so 
> it seems to only affect `Rotate`. Also checked by looking at the code, only 
> `Rotate` was affected by this bug. As such, without 08ba284 only 
> `testTransformInverseCache_Rotate` fails, while others succeed. With the fix, 
> all tests pass.
> 
> Ran the whole test suite afterwards and didn't notice any changes to test 
> results after introducing the fix.

Lukasz Kostyra has updated the pull request incrementally with one additional 
commit since the last revision:

  Remove unnecessary imports from test

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

Changes:
  - all: https://git.openjdk.org/jfx/pull/1392/files
  - new: https://git.openjdk.org/jfx/pull/1392/files/08ba2848..cd8fdde8

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

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

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

Reply via email to