X-Git-Url: http://info.iut-bm.univ-fcomte.fr/pub/gitweb/simgrid.git/blobdiff_plain/5b193c4fadd2fdc28864fe280cf031ba98d56c89..a90563bd9caee92326d0a5587a613537104990bc:/src/bindings/java/jmsg_vm.cpp diff --git a/src/bindings/java/jmsg_vm.cpp b/src/bindings/java/jmsg_vm.cpp index 7a58af6f4f..d7e9ea12b6 100644 --- a/src/bindings/java/jmsg_vm.cpp +++ b/src/bindings/java/jmsg_vm.cpp @@ -65,20 +65,21 @@ JNIEXPORT void JNICALL Java_org_simgrid_msg_VM_setBound(JNIEnv *env, jobject jvm MSG_vm_set_bound(vm, bound); } -JNIEXPORT void JNICALL Java_org_simgrid_msg_VM_create(JNIEnv* env, jobject jvm, jobject jhost, jstring jname, - jint jramsize, jint jmig_netspeed, jint jdp_intensity) +JNIEXPORT void JNICALL Java_org_simgrid_msg_VM_create(JNIEnv* env, jobject jVm, jobject jHost, jstring jname, + jint coreAmount, jint jramsize, jint jmig_netspeed, + jint jdp_intensity) { - msg_host_t host = jhost_get_native(env, jhost); + msg_host_t host = jhost_get_native(env, jHost); const char* name = env->GetStringUTFChars(jname, 0); - msg_vm_t vm = MSG_vm_create(host, name, static_cast(jramsize), static_cast(jmig_netspeed), - static_cast(jdp_intensity)); + msg_vm_t vm = MSG_vm_create(host, name, static_cast(coreAmount), static_cast(jramsize), + static_cast(jmig_netspeed), static_cast(jdp_intensity)); env->ReleaseStringUTFChars(jname, name); - jvm_bind(env, jvm, vm); - jvm = env->NewWeakGlobalRef(jvm); + jvm_bind(env, jVm, vm); + jVm = env->NewWeakGlobalRef(jVm); // We use the extension level of the host, even if that's somehow disturbing - vm->extension_set(JAVA_HOST_LEVEL, (void*)jvm); + vm->extension_set(JAVA_HOST_LEVEL, (void*)jVm); } JNIEXPORT jobjectArray JNICALL Java_org_simgrid_msg_VM_all(JNIEnv* env, jclass cls_arg) @@ -95,16 +96,17 @@ JNIEXPORT jobjectArray JNICALL Java_org_simgrid_msg_VM_all(JNIEnv* env, jclass c vms.push_back(jvm); } } + xbt_dynar_free(&hosts); vms.shrink_to_fit(); int count = vms.size(); jclass cls = jxbt_get_class(env, "org/simgrid/msg/VM"); - if (!cls) + if (not cls) return nullptr; jobjectArray jtable = env->NewObjectArray((jsize)count, cls, nullptr); - if (!jtable) { + if (not jtable) { jxbt_throw_jni(env, "Hosts table allocation failed"); return nullptr; } @@ -135,7 +137,9 @@ JNIEXPORT void JNICALL Java_org_simgrid_msg_VM_shutdown(JNIEnv *env, jobject jvm MSG_vm_shutdown(vm); auto vmList = &simgrid::vm::VirtualMachineImpl::allVms_; vmList->erase( - std::remove_if(vmList->begin(), vmList->end(), [vm](simgrid::s4u::VirtualMachine* it) { return vm == it; }), + std::remove_if(vmList->begin(), vmList->end(), [vm](simgrid::s4u::VirtualMachine* it) { + return vm == it; + }), vmList->end()); } } @@ -177,7 +181,7 @@ JNIEXPORT jobject JNICALL Java_org_simgrid_msg_VM_getVMByName(JNIEnv* env, jclas /* get the VM by name (VMs are just special hosts, unfortunately) */ msg_host_t host = MSG_host_by_name(name); - if (!host) { /* invalid name */ + if (not host) { /* invalid name */ jxbt_throw_host_not_found(env, name); env->ReleaseStringUTFChars(jname, name); return nullptr;