Skip to content

Commit

Permalink
Removed Extraneous Matrix Check in CalRGB Conversion
Browse files Browse the repository at this point in the history
  • Loading branch information
brianholleibm committed Aug 2, 2018
1 parent 716acf6 commit 2a665eb
Show file tree
Hide file tree
Showing 4 changed files with 8 additions and 12 deletions.
12 changes: 0 additions & 12 deletions src/core/colorspace.js
Original file line number Diff line number Diff line change
Expand Up @@ -960,18 +960,6 @@ var CalRGBCS = (function CalRGBCSClosure() {
'] for ' + this.name + ', falling back to default');
this.GR = this.GG = this.GB = 1;
}

if (this.MXA < 0 || this.MYA < 0 || this.MZA < 0 ||
this.MXB < 0 || this.MYB < 0 || this.MZB < 0 ||
this.MXC < 0 || this.MYC < 0 || this.MZC < 0) {
info('Invalid Matrix for ' + this.name + ' [' +
this.MXA + ', ' + this.MYA + ', ' + this.MZA +
this.MXB + ', ' + this.MYB + ', ' + this.MZB +
this.MXC + ', ' + this.MYC + ', ' + this.MZC +
'], falling back to default');
this.MXA = this.MYB = this.MZC = 1;
this.MXB = this.MYA = this.MZA = this.MXC = this.MYC = this.MZB = 0;
}
}

function matrixProduct(a, b, result) {
Expand Down
1 change: 1 addition & 0 deletions test/pdfs/.gitignore
Original file line number Diff line number Diff line change
Expand Up @@ -71,6 +71,7 @@
!issue9418.pdf
!issue9458.pdf
!issue9915_reduced.pdf
!issue9940.pdf
!bad-PageLabels.pdf
!decodeACSuccessive.pdf
!filled-background.pdf
Expand Down
Binary file added test/pdfs/issue9940.pdf
Binary file not shown.
7 changes: 7 additions & 0 deletions test/test_manifest.json
Original file line number Diff line number Diff line change
Expand Up @@ -3978,5 +3978,12 @@
"md5": "ce718efe601cd7491dd00651b4790329",
"rounds": 1,
"type": "eq"
},
{ "id": "issue9940",
"file": "pdfs/issue9940.pdf",
"md5": "6ffef210c4b6cfe423e20430d8af168a",
"rounds": 1,
"link": false,
"type": "eq"
}
]

0 comments on commit 2a665eb

Please sign in to comment.