Unnecessary synchronisation?

[sebbASF]

jna/src/com/sun/jna/internal/Cleaner.java

Lines 61 to 62 in 3656295

	private synchronized CleanerRef add(CleanerRef ref) {
	synchronized (referenceQueue) {

jna/src/com/sun/jna/internal/Cleaner.java

Lines 79 to 80 in 3656295

	private synchronized boolean remove(CleanerRef ref) {
	synchronized (referenceQueue) {

The entire body of both methods is part of the synch. block, so it seems unnecessary to synch the method as well.

[matthiasblaesing]

That might be, given that there we are already hitting the slow path (synchronized), I currently have no cycles to look into something that does not seriously effect performance, so if someone wants to change this, can make it clear, that it is safe and show numbers over JDK 8-21 that this is a problem, I'm willing to review.