[JsonGen] Automatic object lookup

JsonGenerator/source/emitter.py

@@ -16,7 +16,7 @@
 # limitations under the License.
 
 class Emitter():
-    def __init__(self, file_name, indent_size, max_line_length = 220, autoindent=False):
+    def __init__(self, file_name, indent_size, max_line_length = 500, autoindent=False):
         self.file = open(file_name, "w") if file_name else None
         self.indent_size = indent_size
         self.indent = 0

JsonGenerator/source/header_loader.py

@@ -253,6 +253,8 @@ def compute_name(obj, arg, relay=None):
 
         clash_msg = "JSON-RPC name clash detected"
 
+        passed_interfaces = []
+
         event_interfaces = set()
 
         for interface in face.obj.classes:
@@ -476,10 +478,15 @@ def GenerateObject(ctype, was_typdef):
 
                         return "object", { "properties": properties, "required": required }
 
-                    if cppType.full_name == "::%s::Core::JSONRPC::Context" % config.FRAMEWORK_NAMESPACE:
+                    if "Core::JSONRPC::Context" in cppType.full_name:
                         result = "@context", {}
                     elif (cppType.vars and not cppType.methods) or not verify:
                         result = GenerateObject(cppType, isinstance(var.type.Type(), CppParser.Typedef))
+                    elif (not cppType.vars and cppType.methods):
+                        prefix = (cppType.name[1:] if cppType.name[0] == 'I' else cppType.name)
+                        result =  [ "integer", { "size": 32, "signed": False, "@lookupid": prefix } ]
+                        if not [x for x in passed_interfaces if x["name"] == cppType.full_name]:
+                            passed_interfaces.append({ "name": cppType.full_name, "id": "@generate", "type": "uint32_t", "prefix": prefix})
                     else:
                         if cppType.is_iterator:
                             raise CppParseError(var, "iterators must be passed by pointer: %s" % cppType.type)
@@ -737,8 +744,9 @@ def BuildResult(vars, is_property=False, test=False):
             prefix = ""
 
         faces = []
-        faces.append((None, prefix, face.obj.methods))
+        faces.append((prefix, face.obj.methods))
 
+        """
         for method in face.obj.methods:
             if method.retval.meta.lookup:
                 var_type = ResolveTypedef(method.retval.type, method)
@@ -754,9 +762,9 @@ def BuildResult(vars, is_property=False, test=False):
                     schema["@lookups"].append(faces[-1][0][0])
                 else:
                     raise CppParseError(method, "lookup method for an unknown class")
+        """
 
-
-        for lookup_method, prefix, _methods in faces:
+        for prefix, _methods in faces:
           for method in _methods:
 
             if not method.IsVirtual() or method.IsDestructor():
@@ -942,7 +950,7 @@ def BuildResult(vars, is_property=False, test=False):
                 else:
                     raise CppParseError(method, "property method must have one parameter")
 
-            elif not event_params and not method.retval.meta.lookup:
+            elif not event_params:
                 var_type = ResolveTypedef(method.retval.type)
 
                 if var_type and ((isinstance(var_type.Type(), CppParser.Integer) and (var_type.Type().size == "long")) or not verify):
@@ -976,9 +984,6 @@ def BuildResult(vars, is_property=False, test=False):
                 if method.retval.meta.details:
                     obj["description"] = method.retval.meta.details.strip()
 
-                if lookup_method:
-                    obj["@lookup"] = lookup_method
-
                 if method.retval.meta.retval:
                     errors = []
 
@@ -1115,6 +1120,9 @@ def BuildResult(vars, is_property=False, test=False):
 
                     events[prefix + method_name] = obj
 
+        if passed_interfaces:
+            schema["@interfaces"] = passed_interfaces
+
         if methods:
             schema["methods"] = methods
 
@@ -1124,6 +1132,7 @@ def BuildResult(vars, is_property=False, test=False):
         if events:
             schema["events"] = events
 
+
         return schema
 
     schemas = []
@@ -1137,13 +1146,18 @@ def BuildResult(vars, is_property=False, test=False):
                     scanned.append(face.obj.full_name)
 
         for s in schemas:
-            lookups = s["@lookups"] if "@lookups" in s else []
-            for l in lookups:
-                for s2 in schemas:
-                    if StripFrameworkNamespace(s2["@fullname"]) == l:
-                        del s2["@generated"]
-
-        schemas = [s for s in schemas if "@generated" in s]
+            for face in (s["@interfaces"] if "@interfaces" in s else []):
+                for ss in schemas:
+                    if ss["@fullname"] == face["name"] and "methods" in ss:
+                        methods = ss["methods"]
+                        for k,_ in methods.items():
+                            new_k = face["prefix"].lower() + "::" + k
+                            s["methods"][new_k] = methods.pop(k)
+                            s["methods"][new_k]["@lookup"] = face
+                        ss["@generated"] = False
+
+
+        schemas = [s for s in schemas if s.get("@generated")]
 
     return schemas, []
 

JsonGenerator/source/json_loader.py

@@ -975,7 +975,7 @@ def _AddMethods(section, schema, ctor):
         _AddMethods("events", schema, lambda name, obj, method: JsonNotification(name, obj, method))
 
         if not self.methods:
-            raise JsonParseError("no methods, properties or events defined in %s" % self.print_name)
+            raise JsonParseError("no methods, properties or events defined in %s" % self.schema["@fullname"] if "@fullname" in self.schema else self.name)
 
     @property
     def root(self):