projects
/
collectd.git
/ blobdiff
commit
grep
author
committer
pickaxe
?
search:
re
summary
|
shortlog
|
log
|
commit
|
commitdiff
|
tree
raw
|
inline
| side by side
fix typographical mistake in warning message
[collectd.git]
/
src
/
java.c
diff --git
a/src/java.c
b/src/java.c
index
cd01534
..
47f4cd3
100644
(file)
--- a/
src/java.c
+++ b/
src/java.c
@@
-22,11
+22,11
@@
**/
#include "collectd.h"
**/
#include "collectd.h"
+
#include "plugin.h"
#include "common.h"
#include "filter_chain.h"
#include "plugin.h"
#include "common.h"
#include "filter_chain.h"
-#include <pthread.h>
#include <jni.h>
#if !defined(JNI_VERSION_1_2)
#include <jni.h>
#if !defined(JNI_VERSION_1_2)
@@
-507,7
+507,6
@@
static jobject ctoj_oconfig_item (JNIEnv *jvm_env, /* {{{ */
jmethodID m_addchild;
jobject o_key;
jobject o_ocitem;
jmethodID m_addchild;
jobject o_key;
jobject o_ocitem;
- int i;
c_ocitem = (*jvm_env)->FindClass (jvm_env, "org/collectd/api/OConfigItem");
if (c_ocitem == NULL)
c_ocitem = (*jvm_env)->FindClass (jvm_env, "org/collectd/api/OConfigItem");
if (c_ocitem == NULL)
@@
-572,7
+571,7
@@
static jobject ctoj_oconfig_item (JNIEnv *jvm_env, /* {{{ */
(*jvm_env)->DeleteLocalRef (jvm_env, o_key);
/* Call OConfigItem.addValue for each value */
(*jvm_env)->DeleteLocalRef (jvm_env, o_key);
/* Call OConfigItem.addValue for each value */
- for (i = 0; i < ci->values_num; i++) /* {{{ */
+ for (i
nt i
= 0; i < ci->values_num; i++) /* {{{ */
{
jobject o_value;
{
jobject o_value;
@@
-590,7
+589,7
@@
static jobject ctoj_oconfig_item (JNIEnv *jvm_env, /* {{{ */
} /* }}} for (i = 0; i < ci->values_num; i++) */
/* Call OConfigItem.addChild for each child */
} /* }}} for (i = 0; i < ci->values_num; i++) */
/* Call OConfigItem.addChild for each child */
- for (i = 0; i < ci->children_num; i++) /* {{{ */
+ for (i
nt i
= 0; i < ci->children_num; i++) /* {{{ */
{
jobject o_child;
{
jobject o_child;
@@
-618,7
+617,6
@@
static jobject ctoj_data_set (JNIEnv *jvm_env, const data_set_t *ds) /* {{{ */
jmethodID m_add;
jobject o_type;
jobject o_dataset;
jmethodID m_add;
jobject o_type;
jobject o_dataset;
- size_t i;
/* Look up the org/collectd/api/DataSet class */
c_dataset = (*jvm_env)->FindClass (jvm_env, "org/collectd/api/DataSet");
/* Look up the org/collectd/api/DataSet class */
c_dataset = (*jvm_env)->FindClass (jvm_env, "org/collectd/api/DataSet");
@@
-668,7
+666,7
@@
static jobject ctoj_data_set (JNIEnv *jvm_env, const data_set_t *ds) /* {{{ */
/* Decrease reference counter on the java.lang.String object. */
(*jvm_env)->DeleteLocalRef (jvm_env, o_type);
/* Decrease reference counter on the java.lang.String object. */
(*jvm_env)->DeleteLocalRef (jvm_env, o_type);
- for (i = 0; i < ds->ds_num; i++)
+ for (
size_t
i = 0; i < ds->ds_num; i++)
{
jobject o_datasource;
{
jobject o_datasource;
@@
-763,7
+761,6
@@
static jobject ctoj_value_list (JNIEnv *jvm_env, /* {{{ */
jmethodID m_valuelist_constructor;
jobject o_valuelist;
int status;
jmethodID m_valuelist_constructor;
jobject o_valuelist;
int status;
- size_t i;
/* First, create a new ValueList instance..
* Look up the class.. */
/* First, create a new ValueList instance..
* Look up the class.. */
@@
-846,7
+843,7
@@
static jobject ctoj_value_list (JNIEnv *jvm_env, /* {{{ */
return (NULL);
}
return (NULL);
}
- for (i = 0; i < vl->values_len; i++)
+ for (
size_t
i = 0; i < vl->values_len; i++)
{
status = ctoj_value_list_add_value (jvm_env, vl->values[i], ds->ds[i].type,
c_valuelist, o_valuelist);
{
status = ctoj_value_list_add_value (jvm_env, vl->values[i], ds->ds[i].type,
c_valuelist, o_valuelist);
@@
-1117,7
+1114,6
@@
static int jtoc_values_array (JNIEnv *jvm_env, /* {{{ */
value_t *values;
int values_num;
value_t *values;
int values_num;
- int i;
values_num = ds->ds_num;
values_num = ds->ds_num;
@@
-1177,7
+1173,7
@@
static int jtoc_values_array (JNIEnv *jvm_env, /* {{{ */
BAIL_OUT (-1);
}
BAIL_OUT (-1);
}
- for (i = 0; i < values_num; i++)
+ for (i
nt i
= 0; i < values_num; i++)
{
jobject o_number;
int status;
{
jobject o_number;
int status;
@@
-1364,12
+1360,9
@@
static jint JNICALL cjni_api_dispatch_values (JNIEnv *jvm_env, /* {{{ */
static jint JNICALL cjni_api_dispatch_notification (JNIEnv *jvm_env, /* {{{ */
jobject this, jobject o_notification)
{
static jint JNICALL cjni_api_dispatch_notification (JNIEnv *jvm_env, /* {{{ */
jobject this, jobject o_notification)
{
- notification_t n;
+ notification_t n
= { 0 }
;
int status;
int status;
- memset (&n, 0, sizeof (n));
- n.meta = NULL;
-
status = jtoc_notification (jvm_env, &n, o_notification);
if (status != 0)
{
status = jtoc_notification (jvm_env, &n, o_notification);
if (status != 0)
{
@@
-1424,7
+1417,7
@@
static jint JNICALL cjni_api_register_init (JNIEnv *jvm_env, /* {{{ */
static jint JNICALL cjni_api_register_read (JNIEnv *jvm_env, /* {{{ */
jobject this, jobject o_name, jobject o_read)
{
static jint JNICALL cjni_api_register_read (JNIEnv *jvm_env, /* {{{ */
jobject this, jobject o_name, jobject o_read)
{
- user_data_t ud;
+ user_data_t ud
= { 0 }
;
cjni_callback_info_t *cbi;
cbi = cjni_callback_info_create (jvm_env, o_name, o_read, CB_TYPE_READ);
cjni_callback_info_t *cbi;
cbi = cjni_callback_info_create (jvm_env, o_name, o_read, CB_TYPE_READ);
@@
-1433,7
+1426,6
@@
static jint JNICALL cjni_api_register_read (JNIEnv *jvm_env, /* {{{ */
DEBUG ("java plugin: Registering new read callback: %s", cbi->name);
DEBUG ("java plugin: Registering new read callback: %s", cbi->name);
- memset (&ud, 0, sizeof (ud));
ud.data = (void *) cbi;
ud.free_func = cjni_callback_info_destroy;
ud.data = (void *) cbi;
ud.free_func = cjni_callback_info_destroy;
@@
-1448,7
+1440,7
@@
static jint JNICALL cjni_api_register_read (JNIEnv *jvm_env, /* {{{ */
static jint JNICALL cjni_api_register_write (JNIEnv *jvm_env, /* {{{ */
jobject this, jobject o_name, jobject o_write)
{
static jint JNICALL cjni_api_register_write (JNIEnv *jvm_env, /* {{{ */
jobject this, jobject o_name, jobject o_write)
{
- user_data_t ud;
+ user_data_t ud
= { 0 }
;
cjni_callback_info_t *cbi;
cbi = cjni_callback_info_create (jvm_env, o_name, o_write, CB_TYPE_WRITE);
cjni_callback_info_t *cbi;
cbi = cjni_callback_info_create (jvm_env, o_name, o_write, CB_TYPE_WRITE);
@@
-1457,7
+1449,6
@@
static jint JNICALL cjni_api_register_write (JNIEnv *jvm_env, /* {{{ */
DEBUG ("java plugin: Registering new write callback: %s", cbi->name);
DEBUG ("java plugin: Registering new write callback: %s", cbi->name);
- memset (&ud, 0, sizeof (ud));
ud.data = (void *) cbi;
ud.free_func = cjni_callback_info_destroy;
ud.data = (void *) cbi;
ud.free_func = cjni_callback_info_destroy;
@@
-1471,7
+1462,7
@@
static jint JNICALL cjni_api_register_write (JNIEnv *jvm_env, /* {{{ */
static jint JNICALL cjni_api_register_flush (JNIEnv *jvm_env, /* {{{ */
jobject this, jobject o_name, jobject o_flush)
{
static jint JNICALL cjni_api_register_flush (JNIEnv *jvm_env, /* {{{ */
jobject this, jobject o_name, jobject o_flush)
{
- user_data_t ud;
+ user_data_t ud
= { 0 }
;
cjni_callback_info_t *cbi;
cbi = cjni_callback_info_create (jvm_env, o_name, o_flush, CB_TYPE_FLUSH);
cjni_callback_info_t *cbi;
cbi = cjni_callback_info_create (jvm_env, o_name, o_flush, CB_TYPE_FLUSH);
@@
-1480,7
+1471,6
@@
static jint JNICALL cjni_api_register_flush (JNIEnv *jvm_env, /* {{{ */
DEBUG ("java plugin: Registering new flush callback: %s", cbi->name);
DEBUG ("java plugin: Registering new flush callback: %s", cbi->name);
- memset (&ud, 0, sizeof (ud));
ud.data = (void *) cbi;
ud.free_func = cjni_callback_info_destroy;
ud.data = (void *) cbi;
ud.free_func = cjni_callback_info_destroy;
@@
-1501,7
+1491,7
@@
static jint JNICALL cjni_api_register_shutdown (JNIEnv *jvm_env, /* {{{ */
static jint JNICALL cjni_api_register_log (JNIEnv *jvm_env, /* {{{ */
jobject this, jobject o_name, jobject o_log)
{
static jint JNICALL cjni_api_register_log (JNIEnv *jvm_env, /* {{{ */
jobject this, jobject o_name, jobject o_log)
{
- user_data_t ud;
+ user_data_t ud
= { 0 }
;
cjni_callback_info_t *cbi;
cbi = cjni_callback_info_create (jvm_env, o_name, o_log, CB_TYPE_LOG);
cjni_callback_info_t *cbi;
cbi = cjni_callback_info_create (jvm_env, o_name, o_log, CB_TYPE_LOG);
@@
-1510,7
+1500,6
@@
static jint JNICALL cjni_api_register_log (JNIEnv *jvm_env, /* {{{ */
DEBUG ("java plugin: Registering new log callback: %s", cbi->name);
DEBUG ("java plugin: Registering new log callback: %s", cbi->name);
- memset (&ud, 0, sizeof (ud));
ud.data = (void *) cbi;
ud.free_func = cjni_callback_info_destroy;
ud.data = (void *) cbi;
ud.free_func = cjni_callback_info_destroy;
@@
-1524,7
+1513,7
@@
static jint JNICALL cjni_api_register_log (JNIEnv *jvm_env, /* {{{ */
static jint JNICALL cjni_api_register_notification (JNIEnv *jvm_env, /* {{{ */
jobject this, jobject o_name, jobject o_notification)
{
static jint JNICALL cjni_api_register_notification (JNIEnv *jvm_env, /* {{{ */
jobject this, jobject o_name, jobject o_notification)
{
- user_data_t ud;
+ user_data_t ud
= { 0 }
;
cjni_callback_info_t *cbi;
cbi = cjni_callback_info_create (jvm_env, o_name, o_notification,
cjni_callback_info_t *cbi;
cbi = cjni_callback_info_create (jvm_env, o_name, o_notification,
@@
-1534,7
+1523,6
@@
static jint JNICALL cjni_api_register_notification (JNIEnv *jvm_env, /* {{{ */
DEBUG ("java plugin: Registering new notification callback: %s", cbi->name);
DEBUG ("java plugin: Registering new notification callback: %s", cbi->name);
- memset (&ud, 0, sizeof (ud));
ud.data = (void *) cbi;
ud.free_func = cjni_callback_info_destroy;
ud.data = (void *) cbi;
ud.free_func = cjni_callback_info_destroy;
@@
-1568,9
+1556,8
@@
static jint JNICALL cjni_api_register_match_target (JNIEnv *jvm_env, /* {{{ */
if (type == CB_TYPE_MATCH)
{
if (type == CB_TYPE_MATCH)
{
- match_proc_t m_proc;
+ match_proc_t m_proc
= { 0 }
;
- memset (&m_proc, 0, sizeof (m_proc));
m_proc.create = cjni_match_target_create;
m_proc.destroy = cjni_match_target_destroy;
m_proc.match = (void *) cjni_match_target_invoke;
m_proc.create = cjni_match_target_create;
m_proc.destroy = cjni_match_target_destroy;
m_proc.match = (void *) cjni_match_target_invoke;
@@
-1579,9
+1566,8
@@
static jint JNICALL cjni_api_register_match_target (JNIEnv *jvm_env, /* {{{ */
}
else if (type == CB_TYPE_TARGET)
{
}
else if (type == CB_TYPE_TARGET)
{
- target_proc_t t_proc;
+ target_proc_t t_proc
= { 0 }
;
- memset (&t_proc, 0, sizeof (t_proc));
t_proc.create = cjni_match_target_create;
t_proc.destroy = cjni_match_target_destroy;
t_proc.invoke = cjni_match_target_invoke;
t_proc.create = cjni_match_target_create;
t_proc.destroy = cjni_match_target_destroy;
t_proc.invoke = cjni_match_target_invoke;
@@
-1801,14
+1787,13
@@
static cjni_callback_info_t *cjni_callback_info_create (JNIEnv *jvm_env, /* {{{
return (NULL);
}
return (NULL);
}
- cbi =
malloc (
sizeof (*cbi));
+ cbi =
calloc (1,
sizeof (*cbi));
if (cbi == NULL)
{
if (cbi == NULL)
{
- ERROR ("java plugin: cjni_callback_info_create:
m
alloc failed.");
+ ERROR ("java plugin: cjni_callback_info_create:
c
alloc failed.");
(*jvm_env)->ReleaseStringUTFChars (jvm_env, o_name, c_name);
return (NULL);
}
(*jvm_env)->ReleaseStringUTFChars (jvm_env, o_name, c_name);
return (NULL);
}
- memset (cbi, 0, sizeof (*cbi));
cbi->type = type;
cbi->name = strdup (c_name);
cbi->type = type;
cbi->name = strdup (c_name);
@@
-1906,7
+1891,7
@@
static int cjni_callback_register (JNIEnv *jvm_env, /* {{{ */
pthread_mutex_lock (&java_callbacks_lock);
pthread_mutex_lock (&java_callbacks_lock);
- tmp =
(cjni_callback_info_t *)
realloc (java_callbacks,
+ tmp = realloc (java_callbacks,
(java_callbacks_num + 1) * sizeof (*java_callbacks));
if (tmp == NULL)
{
(java_callbacks_num + 1) * sizeof (*java_callbacks));
if (tmp == NULL)
{
@@
-1987,11
+1972,10
@@
static int cjni_init_native (JNIEnv *jvm_env) /* {{{ */
static int cjni_create_jvm (void) /* {{{ */
{
JNIEnv *jvm_env;
static int cjni_create_jvm (void) /* {{{ */
{
JNIEnv *jvm_env;
- JavaVMInitArgs vm_args;
+ JavaVMInitArgs vm_args
= { 0 }
;
JavaVMOption vm_options[jvm_argc];
int status;
JavaVMOption vm_options[jvm_argc];
int status;
- size_t i;
if (jvm != NULL)
return (0);
if (jvm != NULL)
return (0);
@@
-2006,12
+1990,11
@@
static int cjni_create_jvm (void) /* {{{ */
jvm_env = NULL;
jvm_env = NULL;
- memset (&vm_args, 0, sizeof (vm_args));
vm_args.version = JNI_VERSION_1_2;
vm_args.options = vm_options;
vm_args.nOptions = (jint) jvm_argc;
vm_args.version = JNI_VERSION_1_2;
vm_args.options = vm_options;
vm_args.nOptions = (jint) jvm_argc;
- for (i = 0; i < jvm_argc; i++)
+ for (
size_t
i = 0; i < jvm_argc; i++)
{
DEBUG ("java plugin: cjni_create_jvm: jvm_argv[%zu] = %s",
i, jvm_argv[i]);
{
DEBUG ("java plugin: cjni_create_jvm: jvm_argv[%zu] = %s",
i, jvm_argv[i]);
@@
-2067,13
+2050,12
@@
static JNIEnv *cjni_thread_attach (void) /* {{{ */
if (cjni_env == NULL)
{
/* This pointer is free'd in `cjni_jvm_env_destroy'. */
if (cjni_env == NULL)
{
/* This pointer is free'd in `cjni_jvm_env_destroy'. */
- cjni_env =
malloc (
sizeof (*cjni_env));
+ cjni_env =
calloc (1,
sizeof (*cjni_env));
if (cjni_env == NULL)
{
if (cjni_env == NULL)
{
- ERROR ("java plugin: cjni_thread_attach:
m
alloc failed.");
+ ERROR ("java plugin: cjni_thread_attach:
c
alloc failed.");
return (NULL);
}
return (NULL);
}
- memset (cjni_env, 0, sizeof (*cjni_env));
cjni_env->reference_counter = 0;
cjni_env->jvm_env = NULL;
cjni_env->reference_counter = 0;
cjni_env->jvm_env = NULL;
@@
-2088,11
+2070,10
@@
static JNIEnv *cjni_thread_attach (void) /* {{{ */
else
{
int status;
else
{
int status;
- JavaVMAttachArgs args;
+ JavaVMAttachArgs args
= { 0 }
;
assert (cjni_env->jvm_env == NULL);
assert (cjni_env->jvm_env == NULL);
- memset (&args, 0, sizeof (args));
args.version = JNI_VERSION_1_2;
status = (*jvm)->AttachCurrentThread (jvm, (void *) &jvm_env, (void *) &args);
args.version = JNI_VERSION_1_2;
status = (*jvm)->AttachCurrentThread (jvm, (void *) &jvm_env, (void *) &args);
@@
-2169,7
+2150,7
@@
static int cjni_config_add_jvm_arg (oconfig_item_t *ci) /* {{{ */
return (-1);
}
return (-1);
}
- tmp =
(char **)
realloc (jvm_argv, sizeof (char *) * (jvm_argc + 1));
+ tmp = realloc (jvm_argv, sizeof (char *) * (jvm_argc + 1));
if (tmp == NULL)
{
ERROR ("java plugin: realloc failed.");
if (tmp == NULL)
{
ERROR ("java plugin: realloc failed.");
@@
-2205,7
+2186,7
@@
static int cjni_config_load_plugin (oconfig_item_t *ci) /* {{{ */
if (jvm_env == NULL)
return (-1);
if (jvm_env == NULL)
return (-1);
- class =
(java_plugin_class_t *)
realloc (java_classes_list,
+ class = realloc (java_classes_list,
(java_classes_list_len + 1) * sizeof (*java_classes_list));
if (class == NULL)
{
(java_classes_list_len + 1) * sizeof (*java_classes_list));
if (class == NULL)
{
@@
-2230,8
+2211,7
@@
static int cjni_config_load_plugin (oconfig_item_t *ci) /* {{{ */
{ /* Replace all dots ('.') with slashes ('/'). Dots are usually used
thorough the Java community, but (Sun's) `FindClass' and friends need
slashes. */
{ /* Replace all dots ('.') with slashes ('/'). Dots are usually used
thorough the Java community, but (Sun's) `FindClass' and friends need
slashes. */
- size_t i;
- for (i = 0; class->name[i] != 0; i++)
+ for (size_t i = 0; class->name[i] != 0; i++)
if (class->name[i] == '.')
class->name[i] = '/';
}
if (class->name[i] == '.')
class->name[i] = '/';
}
@@
-2289,7
+2269,6
@@
static int cjni_config_plugin_block (oconfig_item_t *ci) /* {{{ */
cjni_callback_info_t *cbi;
jobject o_ocitem;
const char *name;
cjni_callback_info_t *cbi;
jobject o_ocitem;
const char *name;
- size_t i;
jclass class;
jmethodID method;
jclass class;
jmethodID method;
@@
-2304,7
+2283,7
@@
static int cjni_config_plugin_block (oconfig_item_t *ci) /* {{{ */
name = ci->values[0].value.string;
cbi = NULL;
name = ci->values[0].value.string;
cbi = NULL;
- for (i = 0; i < java_callbacks_num; i++)
+ for (
size_t
i = 0; i < java_callbacks_num; i++)
{
if (java_callbacks[i].type != CB_TYPE_CONFIG)
continue;
{
if (java_callbacks[i].type != CB_TYPE_CONFIG)
continue;
@@
-2356,12
+2335,11
@@
static int cjni_config_perform (oconfig_item_t *ci) /* {{{ */
int success;
int errors;
int status;
int success;
int errors;
int status;
- int i;
success = 0;
errors = 0;
success = 0;
errors = 0;
- for (i = 0; i < ci->children_num; i++)
+ for (i
nt i
= 0; i < ci->children_num; i++)
{
oconfig_item_t *child = ci->children + i;
{
oconfig_item_t *child = ci->children + i;
@@
-2724,7
+2702,6
@@
static int cjni_match_target_create (const oconfig_item_t *ci, /* {{{ */
jobject o_ci;
jobject o_tmp;
int type;
jobject o_ci;
jobject o_tmp;
int type;
- size_t i;
cbi_ret = NULL;
o_ci = NULL;
cbi_ret = NULL;
o_ci = NULL;
@@
-2769,7
+2746,7
@@
static int cjni_match_target_create (const oconfig_item_t *ci, /* {{{ */
/* Lets see if we have a matching factory here.. */
cbi_factory = NULL;
/* Lets see if we have a matching factory here.. */
cbi_factory = NULL;
- for (i = 0; i < java_callbacks_num; i++)
+ for (
size_t
i = 0; i < java_callbacks_num; i++)
{
if (java_callbacks[i].type != type)
continue;
{
if (java_callbacks[i].type != type)
continue;
@@
-2801,13
+2778,13
@@
static int cjni_match_target_create (const oconfig_item_t *ci, /* {{{ */
/* Allocate a new callback info structure. This is going to be our user_data
* pointer. */
/* Allocate a new callback info structure. This is going to be our user_data
* pointer. */
- cbi_ret =
malloc (
sizeof (*cbi_ret));
+ cbi_ret =
calloc (1,
sizeof (*cbi_ret));
if (cbi_ret == NULL)
{
if (cbi_ret == NULL)
{
- ERROR ("java plugin: cjni_match_target_create:
m
alloc failed.");
+ ERROR ("java plugin: cjni_match_target_create:
c
alloc failed.");
BAIL_OUT (-1);
}
BAIL_OUT (-1);
}
- memset (cbi_ret, 0, sizeof (*cbi_ret));
+
cbi_ret->object = NULL;
cbi_ret->type = type;
cbi_ret->object = NULL;
cbi_ret->type = type;
@@
-2923,9
+2900,8
@@
static int cjni_match_target_invoke (const data_set_t *ds, /* {{{ */
* `value_list_t'. */
if (cbi->type == CB_TYPE_TARGET)
{
* `value_list_t'. */
if (cbi->type == CB_TYPE_TARGET)
{
- value_list_t new_vl;
+ value_list_t new_vl
= { 0 }
;
- memset (&new_vl, 0, sizeof (new_vl));
status = jtoc_value_list (jvm_env, &new_vl, o_vl);
if (status != 0)
{
status = jtoc_value_list (jvm_env, &new_vl, o_vl);
if (status != 0)
{
@@
-2952,9
+2928,8
@@
static int cjni_match_target_invoke (const data_set_t *ds, /* {{{ */
static int cjni_init_plugins (JNIEnv *jvm_env) /* {{{ */
{
int status;
static int cjni_init_plugins (JNIEnv *jvm_env) /* {{{ */
{
int status;
- size_t i;
- for (i = 0; i < java_callbacks_num; i++)
+ for (
size_t
i = 0; i < java_callbacks_num; i++)
{
if (java_callbacks[i].type != CB_TYPE_INIT)
continue;
{
if (java_callbacks[i].type != CB_TYPE_INIT)
continue;
@@
-2979,9
+2954,8
@@
static int cjni_init_plugins (JNIEnv *jvm_env) /* {{{ */
static int cjni_shutdown_plugins (JNIEnv *jvm_env) /* {{{ */
{
int status;
static int cjni_shutdown_plugins (JNIEnv *jvm_env) /* {{{ */
{
int status;
- size_t i;
- for (i = 0; i < java_callbacks_num; i++)
+ for (
size_t
i = 0; i < java_callbacks_num; i++)
{
if (java_callbacks[i].type != CB_TYPE_SHUTDOWN)
continue;
{
if (java_callbacks[i].type != CB_TYPE_SHUTDOWN)
continue;
@@
-3004,15
+2978,13
@@
static int cjni_shutdown_plugins (JNIEnv *jvm_env) /* {{{ */
static int cjni_shutdown (void) /* {{{ */
{
JNIEnv *jvm_env;
static int cjni_shutdown (void) /* {{{ */
{
JNIEnv *jvm_env;
- JavaVMAttachArgs args;
+ JavaVMAttachArgs args
= { 0 }
;
int status;
int status;
- size_t i;
if (jvm == NULL)
return (0);
jvm_env = NULL;
if (jvm == NULL)
return (0);
jvm_env = NULL;
- memset (&args, 0, sizeof (args));
args.version = JNI_VERSION_1_2;
status = (*jvm)->AttachCurrentThread (jvm, (void *) &jvm_env, &args);
args.version = JNI_VERSION_1_2;
status = (*jvm)->AttachCurrentThread (jvm, (void *) &jvm_env, &args);
@@
-3027,7
+2999,7
@@
static int cjni_shutdown (void) /* {{{ */
cjni_shutdown_plugins (jvm_env);
/* Release all the global references to callback functions */
cjni_shutdown_plugins (jvm_env);
/* Release all the global references to callback functions */
- for (i = 0; i < java_callbacks_num; i++)
+ for (
size_t
i = 0; i < java_callbacks_num; i++)
{
if (java_callbacks[i].object != NULL)
{
{
if (java_callbacks[i].object != NULL)
{
@@
-3040,7
+3012,7
@@
static int cjni_shutdown (void) /* {{{ */
sfree (java_callbacks);
/* Release all the global references to directly loaded classes. */
sfree (java_callbacks);
/* Release all the global references to directly loaded classes. */
- for (i = 0; i < java_classes_list_len; i++)
+ for (
size_t
i = 0; i < java_classes_list_len; i++)
{
if (java_classes_list[i].object != NULL)
{
{
if (java_classes_list[i].object != NULL)
{
@@
-3061,7
+3033,7
@@
static int cjni_shutdown (void) /* {{{ */
pthread_key_delete (jvm_env_key);
/* Free the JVM argument list */
pthread_key_delete (jvm_env_key);
/* Free the JVM argument list */
- for (i = 0; i < jvm_argc; i++)
+ for (
size_t
i = 0; i < jvm_argc; i++)
sfree (jvm_argv[i]);
jvm_argc = 0;
sfree (jvm_argv);
sfree (jvm_argv[i]);
jvm_argc = 0;
sfree (jvm_argv);