-
-
Notifications
You must be signed in to change notification settings - Fork 52
Adjusted alternate version to preserve formatting #145
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
Conversation
The first alt version added some whitespaces that broke the outline, so I simply filled those up. I also expanded tabs to spaces since there is no size limit restriction for the alternate version. The second alt version attempted to force binary mode with _setmode() and _fileno() calls, plus some additional headers. Rather than doing it this way, I hack in the binary mode adjustments via preprocessor. Actually I did a similar trick for IOCCC 2011, but I guess I didn't submit that version: https://uguu.org/archive/ioccc2011/akari/source/binmode_main.h The file above uses _setmode() and _fileno(). The leading underscore indicates that they were nonstandard extensions. For the binmode_main.h in this change, I have opted for setmode() and STDIN_FILENO instead, which are more widely available.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
👍
The tarballs were updated. Removed reference to the previous mod from `thanks-for-help.md`. Updated the `2012/omoikane/index.html` page. Updated sitemap.
First of all thanks to @uguu-org. Second of all please remove the alt code from the thanks file as it seems like what I did no longer stands. If I didn't do anything else there then by all means remove that item entirely. I don't have the time to do it myself. Thanks. No I did not look if this was already done but just in case. |
@xexyl Your alt code removed a dependency on /dev/urandom, so I thought it was worth keeping around for that. Thanks for your effort in fixing the entries :) |
Ah. That was for macOS - it doesn't have /dev/urandom (unfortunately). So if you removed that that could be an issue. Don't have the time to check I'm afraid (I did not even look at the log here - I just noticed you changed the alt code.)
I'm glad you appreciated it! Means a lot - more than you might know. It was a lot of fun and also educational. Best wishes! |
The tarballs were updated. Removed reference to the previous mod from `thanks-for-help.md`. Updated the `2012/omoikane/index.html` page. Updated sitemap.
The first alt version added some whitespaces that broke the outline, so I simply filled those up. I also expanded tabs to spaces since there is no size limit restriction for the alternate version.
The second alt version attempted to force binary mode with _setmode() and _fileno() calls, plus some additional headers. Rather than doing it this way, I hack in the binary mode adjustments via preprocessor.
Actually I did a similar trick for IOCCC 2011, but I guess I didn't submit that version:
https://uguu.org/archive/ioccc2011/akari/source/binmode_main.h
The file above uses _setmode() and _fileno(). The leading underscore indicates that they were nonstandard extensions. For the binmode_main.h in this change, I have opted for setmode() and STDIN_FILENO instead, which are more widely available.