Throw error when a name table entry is not existing + Improve throw e…

…rror message of Font.download()
jssuttles · Jun 23, 2016 · 00a408d · 00a408d
1 parent 5bd2de0
commit 00a408d
Show file tree

Hide file tree

Showing 2 changed files with 6 additions and 1 deletion.
diff --git a/src/font.js b/src/font.js
@@ -320,7 +320,7 @@ Font.prototype.download = function() {
             });
         },
         function(err) {
-            throw err;
+            throw new Error(err.name + ': ' + err.message);
         });
     } else {
         var fs = require('fs');

diff --git a/src/tables/name.js b/src/tables/name.js
@@ -744,6 +744,11 @@ function makeNameTable(names, ltag) {
         }
 
         nameID = parseInt(id);
+
+        if (isNaN(nameID)) {
+            throw new Error('Name table entry "' + key + '" does not exist, see nameTableNames for complete list.');
+        }
+
         namesWithNumericKeys[nameID] = names[key];
         nameIDs.push(nameID);
     }