Skip to content

fixed import stream identifier #11

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

Merged
merged 2 commits into from
Aug 9, 2019

Conversation

mrtsbt
Copy link
Contributor

@mrtsbt mrtsbt commented Aug 9, 2019

I just noticed a mistake in my last commit: Though io.ReadSeeker is an interface, it still needs to be passed as a pointer argument. Otherwise the identifier might still change on multiple calls.
I'm very sorry for the inconvenience!

@mrtsbt
Copy link
Contributor Author

mrtsbt commented Aug 9, 2019

@phpdave11 If this is done, I would try to update the contrib package in https://github.com/jung-kurt/gofpdf if that's okay with you?

@phpdave11
Copy link
Owner

@mrtsbt sounds good, thank you!

@phpdave11 phpdave11 merged commit 3bfcd8e into phpdave11:master Aug 9, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants