Java FFI: enable inserting additionnal Java files to a project
[nit.git] / src / modelize_property.nit
index 09c13e2..5a4ec42 100644 (file)
@@ -48,6 +48,7 @@ redef class ModelBuilder
                var mclassdef = nclassdef.mclassdef.as(not null)
                if mclassdef.in_hierarchy == null then return # Skip error
                for superclassdef in mclassdef.in_hierarchy.direct_greaters do
+                       if not mclassdef2nclassdef.has_key(superclassdef) then continue
                        build_properties(mclassdef2nclassdef[superclassdef])
                end
 
@@ -55,10 +56,10 @@ redef class ModelBuilder
                        npropdef.build_property(self, nclassdef)
                end
                for npropdef in nclassdef.n_propdefs do
-                       npropdef.build_signature(self, nclassdef)
+                       npropdef.build_signature(self)
                end
                for npropdef in nclassdef.n_propdefs do
-                       npropdef.check_signature(self, nclassdef)
+                       npropdef.check_signature(self)
                end
                process_default_constructors(nclassdef)
        end
@@ -83,14 +84,14 @@ redef class ModelBuilder
 
                if not nclassdef isa AStdClassdef then return
 
-               var mmodule = nclassdef.mclassdef.mmodule
+               var mmodule = mclassdef.mmodule
                # Do we inherit for a constructor?
                var combine = new Array[MMethod]
                var inhc: nullable MClass = null
                for st in mclassdef.supertypes do
                        var c = st.mclass
                        if not c.kind.need_init then continue
-                       st = st.anchor_to(mmodule, nclassdef.mclassdef.bound_mtype)
+                       st = st.anchor_to(mmodule, mclassdef.bound_mtype)
                        var candidate = self.try_get_mproperty_by_name2(nclassdef, mmodule, st, "init").as(nullable MMethod)
                        if candidate != null and candidate.intro.msignature.arity == 0 then
                                combine.add(candidate)
@@ -264,9 +265,9 @@ redef class APropdef
        var mpropdef: nullable MPROPDEF writable
 
        private fun build_property(modelbuilder: ModelBuilder, nclassdef: AClassdef) is abstract
-       private fun build_signature(modelbuilder: ModelBuilder, nclassdef: AClassdef) is abstract
-       private fun check_signature(modelbuilder: ModelBuilder, nclassdef: AClassdef) is abstract
-       private fun new_property_visibility(modelbuilder: ModelBuilder, nclassdef: AClassdef, nvisibility: nullable AVisibility): MVisibility
+       private fun build_signature(modelbuilder: ModelBuilder) is abstract
+       private fun check_signature(modelbuilder: ModelBuilder) is abstract
+       private fun new_property_visibility(modelbuilder: ModelBuilder, mclassdef: MClassDef, nvisibility: nullable AVisibility): MVisibility
        do
                var mvisibility = public_visibility
                if nvisibility != null then
@@ -276,7 +277,7 @@ redef class APropdef
                                mvisibility = public_visibility
                        end
                end
-               if nclassdef.mclassdef.mclass.visibility == private_visibility then
+               if mclassdef.mclass.visibility == private_visibility then
                        if mvisibility == protected_visibility then
                                assert nvisibility != null
                                modelbuilder.error(nvisibility, "Error: The only legal visibility for properties in a private class is private.")
@@ -300,7 +301,7 @@ redef class APropdef
                end
        end
 
-       private fun check_redef_property_visibility(modelbuilder: ModelBuilder, nclassdef: AClassdef, nvisibility: nullable AVisibility, mprop: MProperty)
+       private fun check_redef_property_visibility(modelbuilder: ModelBuilder, nvisibility: nullable AVisibility, mprop: MProperty)
        do
                if nvisibility == null then return
                var mvisibility = nvisibility.mvisibility
@@ -347,15 +348,16 @@ redef class ASignature
        var ret_type: nullable MType = null
 
        # Visit and fill information about a signature
-       private fun visit_signature(modelbuilder: ModelBuilder, nclassdef: AClassdef): Bool
+       private fun visit_signature(modelbuilder: ModelBuilder, mclassdef: MClassDef): Bool
        do
+               var mmodule = mclassdef.mmodule
                var param_names = self.param_names
                var param_types = self.param_types
                for np in self.n_params do
                        param_names.add(np.n_id.text)
                        var ntype = np.n_type
                        if ntype != null then
-                               var mtype = modelbuilder.resolve_mtype(nclassdef, ntype)
+                               var mtype = modelbuilder.resolve_mtype(mmodule, mclassdef, ntype)
                                if mtype == null then return false # Skip error
                                for i in [0..param_names.length-param_types.length[ do
                                        param_types.add(mtype)
@@ -372,7 +374,7 @@ redef class ASignature
                end
                var ntype = self.n_type
                if ntype != null then
-                       self.ret_type = modelbuilder.resolve_mtype(nclassdef, ntype)
+                       self.ret_type = modelbuilder.resolve_mtype(mmodule, mclassdef, ntype)
                        if self.ret_type == null then return false # Skip errir
                end
 
@@ -381,7 +383,7 @@ redef class ASignature
        end
 
        # Build a visited signature
-       fun build_signature(modelbuilder: ModelBuilder, nclassdef: AClassdef): nullable MSignature
+       fun build_signature(modelbuilder: ModelBuilder): nullable MSignature
        do
                if param_names.length != param_types.length then
                        # Some parameters are typed, other parameters are not typed.
@@ -447,7 +449,7 @@ redef class AMethPropdef
                var mprop: nullable MMethod = null
                if not is_init or n_kwredef != null then mprop = modelbuilder.try_get_mproperty_by_name(name_node, mclassdef, name).as(nullable MMethod)
                if mprop == null then
-                       var mvisibility = new_property_visibility(modelbuilder, nclassdef, self.n_visibility)
+                       var mvisibility = new_property_visibility(modelbuilder, mclassdef, self.n_visibility)
                        mprop = new MMethod(mclassdef, name, mvisibility)
                        mprop.is_init = is_init
                        mprop.is_new = n_kwnew != null
@@ -460,7 +462,7 @@ redef class AMethPropdef
                                        if not self.check_redef_keyword(modelbuilder, nclassdef, n_kwredef, true, mprop) then return
                                end
                        end
-                       check_redef_property_visibility(modelbuilder, nclassdef, self.n_visibility, mprop)
+                       check_redef_property_visibility(modelbuilder, self.n_visibility, mprop)
                end
                nclassdef.mprop2npropdef[mprop] = self
 
@@ -491,11 +493,12 @@ redef class AMethPropdef
                end
        end
 
-       redef fun build_signature(modelbuilder, nclassdef)
+       redef fun build_signature(modelbuilder)
        do
                var mpropdef = self.mpropdef
                if mpropdef == null then return # Error thus skiped
-               var mmodule = mpropdef.mclassdef.mmodule
+               var mclassdef = mpropdef.mclassdef
+               var mmodule = mclassdef.mmodule
                var nsig = self.n_signature
 
                # Retrieve info from the signature AST
@@ -504,7 +507,7 @@ redef class AMethPropdef
                var vararg_rank = -1
                var ret_type: nullable MType = null # Return type from the AST
                if nsig != null then
-                       if not nsig.visit_signature(modelbuilder, nclassdef) then return
+                       if not nsig.visit_signature(modelbuilder, mclassdef) then return
                        param_names = nsig.param_names
                        param_types = nsig.param_types
                        vararg_rank = nsig.vararg_rank
@@ -527,8 +530,8 @@ redef class AMethPropdef
                        end
                else if mpropdef.mproperty.is_init then
                        # FIXME UGLY: inherit signature from a super-constructor
-                       for msupertype in nclassdef.mclassdef.supertypes do
-                               msupertype = msupertype.anchor_to(mmodule, nclassdef.mclassdef.bound_mtype)
+                       for msupertype in mclassdef.supertypes do
+                               msupertype = msupertype.anchor_to(mmodule, mclassdef.bound_mtype)
                                var candidate = modelbuilder.try_get_mproperty_by_name2(self, mmodule, msupertype, mpropdef.mproperty.name)
                                if candidate != null then
                                        if msignature == null then
@@ -572,11 +575,12 @@ redef class AMethPropdef
                mpropdef.is_extern = self isa AExternPropdef
        end
 
-       redef fun check_signature(modelbuilder, nclassdef)
+       redef fun check_signature(modelbuilder)
        do
                var mpropdef = self.mpropdef
                if mpropdef == null then return # Error thus skiped
-               var mmodule = mpropdef.mclassdef.mmodule
+               var mclassdef = mpropdef.mclassdef
+               var mmodule = mclassdef.mmodule
                var nsig = self.n_signature
                var mysignature = self.mpropdef.msignature
                if mysignature == null then return # Error thus skiped
@@ -599,8 +603,8 @@ redef class AMethPropdef
                                for i in [0..mysignature.arity[ do
                                        var myt = mysignature.mparameters[i].mtype
                                        var prt = msignature.mparameters[i].mtype
-                                       if not myt.is_subtype(mmodule, nclassdef.mclassdef.bound_mtype, prt) or
-                                                       not prt.is_subtype(mmodule, nclassdef.mclassdef.bound_mtype, myt) then
+                                       if not myt.is_subtype(mmodule, mclassdef.bound_mtype, prt) or
+                                                       not prt.is_subtype(mmodule, mclassdef.bound_mtype, myt) then
                                                modelbuilder.error(nsig.n_params[i], "Redef Error: Wrong type for parameter `{mysignature.mparameters[i].name}'. found {myt}, expected {prt} as in {mpropdef.mproperty.intro}.")
                                        end
                                end
@@ -609,7 +613,7 @@ redef class AMethPropdef
                                if ret_type == null then
                                        # Inherit the return type
                                        ret_type = precursor_ret_type
-                               else if not ret_type.is_subtype(mmodule, nclassdef.mclassdef.bound_mtype, precursor_ret_type) then
+                               else if not ret_type.is_subtype(mmodule, mclassdef.bound_mtype, precursor_ret_type) then
                                        modelbuilder.error(nsig.n_type.as(not null), "Redef Error: Wrong return type. found {ret_type}, expected {precursor_ret_type} as in {mpropdef.mproperty.intro}.")
                                end
                        end
@@ -659,12 +663,12 @@ redef class AAttrPropdef
                        # Old attribute style
                        var mprop = modelbuilder.try_get_mproperty_by_name(nid, mclassdef, name)
                        if mprop == null then
-                               var mvisibility = new_property_visibility(modelbuilder, nclassdef, self.n_visibility)
+                               var mvisibility = new_property_visibility(modelbuilder, mclassdef, self.n_visibility)
                                mprop = new MAttribute(mclassdef, name, mvisibility)
                                if not self.check_redef_keyword(modelbuilder, nclassdef, self.n_kwredef, false, mprop) then return
                        else
                                assert mprop isa MAttribute
-                               check_redef_property_visibility(modelbuilder, nclassdef, self.n_visibility, mprop)
+                               check_redef_property_visibility(modelbuilder, self.n_visibility, mprop)
                                if not self.check_redef_keyword(modelbuilder, nclassdef, self.n_kwredef, true, mprop) then return
                        end
                        nclassdef.mprop2npropdef[mprop] = self
@@ -679,12 +683,12 @@ redef class AAttrPropdef
                                var readname = name.substring_from(1)
                                var mreadprop = modelbuilder.try_get_mproperty_by_name(nid, mclassdef, readname).as(nullable MMethod)
                                if mreadprop == null then
-                                       var mvisibility = new_property_visibility(modelbuilder, nclassdef, nreadable.n_visibility)
+                                       var mvisibility = new_property_visibility(modelbuilder, mclassdef, nreadable.n_visibility)
                                        mreadprop = new MMethod(mclassdef, readname, mvisibility)
                                        if not self.check_redef_keyword(modelbuilder, nclassdef, nreadable.n_kwredef, false, mreadprop) then return
                                else
                                        if not self.check_redef_keyword(modelbuilder, nclassdef, nreadable.n_kwredef, true, mreadprop) then return
-                                       check_redef_property_visibility(modelbuilder, nclassdef, nreadable.n_visibility, mreadprop)
+                                       check_redef_property_visibility(modelbuilder, nreadable.n_visibility, mreadprop)
                                end
                                nclassdef.mprop2npropdef[mreadprop] = self
 
@@ -699,12 +703,12 @@ redef class AAttrPropdef
                                var writename = name.substring_from(1) + "="
                                var mwriteprop = modelbuilder.try_get_mproperty_by_name(nid, mclassdef, writename).as(nullable MMethod)
                                if mwriteprop == null then
-                                       var mvisibility = new_property_visibility(modelbuilder, nclassdef, nwritable.n_visibility)
+                                       var mvisibility = new_property_visibility(modelbuilder, mclassdef, nwritable.n_visibility)
                                        mwriteprop = new MMethod(mclassdef, writename, mvisibility)
                                        if not self.check_redef_keyword(modelbuilder, nclassdef, nwritable.n_kwredef, false, mwriteprop) then return
                                else
                                        if not self.check_redef_keyword(modelbuilder, nclassdef, nwritable.n_kwredef, true, mwriteprop) then return
-                                       check_redef_property_visibility(modelbuilder, nclassdef, nwritable.n_visibility, mwriteprop)
+                                       check_redef_property_visibility(modelbuilder, nwritable.n_visibility, mwriteprop)
                                end
                                nclassdef.mprop2npropdef[mwriteprop] = self
 
@@ -725,12 +729,12 @@ redef class AAttrPropdef
                        var readname = name
                        var mreadprop = modelbuilder.try_get_mproperty_by_name(nid2, mclassdef, readname).as(nullable MMethod)
                        if mreadprop == null then
-                               var mvisibility = new_property_visibility(modelbuilder, nclassdef, self.n_visibility)
+                               var mvisibility = new_property_visibility(modelbuilder, mclassdef, self.n_visibility)
                                mreadprop = new MMethod(mclassdef, readname, mvisibility)
                                if not self.check_redef_keyword(modelbuilder, nclassdef, n_kwredef, false, mreadprop) then return
                        else
                                if not self.check_redef_keyword(modelbuilder, nclassdef, n_kwredef, true, mreadprop) then return
-                               check_redef_property_visibility(modelbuilder, nclassdef, self.n_visibility, mreadprop)
+                               check_redef_property_visibility(modelbuilder, self.n_visibility, mreadprop)
                        end
                        nclassdef.mprop2npropdef[mreadprop] = self
 
@@ -747,7 +751,7 @@ redef class AAttrPropdef
                        if mwriteprop == null then
                                var mvisibility
                                if nwritable != null then
-                                       mvisibility = new_property_visibility(modelbuilder, nclassdef, nwritable.n_visibility)
+                                       mvisibility = new_property_visibility(modelbuilder, mclassdef, nwritable.n_visibility)
                                else
                                        mvisibility = private_visibility
                                end
@@ -756,7 +760,7 @@ redef class AAttrPropdef
                        else
                                if not self.check_redef_keyword(modelbuilder, nclassdef, nwkwredef, true, mwriteprop) then return
                                if nwritable != null then
-                                       check_redef_property_visibility(modelbuilder, nclassdef, nwritable.n_visibility, mwriteprop)
+                                       check_redef_property_visibility(modelbuilder, nwritable.n_visibility, mwriteprop)
                                end
                        end
                        nclassdef.mprop2npropdef[mwriteprop] = self
@@ -768,16 +772,17 @@ redef class AAttrPropdef
                end
        end
 
-       redef fun build_signature(modelbuilder, nclassdef)
+       redef fun build_signature(modelbuilder)
        do
                var mpropdef = self.mpropdef
                if mpropdef == null then return # Error thus skiped
-               var mmodule = mpropdef.mclassdef.mmodule
+               var mclassdef = mpropdef.mclassdef
+               var mmodule = mclassdef.mmodule
                var mtype: nullable MType = null
 
                var ntype = self.n_type
                if ntype != null then
-                       mtype = modelbuilder.resolve_mtype(nclassdef, ntype)
+                       mtype = modelbuilder.resolve_mtype(mmodule, mclassdef, ntype)
                        if mtype == null then return
                end
 
@@ -785,7 +790,7 @@ redef class AAttrPropdef
                if mtype == null then
                        if nexpr != null then
                                if nexpr isa ANewExpr then
-                                       mtype = modelbuilder.resolve_mtype(nclassdef, nexpr.n_type)
+                                       mtype = modelbuilder.resolve_mtype(mmodule, mclassdef, nexpr.n_type)
                                else if nexpr isa AIntExpr then
                                        var cla = modelbuilder.try_get_mclass_by_name(nexpr, mmodule, "Int")
                                        if cla != null then mtype = cla.mclass_type
@@ -814,7 +819,7 @@ redef class AAttrPropdef
                else
                        assert ntype != null
                        if nexpr isa ANewExpr then
-                               var xmtype = modelbuilder.resolve_mtype(nclassdef, nexpr.n_type)
+                               var xmtype = modelbuilder.resolve_mtype(mmodule, mclassdef, nexpr.n_type)
                                if xmtype == mtype and modelbuilder.toolcontext.opt_warn.value >= 2 then
                                        modelbuilder.warning(ntype, "Warning: useless type definition")
                                end
@@ -845,11 +850,12 @@ redef class AAttrPropdef
                end
        end
 
-       redef fun check_signature(modelbuilder, nclassdef)
+       redef fun check_signature(modelbuilder)
        do
                var mpropdef = self.mpropdef
                if mpropdef == null then return # Error thus skiped
-               var mmodule = mpropdef.mclassdef.mmodule
+               var mclassdef = mpropdef.mclassdef
+               var mmodule = mclassdef.mmodule
                var ntype = self.n_type
                var mtype = self.mpropdef.static_mtype
                if mtype == null then return # Error thus skiped
@@ -869,23 +875,24 @@ redef class AAttrPropdef
                # Check getter and setter
                var meth = self.mreadpropdef
                if meth != null then
-                       self.check_method_signature(modelbuilder, nclassdef, meth)
+                       self.check_method_signature(modelbuilder, meth)
                        var node: nullable ANode = ntype
                        if node == null then node = self
                        modelbuilder.check_visibility(node, mtype, meth)
                end
                meth = self.mwritepropdef
                if meth != null then
-                       self.check_method_signature(modelbuilder, nclassdef, meth)
+                       self.check_method_signature(modelbuilder, meth)
                        var node: nullable ANode = ntype
                        if node == null then node = self
                        modelbuilder.check_visibility(node, mtype, meth)
                end
        end
 
-       private fun check_method_signature(modelbuilder: ModelBuilder, nclassdef: AClassdef, mpropdef: MMethodDef)
+       private fun check_method_signature(modelbuilder: ModelBuilder, mpropdef: MMethodDef)
        do
-               var mmodule = mpropdef.mclassdef.mmodule
+               var mclassdef = mpropdef.mclassdef
+               var mmodule = mclassdef.mmodule
                var nsig = self.n_type
                var mysignature = mpropdef.msignature
                if mysignature == null then return # Error thus skiped
@@ -916,8 +923,8 @@ redef class AAttrPropdef
                                for i in [0..mysignature.arity[ do
                                        var myt = mysignature.mparameters[i].mtype
                                        var prt = msignature.mparameters[i].mtype
-                                       if not myt.is_subtype(mmodule, nclassdef.mclassdef.bound_mtype, prt) and
-                                                       not prt.is_subtype(mmodule, nclassdef.mclassdef.bound_mtype, myt) then
+                                       if not myt.is_subtype(mmodule, mclassdef.bound_mtype, prt) and
+                                                       not prt.is_subtype(mmodule, mclassdef.bound_mtype, myt) then
                                                var node: ANode
                                                if nsig != null then node = nsig else node = self
                                                modelbuilder.error(node, "Redef Error: Wrong type for parameter `{mysignature.mparameters[i].name}'. found {myt}, expected {prt}.")
@@ -928,7 +935,7 @@ redef class AAttrPropdef
                                if ret_type == null then
                                        # Inherit the return type
                                        ret_type = precursor_ret_type
-                               else if not ret_type.is_subtype(mmodule, nclassdef.mclassdef.bound_mtype, precursor_ret_type) then
+                               else if not ret_type.is_subtype(mmodule, mclassdef.bound_mtype, precursor_ret_type) then
                                        var node: ANode
                                        if nsig != null then node = nsig else node = self
                                        modelbuilder.error(node, "Redef Error: Wrong return type. found {ret_type}, expected {precursor_ret_type}.")
@@ -947,7 +954,7 @@ redef class ATypePropdef
                var name = self.n_id.text
                var mprop = modelbuilder.try_get_mproperty_by_name(self.n_id, mclassdef, name)
                if mprop == null then
-                       var mvisibility = new_property_visibility(modelbuilder, nclassdef, self.n_visibility)
+                       var mvisibility = new_property_visibility(modelbuilder, mclassdef, self.n_visibility)
                        mprop = new MVirtualTypeProp(mclassdef, name, mvisibility)
                        for c in name.chars do if c >= 'a' and c<= 'z' then
                                modelbuilder.warning(n_id, "Warning: lowercase in the virtual type {name}")
@@ -957,7 +964,7 @@ redef class ATypePropdef
                else
                        if not self.check_redef_keyword(modelbuilder, nclassdef, self.n_kwredef, true, mprop) then return
                        assert mprop isa MVirtualTypeProp
-                       check_redef_property_visibility(modelbuilder, nclassdef, self.n_visibility, mprop)
+                       check_redef_property_visibility(modelbuilder, self.n_visibility, mprop)
                end
                nclassdef.mprop2npropdef[mprop] = self
 
@@ -967,22 +974,23 @@ redef class ATypePropdef
                set_doc(mpropdef)
        end
 
-       redef fun build_signature(modelbuilder, nclassdef)
+       redef fun build_signature(modelbuilder)
        do
                var mpropdef = self.mpropdef
                if mpropdef == null then return # Error thus skiped
-               var mmodule = mpropdef.mclassdef.mmodule
+               var mclassdef = mpropdef.mclassdef
+               var mmodule = mclassdef.mmodule
                var mtype: nullable MType = null
 
                var ntype = self.n_type
-               mtype = modelbuilder.resolve_mtype(nclassdef, ntype)
+               mtype = modelbuilder.resolve_mtype(mmodule, mclassdef, ntype)
                if mtype == null then return
 
                mpropdef.bound = mtype
                # print "{mpropdef}: {mtype}"
        end
 
-       redef fun check_signature(modelbuilder, nclassdef)
+       redef fun check_signature(modelbuilder)
        do
                var mpropdef = self.mpropdef
                if mpropdef == null then return # Error thus skiped
@@ -995,8 +1003,9 @@ redef class ATypePropdef
                # Fast case: the bound is not a formal type
                if not bound isa MVirtualType then return
 
-               var mmodule = nclassdef.mclassdef.mmodule
-               var anchor = nclassdef.mclassdef.bound_mtype
+               var mclassdef = mpropdef.mclassdef
+               var mmodule = mclassdef.mmodule
+               var anchor = mclassdef.bound_mtype
 
                # Slow case: progress on each resolution until: (i) we loop, or (ii) we found a non formal type
                var seen = [self.mpropdef.mproperty.mvirtualtype]