Fix illegal encoding for MacRomanEncoding

[daneren2005]

With Mac in the from encoding, would get the following error: mb_convert_encoding(): Illegal character encoding specified. Fixes #229

[k00ni]

Hi @daneren2005, can you ... (posted list outdated, see the latest at the bottom)

[daneren2005]

I made my own package because I gave up. I don't intend on spending any more time on this. If anyone else wants to spend the time they are more then welcome.

[k00ni]

Hello @daneren2005, no problem, thank your for your efforts though. I will re-open this PR, maybe we can finish it another time.

If someone else wants to help, this is what is left to do here:

• add a small test case which shows the faulty behavior (with a PDF if possible),
• build on top of current master branch

Name change in composer.json can be ignored.

[j0k3r]

I cleaned up the PR.
We are now waiting for a PDF sample to reproduce the bug and ensure the PR is fixing the issue.

[fbett]

test.pdf

This file reproduces the problem.

[k00ni]

I created the branch pr/238 here and tried the PDF given by @fbett. The PDFParser is fine in PHP 5.6 - 7.4, but raises an error in PHP 8.0. He complains:

mb_convert_encoding(): Argument #3 ($from) contains invalid encoding "Mac"

https://travis-ci.org/github/smalot/pdfparser/jobs/701562931#L257

As far as i can tell no one changed related code, so why is the error gone before PHP 8.0?

---

With the fix from daneren2005 all tests run without an error on all PHP versions. 👍

https://travis-ci.org/github/smalot/pdfparser/builds/701564912

Left to do:

• add PDF file from @fbett to this PR (which triggered the problem in PHP 8) - the code of this PR is outdated therefore its not possible to just add PDF file
• close this PR (because its code is outdated)
• merge in pr/238 instead (its based on the work of daneren2005)

Any objections?

[k00ni]

Closed, replacement will be merged instead: #313

Thank you for your effort everyone!