[RFC][plugin]: fix fixme

[Andrew Su]

----- Original Message -----
> Hi.
> 
> This patch implements the fixme in PluginObjectStore
> using the "wrapped" idea in that comment. It also
> synchronizes modifications to the 3 hash maps because,
> afaics, they can be called concurrently.
> 
> ChangeLog:
> +2011-03-30 Denis Lila <dlila at redhat.com>
> +
> + * plugin/icedteanp/java/sun/applet/PluginObjectStore.java
> + (wrapped): New static variable.
> + (getNextID): New function.
> + (reference): Using getNextID and synchronized.
> + (unreference): Synchronized.
> + (dump): Improve iteration.
> 
> Any criticism is always welcome.
> 
> Thank you,
> Denis.

Hi Denis,

+    	if (wrapped)
+            while(objects.containsKey(nextUniqueIdentifier))
+                nextUniqueIdentifier++;
+        return nextUniqueIdentifier++;
+    }
Are we not worried about the identifiers going into negatives?
I think there's an extra space before the return.

Other than that, the patch looks ok to me.

Cheers,
 Andrew