more todo notes

Author: Hongbo Zhang

jscomp/test/module_as_class_ffi.js

@@ -1,10 +1,16 @@
 'use strict';
 
-var Mk = require("xx/foo_class");
+var Xx$slashfoo_class = require("xx/foo_class");
+var Mk                = require("xx/foo_class");
 
 function f() {
   return new Mk(3);
 }
 
+function v() {
+  return Xx$slashfoo_class.ff(3);
+}
+
 exports.f = f;
+exports.v = v;
 /* xx/foo_class Not a pure module */

jscomp/test/module_as_class_ffi.ml

@@ -5,3 +5,19 @@ external mk : int -> t = "xx/foo_class" [@@bs.new] [@@bs.module]
 
 let f () =
   mk 3  
+
+(*
+TODO: more error checking
+1. [@@bs.module] can only be used once
+2. here [bs.new] should not have any payload
+3. consolidate all [bs.module] 
+external mk2 : int -> t = "xx/foo_class" [@@bs.new "x"] [@@bs.module]
+
+let ff () =
+  mk2 3  
+*)
+
+external ff : int -> t = "" [@@bs.module "xx/foo_class"]
+
+let v () =
+  ff 3