Logo AND Algorithmique Numérique Distribuée

Public GIT Repository
don't leak the routing name up to into Java
authorMartin Quinson <martin.quinson@loria.fr>
Thu, 11 Feb 2016 23:35:25 +0000 (00:35 +0100)
committerMartin Quinson <martin.quinson@loria.fr>
Thu, 11 Feb 2016 23:38:02 +0000 (00:38 +0100)
we were implementing instanceof with char*...
That was during the (golden) C age of SimGrid.

include/simgrid/msg.h
src/bindings/java/jmsg_as.cpp
src/bindings/java/jmsg_as.h
src/bindings/java/org/simgrid/msg/As.java
src/include/surf/surf.h
src/msg/msg_environment.cpp
src/surf/surf_routing.cpp

index 97c370b..2c8c022 100644 (file)
@@ -233,7 +233,6 @@ XBT_PUBLIC(const char *) MSG_environment_as_get_name(msg_as_t as);
 XBT_PUBLIC(msg_as_t) MSG_environment_as_get_by_name(const char * name);
 XBT_PUBLIC(xbt_dict_t) MSG_environment_as_get_routing_sons(msg_as_t as);
 XBT_PUBLIC(const char *) MSG_environment_as_get_property_value(msg_as_t as, const char *name);
-XBT_PUBLIC(const char *) MSG_environment_as_get_model(msg_as_t as);
 XBT_PUBLIC(xbt_dynar_t) MSG_environment_as_get_hosts(msg_as_t as);
 
 /************************** File handling ***********************************/
index 474bccd..e1d50eb 100644 (file)
@@ -125,12 +125,6 @@ Java_org_simgrid_msg_As_getProperty(JNIEnv *env, jobject jas, jobject jname) {
   return jproperty;
 }
 
-JNIEXPORT jobject JNICALL
-Java_org_simgrid_msg_As_getModel(JNIEnv * env, jobject jas) {
-  msg_as_t as = jas_get_native(env, jas);
-  return env->NewStringUTF(MSG_environment_as_get_model(as));
-}
-
 JNIEXPORT jobjectArray JNICALL
 Java_org_simgrid_msg_As_getHosts(JNIEnv * env, jobject jas)
 {
index d8f14e5..0c1d346 100644 (file)
@@ -31,9 +31,6 @@ Java_org_simgrid_msg_As_getSons(JNIEnv * env, jobject jas);
 JNIEXPORT jobject JNICALL
 Java_org_simgrid_msg_As_getProperty(JNIEnv *env, jobject jhost, jobject jname);
 
-JNIEXPORT jobject JNICALL
-Java_org_simgrid_msg_As_getModel(JNIEnv * env, jobject jas);
-
 JNIEXPORT jobjectArray JNICALL
 Java_org_simgrid_msg_As_getHosts(JNIEnv * env, jobject jas);
 
index 35248f8..40b68dc 100644 (file)
@@ -22,8 +22,6 @@ public class As {
 
        public native String getProperty(String name);
 
-       public native String getModel();
-
        public native Host[] getHosts();
 
        /**
index 166ce02..faf5f7e 100644 (file)
@@ -904,7 +904,6 @@ XBT_PUBLIC_DATA(AS_t) surf_AS_get_routing_root(void);
 XBT_PUBLIC_DATA(const char *) surf_AS_get_name(AS_t as);
 XBT_PUBLIC_DATA(AS_t) surf_AS_get_by_name(const char * name);
 XBT_PUBLIC_DATA(xbt_dict_t) surf_AS_get_routing_sons(AS_t as);
-XBT_PUBLIC_DATA(const char *) surf_AS_get_model(AS_t as);
 XBT_PUBLIC_DATA(xbt_dynar_t) surf_AS_get_hosts(AS_t as);
 XBT_PUBLIC_DATA(void) surf_AS_get_graph(AS_t as, xbt_graph_t graph, xbt_dict_t nodes, xbt_dict_t edges);
 XBT_PUBLIC_DATA(AS_t) surf_platf_get_root(routing_platf_t platf);
index b1123b3..b9c604a 100644 (file)
@@ -75,10 +75,6 @@ const char *MSG_environment_as_get_property_value(msg_as_t as, const char *name)
   return (char*) xbt_dict_get_or_null(dict, name);
 }
 
-const char *MSG_environment_as_get_model(msg_as_t as) {
-  return surf_AS_get_model(as);
-}
-
 xbt_dynar_t MSG_environment_as_get_hosts(msg_as_t as) {
   return surf_AS_get_hosts(as);
 }
index 119b9ab..34b7ecf 100644 (file)
@@ -876,11 +876,6 @@ xbt_dict_t surf_AS_get_routing_sons(simgrid::surf::As *as)
   return as->p_routingSons;
 }
 
-const char *surf_AS_get_model(simgrid::surf::As *as)
-{
-  return as->p_modelDesc->name;
-}
-
 xbt_dynar_t surf_AS_get_hosts(simgrid::surf::As *as)
 {
   xbt_dynar_t elms = as->p_indexNetworkElm;