[Rev 02] RFR: 8244297: memory leak test utility

Florian Kirmaier fkirmaier at openjdk.java.net
Tue May 5 20:45:31 UTC 2020


> It's based on the discussion of my previous PR: https://github.com/openjdk/jfx/pull/71
> 
> I Added test utility class copied from JMemoryBuddy and used it to simplify 4 of the existing unit tests.
> 
> It's a direct copy of my project [JMemoryBuddy](https://github.com/Sandec/JMemoryBuddy) without any changes.
> I'm also using it in most of the projects I'm involved with and in my experience, the tests with this Library are very
> stable. I can't remember wrong test results. Sometimes the memory behaviour of some libraries itself is not stable but
> the tests with JMemoryBuddy are basically always correct.

Florian Kirmaier has updated the pull request incrementally with one additional commit since the last revision:

  JDK-8244297
  Added some javadoc to the testutility
  Some methods/fields are now private

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

Changes:
  - all: https://git.openjdk.java.net/jfx/pull/204/files
  - new: https://git.openjdk.java.net/jfx/pull/204/files/e7d9ad65..72d12283

Webrevs:
 - full: https://webrevs.openjdk.java.net/jfx/204/webrev.02
 - incr: https://webrevs.openjdk.java.net/jfx/204/webrev.01-02

  Stats: 50 lines in 1 file changed: 39 ins; 1 del; 10 mod
  Patch: https://git.openjdk.java.net/jfx/pull/204.diff
  Fetch: git fetch https://git.openjdk.java.net/jfx pull/204/head:pull/204

PR: https://git.openjdk.java.net/jfx/pull/204


More information about the openjfx-dev mailing list