full support interface inheritance

src/bindx/BindMacros.hx

@@ -4,11 +4,13 @@ import bindx.GenericError;
 import haxe.macro.Type;
 import haxe.macro.Expr;
 import haxe.macro.Context;
+import haxe.macro.TypeTools;
 
 using haxe.macro.Tools;
 using Lambda;
 using StringTools;
 using bindx.MetaUtils;
+using haxe.macro.Tools;
 
 class BindMacros {
     #if macro
@@ -24,21 +26,25 @@ class BindMacros {
      */
     static public inline var FORCE = "force";
 
-	static var processed:Array<Type> = [];
+	static var processed:Array<String> = [];
 
     static var bindingSignalProvider:IBindingSignalProvider;
     
+    static var interfaceBindableFields:Map<String, Array<String>>;
+
     macro static public function buildIBindable():Array<Field> {
         var type = Context.getLocalType();
-        if (processed.indexOf(type) > -1) {
+        var tName = type.toComplexType().toString();
+        if (processed.indexOf(tName) > -1) {
             return null;
         }
-        processed.push(type);
+        processed.push(tName);
 
         var classType = type.getClass();
         
         if (bindingSignalProvider == null) {
             bindingSignalProvider = new bindx.BindSignal.BindSignalProvider();
+            interfaceBindableFields = new Map();
         }
         
         var fields = Context.getBuildFields();
@@ -47,12 +53,15 @@ class BindMacros {
         if (meta != null) injectBindableMeta(fields, meta);
         
         if (classType.isInterface) {
+            var a = [];
             for (f in fields) {
                 for (m in f.meta) if (m.name == MetaUtils.BINDABLE_META) {
+                    a.push(f.name);
                     if (m.params.length > 0)
                         Context.warning('Interface doesn\'t support @:bindable meta params', m.pos);
                 }
             }
+            interfaceBindableFields.set(typeName(classType), a);
             return fields;
         }
         
@@ -79,16 +88,17 @@ class BindMacros {
     
     static function getBindableFieldsFromInterfaces(classType:ClassType):Map<String, ClassType> {
         var interfaceFields = new Map();
-        for (i in classType.interfaces) {
+        
-            var t = i.t.get();
+        function iter(t:ClassType) {
-            if (@:privateAccess Bind.isBindable(t)) {
+            var a = interfaceBindableFields.get(typeName(t));
-                for (f in t.fields.get()) {
+            if (a != null) {
-                    if (f.meta.has(MetaUtils.BINDABLE_META)) {
+                for (i in a)
-                        interfaceFields.set(f.name, t);
+                    interfaceFields.set(i, t);
-                    }
-                }
             }
+            for (it in t.interfaces) iter(it.t.get());
         }
+        
+        for (i in classType.interfaces) iter(i.t.get());
         return interfaceFields;
     }
 

test/InheritanceTest.hx

@@ -46,7 +46,9 @@ interface IIBindable extends IBindable {
     var i(default, set):Int;
 }
 
-class BindableParent implements IIBindable {
+interface IIIBindable extends IIBindable { }
+
+class BindableParent implements IIIBindable {
 	public function new() {}
     
     @:bindable
@@ -54,7 +56,7 @@ class BindableParent implements IIBindable {
 }
 
 @:bindable
-class BindableChild extends BindableParent {
+class BindableChild extends BindableParent implements IIBindable {
 	
 	public var s:String;
 }