CCITTFaxStream: fix regression when EncodedByteAlign is true and EndOfLine is false #5597
Conversation
timvandermeij
force-pushed
the
ccitt-regression
branch
from
6da2949 to
74c5044
Compare
|
/botio-linux preview |
From: Bot.io (Linux)ReceivedCommand cmd_preview from @timvandermeij received. Current queue size: 0 Live output at: http://107.21.233.14:8877/343c0df86ea76f8/output.txt |
From: Bot.io (Linux)SuccessFull output at http://107.21.233.14:8877/343c0df86ea76f8/output.txt Total script time: 0.78 mins Published
|
|
/botio test |
From: Bot.io (Linux)ReceivedCommand cmd_test from @timvandermeij received. Current queue size: 0 Live output at: http://107.21.233.14:8877/1f6d1dfebc34556/output.txt |
From: Bot.io (Windows)ReceivedCommand cmd_test from @timvandermeij received. Current queue size: 0 Live output at: http://107.22.172.223:8877/5a52587006d6c05/output.txt |
timvandermeij
changed the title
timvandermeij
force-pushed
the
ccitt-regression
branch
from
74c5044 to
b835ef4
Compare
|
Note: After I ran the tests, I have amended the commit message. Nothing has been changed in the commit itself. |
From: Bot.io (Windows)SuccessFull output at http://107.22.172.223:8877/5a52587006d6c05/output.txt Total script time: 17.59 mins
|
From: Bot.io (Linux)SuccessFull output at http://107.21.233.14:8877/1f6d1dfebc34556/output.txt Total script time: 23.10 mins
|
|
change looks trivial. it passes the PDFs the original PR addressed, passes regression tests where the new PDF was added. |
|
Thanks for the patch! /botio makeref |
From: Bot.io (Linux)ReceivedCommand cmd_makeref from @Snuffleupagus received. Current queue size: 0 Live output at: http://107.21.233.14:8877/07fe09dc49142d3/output.txt |
From: Bot.io (Windows)ReceivedCommand cmd_makeref from @Snuffleupagus received. Current queue size: 0 Live output at: http://107.22.172.223:8877/02a53d4fef97ea8/output.txt |
From: Bot.io (Windows)SuccessFull output at http://107.22.172.223:8877/02a53d4fef97ea8/output.txt Total script time: 17.18 mins
|
From: Bot.io (Linux)SuccessFull output at http://107.21.233.14:8877/07fe09dc49142d3/output.txt Total script time: 23.05 mins
|
CCITTFaxStream: fix regression when EncodedByteAlign is true and EndOfLine is false
CCITTFaxStream: fix regression when EncodedByteAlign is true and EndOfLine is false
Fixes #5592.
I have manually checked the files from #5136 to be sure that they still render the same as before. This PR makes sure that PDFs where EncodedByteAlign is true and EndOfLine is false also enter the EOL check. See http://logs.glob.uno/?c=mozilla%23pdfjs&s=29+Dec+2014&e=29+Dec+2014#c28841 for more information.