Merge pull request #515 from dontcallmedom/doublecomma-295

Avoid double commas bug in WebIDL oldschool
w3c · Nov 4, 2015 · 533f3db · 533f3db
2 parents bbb924b + 2f4a589
commit 533f3db
Showing 1 changed file with 2 additions and 2 deletions.
diff --git a/js/core/webidl-oldschool.js b/js/core/webidl-oldschool.js
@@ -891,7 +891,7 @@ define(
                         else {
                             sn.element("a", {}, span, it.isUnionType ? "(" + it.datatype.join(" or ") + ")" : it.datatype);
                         }
-                        if (it.declaration) sn.text(", " + it.declaration, dt);
+                        if (it.declaration.trim()) sn.text(", " + it.declaration, dt);
                         if (it.nullable) sn.text(", nullable", dt);
                          if (this.conf.idlOldStyleExceptions && it.raises.length) {
                             var table = sn.element("table", { "class": "exceptions" }, desc);
@@ -1092,7 +1092,7 @@ define(
                         else {
                             sn.element("a", {}, span, it.isUnionType ? "(" + it.datatype.join(" or ") + ")" : it.datatype);
                         }
-                        if (it.declaration) sn.text(", " + it.declaration, dt);
+                        if (it.declaration.trim()) sn.text(", " + it.declaration, dt);
                         if (it.nullable) sn.text(", nullable", dt);
                         if (it.defaultValue) {
                             sn.text(", defaulting to ", dt);