js:add @namespace annotation to namespaces
authorJames Gilles <jhgilles@mit.edu>
Mon, 11 Jul 2016 05:13:47 +0000 (01:13 -0400)
committerJames Gilles <jhgilles@mit.edu>
Mon, 11 Jul 2016 05:40:38 +0000 (01:40 -0400)
js/flatbuffers.js
src/idl_gen_js.cpp
tests/monster_test.bfbs
tests/monster_test_generated.js
tests/namespace_test/namespace_test1_generated.js
tests/namespace_test/namespace_test2_generated.js

index 6c3d15c..511c9a2 100644 (file)
@@ -2,6 +2,10 @@
 /// @addtogroup flatbuffers_javascript_api
 /// @{
 /// @cond FLATBUFFERS_INTERNAL
+/**
+ * @const
+ * @namespace
+ */
 var flatbuffers = {};
 
 /**
index 32a06f1..d7c913e 100644 (file)
@@ -110,7 +110,7 @@ class JsGenerator : public BaseGenerator {
   std::string &exports = *exports_ptr;
   for (auto it = sorted_namespaces.begin();
        it != sorted_namespaces.end(); it++) {
-    code += "/**\n * @const\n*/\n";
+    code += "/**\n * @const\n * @namespace\n */\n";
     if (it->find('.') == std::string::npos) {
       code += "var ";
       exports += "this." + *it + " = " + *it + ";\n";
index e131ac8..1e8a270 100644 (file)
Binary files a/tests/monster_test.bfbs and b/tests/monster_test.bfbs differ
index f67e848..7c4c28f 100644 (file)
@@ -2,22 +2,26 @@
 
 /**
  * @const
-*/
+ * @namespace
+ */
 var MyGame = MyGame || {};
 
 /**
  * @const
-*/
+ * @namespace
+ */
 MyGame.Example = MyGame.Example || {};
 
 /**
  * @const
-*/
+ * @namespace
+ */
 MyGame.Example2 = MyGame.Example2 || {};
 
 /**
  * @const
-*/
+ * @namespace
+ */
 MyGame.OtherNameSpace = MyGame.OtherNameSpace || {};
 
 /**
index 769b523..e639056 100644 (file)
@@ -2,12 +2,14 @@
 
 /**
  * @const
-*/
+ * @namespace
+ */
 var NamespaceA = NamespaceA || {};
 
 /**
  * @const
-*/
+ * @namespace
+ */
 NamespaceA.NamespaceB = NamespaceA.NamespaceB || {};
 
 /**
index c195370..f76d5a9 100644 (file)
@@ -2,17 +2,20 @@
 
 /**
  * @const
-*/
+ * @namespace
+ */
 var NamespaceA = NamespaceA || {};
 
 /**
  * @const
-*/
+ * @namespace
+ */
 NamespaceA.NamespaceB = NamespaceA.NamespaceB || {};
 
 /**
  * @const
-*/
+ * @namespace
+ */
 var NamespaceC = NamespaceC || {};
 
 /**