From 5c4f26019e662e553cd402cba4ccd71a4ff450be Mon Sep 17 00:00:00 2001 From: Vivien Nicolas <21@vingtetun.org> Date: Mon, 20 Jun 2011 03:01:09 +0200 Subject: [PATCH] Check for more missing tables in TrueType (sigh) --- fonts.js | 9 ++++----- 1 file changed, 4 insertions(+), 5 deletions(-) diff --git a/fonts.js b/fonts.js index 281eb94a8..72fabd2e3 100644 --- a/fonts.js +++ b/fonts.js @@ -694,7 +694,7 @@ var TrueType = function(aName, aFile, aProperties) { // If any tables are still in the array this means some required tables are // missing, which means that we need to rebuild the font in order to pass // the sanitizer. - if (requiredTables.length && requiredTables[0] == "OS/2") { + if (requiredTables.length == 1 && requiredTables[0] == "OS/2") { var OS2 = [ 0x00, 0x03, // version 0x02, 0x24, // xAvgCharWidth @@ -812,11 +812,10 @@ var TrueType = function(aName, aFile, aProperties) { this.data = fontData; return; - } else if (requiredTables.lenght) { - error("Table " + requiredTables[0] + " is missing from the TrueType font"); - } else { - this.data = aFile; + } else if (requiredTables.length) { + warn("Missing " + requiredTables + " in the TrueType font"); } + this.data = aFile; }; TrueType.prototype = {