Changeset 3288

Timestamp:

01/05/09 10:11:11 (10 months ago)

Author:

dr2chase

Message:

[repository] native imports generate fake apis, sort of

Location:

trunk/ProjectFortress/src/com/sun/fortress

Files:

• nodes_util/NodeComparator.java (modified) (1 diff)
• repository/ForeignJava.java (modified) (5 diffs)
• repository/GraphRepository.java (modified) (1 diff)

trunk/ProjectFortress/src/com/sun/fortress/nodes_util/NodeComparator.java

@@ -1 +1 @@
 /*******************************************************************************
-    Copyright 2008 Sun Microsystems, Inc.,
+    Copyright 2009 Sun Microsystems, Inc.,
     4150 Network Circle, Santa Clara, California 95054, U.S.A.
     All rights reserved.

trunk/ProjectFortress/src/com/sun/fortress/repository/ForeignJava.java

@@ -46 +46 @@
 import com.sun.fortress.nodes.Param;
 import com.sun.fortress.nodes.StaticParam;
+import com.sun.fortress.nodes.TraitDecl;
 import com.sun.fortress.nodes.TraitObjectDecl;
 import com.sun.fortress.nodes.TraitTypeWhere;
@@ -63 +64 @@
 public class ForeignJava {
 
-    /** given an API Name, what Java class does it import?  (Name of existing
+    /** given an API Name, what Java classes does it import?  (Name of existing
      * Java class, not its wrapper.)
      */
     MultiMap<APIName, Class> javaImplementedAPIs = new MultiMap<APIName, Class>();
+    
+    /**
+     * Given a foreign API Name, what other (foreign) APIs does it import?
+     */
     MultiMap<APIName, APIName> generatedImports = new MultiMap<APIName, APIName>();
     
@@ -84 +89 @@
     MultiMap<Class, String> itemsFromClasses = new MultiMap<Class, String>();
     
+    Map<APIName, ApiIndex> cachedFakeApis = new HashMap<APIName, ApiIndex>();
+    
+    /**
+     * Stores the FnDecl for a particular method -- also acts as an is-visited
+     * marker.
+     */
     Bijection<Method, FnDecl> methodToDecl = new HashBijection<Method, FnDecl>();
-    Bijection<Class, TraitObjectDecl> classToTraitDecl = new HashBijection<Class, TraitObjectDecl>();
+    
+    /**
+     * Stores the TraitDecl for a trait.
+     */
+    Bijection<Class, TraitDecl> classToTraitDecl = new HashBijection<Class, TraitDecl>();
 
     /* Special cases
@@ -275 +290 @@
         List<BaseType> excludesC = Collections.emptyList();
         Option<List<BaseType>> comprisesC = Option.none();
-        TraitObjectDecl tod = NodeFactory.makeTraitDecl (span, mods, name, sparams, extendsC, whereC, decls, excludesC, comprisesC);
+        TraitDecl td = NodeFactory.makeTraitDecl (span, mods, name, sparams, extendsC, whereC, decls, excludesC, comprisesC);
         // Need to fake up an API for this class, too.
-        classToTraitDecl.put(imported_class, tod);
-        apiToStaticDecls.putItem(api_name, tod);
+        classToTraitDecl.put(imported_class, td);
+        apiToStaticDecls.putItem(api_name, td);
         javaImplementedAPIs.putItem(api_name, imported_class);
+        
+        /* Invalidate any old entry.
+         * Let's hope this does not cause problems down the line.
+         */
+        cachedFakeApis.remove(api_name);
     }
     
@@ -366 +386 @@
 
     public ApiIndex fakeApi(APIName name) {
-        Set<Class> classes = javaImplementedAPIs.get(name);
-        // Need to generate wrappers for all these classes,
-        //  if they do not already exist.
-
-        List<Import> imports = new ArrayList<Import>();
-        for (APIName a : generatedImports.get(name)) {
-            AliasedAPIName aan = NodeFactory.makeAliasedAPIName(a);
-            /*
-             * Hoping to lie, slightly, to static analysis.
-             * This is technically speaking a "foreign" import,
-             * but the import is already known to the ForeignJava
-             * data structures, and this allows use of fully qualified
-             * (hence unambiguous) references to classes from other packages
-             * in the generated API.
-             * 
-             * So, one lie -- no foreign annotation.
-             */
-            ImportApi iapi = NodeFactory.makeImportApi(span, Option.<String>none(), Useful.list(aan));
-            imports.add(iapi);
+        ApiIndex result = cachedFakeApis.get(name);
+        if (result == null) {
+
+            Set<Class> classes = javaImplementedAPIs.get(name);
+            // Need to generate wrappers for all these classes,
+            // if they do not already exist.
+
+            List<Import> imports = new ArrayList<Import>();
+            Set<APIName> gi = generatedImports.get(name);
+            if (gi != null)
+            for (APIName a : gi) {
+                AliasedAPIName aan = NodeFactory.makeAliasedAPIName(a);
+                /*
+                 * Hoping to lie, slightly, to static analysis. This is
+                 * technically speaking a "foreign" import, but the import is
+                 * already known to the ForeignJava data structures, and this
+                 * allows use of fully qualified (hence unambiguous) references
+                 * to classes from other packages in the generated API.
+                 * 
+                 * So, one lie -- no foreign annotation.
+                 */
+                ImportApi iapi = NodeFactory.makeImportApi(span, Option
+                        .<String> none(), Useful.list(aan));
+                imports.add(iapi);
+            }
+
+            List<Decl> decls = new ArrayList<Decl>();
+            for (Decl d : apiToStaticDecls.get(name)) {
+                decls.add(d);
+            }
+            Api a = NodeFactory.makeApi(span, name, imports, decls);
+
+            result = IndexBuilder.builder.buildApiIndex(a, Long.MIN_VALUE + 2);
+            cachedFakeApis.put(name, result);
         }
-        
-        List<Decl> decls = new ArrayList<Decl>();
-        for (Decl d : apiToStaticDecls.get(name)) {
-            decls.add(d);
+        return result;
+        
+    }
+    
+    public Map<APIName, ApiIndex> augmentApiMap(Map<APIName, ApiIndex> map) {
+        for (APIName a : javaImplementedAPIs.keySet()) {
+            map.put(a, fakeApi(a));
         }
-        Api a =  NodeFactory.makeApi( span,  name,
-                    imports,
-                    decls);
-        
-        return IndexBuilder.builder.buildApiIndex(a, Long.MIN_VALUE+2);
-        
+        return map;
     }
 

trunk/ProjectFortress/src/com/sun/fortress/repository/GraphRepository.java

@@ -158 +158 @@
     @Override
     public Map<APIName, ApiIndex> apis() {
-        return cache.apis();
+        return foreignJava.augmentApiMap(cache.apis.copy());
     }