[Pkg-javascript-commits] [pdf.js] 57/119: Don't map glyphs to certain problematic General Punctuation Unicode locations (bug 911034)

[David Prévot]

This is an automated email from the git hooks/post-receive script.

taffit pushed a commit to branch master
in repository pdf.js.

commit fda858ae336417fdb4936ce5ff28059df79623de
Author: Jonas Jenwald <jonas.jenwald at gmail.com>
Date:   Thu Apr 9 16:09:24 2015 +0200

    Don't map glyphs to certain problematic General Punctuation Unicode locations (bug 911034)
    
    Fixes the remaining missing characters in https://bugzilla.mozilla.org/show_bug.cgi?id=911034.
    
    For reference, see http://www.unicode.org/charts/PDF/U2000.pdf (and also http://en.wikipedia.org/wiki/General_Punctuation_%28Unicode_block%29).
---
 src/core/fonts.js       |   7 +++++++
 test/pdfs/.gitignore    |   1 +
 test/pdfs/bug911034.pdf | Bin 0 -> 56514 bytes
 test/test_manifest.json |   7 +++++++
 4 files changed, 15 insertions(+)

diff --git a/src/core/fonts.js b/src/core/fonts.js
index 57d1328..ba025ed 100644
--- a/src/core/fonts.js
+++ b/src/core/fonts.js
@@ -2666,6 +2666,11 @@ var Font = (function FontClosure() {
     if (code >= 0x80 && code <= 0x9F) { // Control chars
       return true;
     }
+    if ((code >= 0x2000 && code <= 0x200F) || // General punctuation chars
+        (code >= 0x2028 && code <= 0x202F) ||
+        (code >= 0x2060 && code <= 0x206F)) {
+      return true;
+    }
     if (code >= 0xFFF0 && code <= 0xFFFF) { // Specials Unicode block
       return true;
     }
@@ -2674,6 +2679,8 @@ var Font = (function FontClosure() {
       case 0xA0: // Non breaking space
       case 0xAD: // Soft hyphen
       case 0x0E33: // Thai character SARA AM
+      case 0x2011: // Non breaking hyphen
+      case 0x205F: // Medium mathematical space
       case 0x25CC: // Dotted circle (combining mark)
         return true;
     }
diff --git a/test/pdfs/.gitignore b/test/pdfs/.gitignore
index a744db6..d65b22c 100644
--- a/test/pdfs/.gitignore
+++ b/test/pdfs/.gitignore
@@ -30,6 +30,7 @@
 !bug847420.pdf
 !bug860632.pdf
 !bug894572.pdf
+!bug911034.pdf
 !bug1108301.pdf
 !pdfjsbad1586.pdf
 !freeculture.pdf
diff --git a/test/pdfs/bug911034.pdf b/test/pdfs/bug911034.pdf
new file mode 100644
index 0000000..344161c
Binary files /dev/null and b/test/pdfs/bug911034.pdf differ
diff --git a/test/test_manifest.json b/test/test_manifest.json
index 4ddf669..5a51151 100644
--- a/test/test_manifest.json
+++ b/test/test_manifest.json
@@ -52,6 +52,13 @@
        "type": "eq",
        "about": "Fonts referenced only by name and not by an object identifier."
     },
+    {  "id": "bug911034",
+       "file": "pdfs/bug911034.pdf",
+       "md5": "54ee432a4e16b26b242fbf549cdad177",
+       "rounds": 1,
+       "link": false,
+       "type": "eq"
+    },
     {  "id": "bug921760",
        "file": "pdfs/bug921760.pdf",
        "md5": "1aa136d786a65b0d7cce7bdb3c58c6c3",

-- 
Alioth's /usr/local/bin/git-commit-notice on /srv/git.debian.org/git/pkg-javascript/pdf.js.git