> This PR proposes to remove the use of `Unsafe` in the class 
> `XEmbeddingContainer ` and replace it with the supported FFM API.
> 
> I tried to make this PR as small as possible while opening up for migration 
> of other classes later on (such as `XEmbedServer` which can be modified 
> similarly under a separate PR).
> 
> There are also three drive-by fixes in this PR:
>  * Moved JavaDocs for `XAtom` to its proper location and fixed two typos in 
> the text
>  * Declared a field in `XEmbeddingContainer` as `private final`
>  * In `XAtom`, use a utility method `assertAtomInitialized()` instead of the 
> current duplicated code

Per Minborg has updated the pull request incrementally with one additional 
commit since the last revision:

  Suppress restricted warning

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

Changes:
  - all: https://git.openjdk.org/jdk/pull/17846/files
  - new: https://git.openjdk.org/jdk/pull/17846/files/530f0a70..979c2c61

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

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

PR: https://git.openjdk.org/jdk/pull/17846

Reply via email to