This logic is essentially repeating itself

  if (!g_thread_supported())
  {
    XInitThreads();
    g_thread_init(NULL);
  }

  if (!g_module_supported()) {
    return JNI_FALSE;
  }

It would be better to do this instead:


  if (!g_thread_supported())
  {
    XInitThreads();
    g_thread_init(NULL);
    return JNI_FALSE;
  }

Because it's nicer.