ZF-8396: RunLengthDecode Filter support for PDF documents

Issue Type:
New Feature
Created:
2009-11-25T17:07:55.000+0000
Last Updated:
2009-12-18T07:04:06.000+0000
Status:
Resolved
Fix version(s):
  • 1.9.7 (11/Jan/10)
Reporter:
Christopher Thomas (cwt137)
Assignee:
Alexander Veremyev (alexander)
Tags:
  • Zend_Pdf
Related issues:
Attachments:

Description

Hello,

It would be nice if the RunLengthDecode filter was in Zend_Pdf. I signed and sent the CLA a week ago but haven't gotten a response yet. So, I don't think it is safe for me to post my patch. Please advise.

Comments

Posted by Christopher Thomas (cwt137) on 2009-12-16T07:46:41.000+0000

I got my CLA approved. Attached is my patch. It includes a unit test.

Posted by Christopher Thomas (cwt137) on 2009-12-16T08:50:34.000+0000

Sorry. I forgot a couple of lines in the Stream.php file. Attached is the new patch.

Posted by Alexander Veremyev (alexander) on 2009-12-18T07:04:06.000+0000

Christopher,

I've applied attached patch. Thanks for your job!

BTW Provided implementation produced 'Uninitialized string offset' at 'library/Zend/Pdf/Filter/RunLength.php:58' warning while running unit tests.

I also added some improvements: using strspn() and str_repeat() functions and non-repeatable character sequences recognition.