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

fix(lzhuf): Fix lzhuf.h header file #137

Merged
merged 1 commit into from Mar 30, 2022
Merged

fix(lzhuf): Fix lzhuf.h header file #137

merged 1 commit into from Mar 30, 2022

Conversation

wb8tyw
Copy link
Owner

@wb8tyw wb8tyw commented Mar 30, 2022

lzhuf.h:
Add missing include of errno.h
Remove dangerous commented out code to hack around missing include.

lzhuf.h:
  Add missing include of errno.h
  Remove dangerous commented out code to hack around missing include.
@wb8tyw
Copy link
Owner Author

wb8tyw commented Mar 30, 2022

Want to get that bad code out of at least my fork of d-rats as soon as possible.
Would be a good idea to get that fixed in the https://github.com/maurizioandreotti/D-Rats upstream before someone else things that you can define errno to 0 and expect things to wok because they saw it in d-rats.

@wb8tyw wb8tyw merged commit 08ff004 into python3_tyw Mar 30, 2022
@wb8tyw wb8tyw deleted the ka5mye_lzhuf_fix branch March 31, 2022 13:06
wb8tyw added a commit that referenced this pull request Sep 5, 2022
lzhuf.h:
  Add missing include of errno.h
  Remove dangerous commented out code to hack around missing include.

Co-authored-by: John E. Malmberg <wb8tyw@gmail.com>
wb8tyw added a commit that referenced this pull request Sep 6, 2022
lzhuf.h:
  Add missing include of errno.h
  Remove dangerous commented out code to hack around missing include.

Co-authored-by: John E. Malmberg <wb8tyw@gmail.com>
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