Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Initialize no longer needed #41

Closed
Phyks opened this issue Apr 26, 2014 · 3 comments
Closed

Initialize no longer needed #41

Phyks opened this issue Apr 26, 2014 · 3 comments

Comments

@Phyks
Copy link

Phyks commented Apr 26, 2014

Hello,

According to this page, initialize is no longer needed :

2014/03/24: Bugfixes and improvements for fauly PDFs.
API changes:

    PDFDocument.initialize() method is removed and no longer needed. A password is given as an argument of a PDFDocument constructor. 
@Phyks
Copy link
Author

Phyks commented May 30, 2016

Did I miss something, or do you confirm this can be safely removed? Do you want a PR for this?

EDIT: Oops, I missed this was already done in #43.

@kanzure
Copy link
Owner

kanzure commented May 30, 2016

Yeah #43 still needs a little bit more review, but it's probably ready.

@Phyks
Copy link
Author

Phyks commented May 30, 2016

Ok, so closing this issue as it is handled by #43.

@Phyks Phyks closed this as completed May 30, 2016
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

3 participants