Looks good to me.
Naoto
(9/20/10 7:33 AM), Omair Majid wrote:
Hi,
Naoto Sato<naoto.s...@...> writes:
Thank you for catching the error. I think your fix is fine but would
like some comments noting that the extra call to getX11InputMethodData()
in DestroyXIMCallback is merely to free up the pX11IMData, otherwise,
the call is kind of cryptic.
Thanks for reviewing the patch! Updated webrev is available at:
http://cr.openjdk.java.net/~omajid/webrevs/crash-on-xim-server-restart/webrev.01/
Thanks,
Omair
