allow any whitespace between obj parts

CHANGELOG.md

@@ -5,6 +5,8 @@ The format is based on [Keep a Changelog](http://keepachangelog.com/en/1.0.0/)
 and this project adheres to [Semantic Versioning](http://semver.org/spec/v2.0.0.html).
 
 ## [Unreleased]
+### Fixed
+- allow any whitespace between obj version and rev numbers when parsing PDF (fixes parsing of some PDF)
 
 ## [2.3.5] - 2020-03-03
 ### Fixed

lib/object/name.js

@@ -53,7 +53,7 @@ class PDFName {
         return undefined
       }
 
-      throw new Error('Invalid name')
+      throw new Error('Name must start with a leading slash, found: ' + lexer.getString(1))
     }
 
     lexer.shift(1)

lib/object/object.js

@@ -43,13 +43,13 @@ class PDFObject {
     if (id === undefined && !trial) {
       throw new Error('Invalid object')
     }
-    lexer.skipSpace(1, trial)
+    lexer.skipWhitespace(1, trial)
     const generation = lexer.readNumber(trial)
     if (generation === undefined && !trial) {
       throw new Error('Invalid object')
     }
 
-    lexer.skipSpace(1, trial)
+    lexer.skipWhitespace(1, trial)
     if (lexer.getString(3) !== 'obj') {
       if (trial) {
         lexer.pos = before

lib/object/reference.js

@@ -32,13 +32,13 @@ class PDFReference {
       throw new Error('Invalid indirect')
     }
 
-    lexer.skipSpace(1, trial)
+    lexer.skipWhitespace(1, trial)
     const generation = lexer.readNumber(trial)
     if (generation === undefined && !trial) {
       throw new Error('Invalid indirect')
     }
 
-    lexer.skipSpace(1, trial)
+    lexer.skipWhitespace(1, trial)
     if (lexer.getString(1) !== 'R') {
       if (trial) {
         lexer.pos = before

test/pdfs/parser/google.js

@@ -0,0 +1,11 @@
+const pdf = require('../../../')
+const path = require('path')
+const fs = require('fs')
+
+module.exports = function(doc) {
+  const img = new pdf.Image(fs.readFileSync(path.join(__dirname, '/fixtures/google.pdf')))
+
+  doc.image(img, {
+    wrap: false, x: 0, y: 841.89, width: 595.28
+  })
+}