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

Update deflate.cpp #8

Merged
merged 1 commit into from
May 4, 2016
Merged

Update deflate.cpp #8

merged 1 commit into from
May 4, 2016

Conversation

jordanius
Copy link
Contributor

My first pull request ever on here, so apologies in advance if I'm not doing this right. Just corrected what I'm pretty sure was a very simple typo - removing the extra ", 1" on the end there makes the package compile again on all systems, so unless I'm wrong (and I've never programmed in any version of C/C++, so I could be - I'm only troubleshooting the error that gcc led me directly to), I think its inclusion there on the end was likely a mistake. Thanks for all the time you spend updating ECT - it's a fantastic tool, and I make sure to mention it when and wherever I get the chance.

My first pull request ever on here, so apologies in advance if I'm not doing this right. Just corrected what I'm pretty sure was a very simple typo - removing the extra ", 1" on the end there makes the package compile again on all systems, so unless I'm wrong (and I've never programmed in any version of C/C++, so I could be - I'm only troubleshooting the error that gcc led me directly to), I think its inclusion there on the end was likely a mistake. Thanks for all the time you spend updating ECT - it's a fantastic tool, and I make sure to mention it when and wherever I get the chance.
@fhanau fhanau merged commit 1475688 into fhanau:master May 4, 2016
@fhanau
Copy link
Owner

fhanau commented May 4, 2016

Thank you!

@jordanius jordanius deleted the patch-1 branch May 6, 2016 07:22
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.

None yet

2 participants