Skip to content
This repository has been archived by the owner on Oct 12, 2022. It is now read-only.

Fix a static if (__USE_MISC) section in core.sys.linux.netinet.in_. #1911

Merged
merged 1 commit into from Sep 1, 2017

Conversation

jmdavis
Copy link
Member

@jmdavis jmdavis commented Sep 1, 2017

Apparently, I made it cover more than it was supposed to. I think that
the fact that the #endif ending that section in the header file was
right after a #define threw me off, and I incorrectly thought that
the #endif went with the #define even though that doesn't make sense. In
either case, this should fix my mistake.

And man does the way that github show this diff look weird. All it does is add a brace in order to end a static if earlier, remove the brace that used to end the static if, and then fix the indentation. :|

Apparently, I made it cover more than it was supposed to. I think that
the fact that the #endif ending that section in the header file was
right after a #define threw me off, and I incorrectly thought that
the #endif went with the #define even though that doesn't make sense. In
either case, this should fix my mistake.
@dlang-bot
Copy link
Contributor

Thanks for your pull request, @jmdavis!

Bugzilla references

Your PR doesn't reference any Bugzilla issue.

If your PR contains non-trivial changes, please reference a Bugzilla issue or create a manual changelog.

@PetarKirov
Copy link
Member

And man does the way that github show this diff look weird. All it does is add a brace in order to end a static if earlier, remove the brace that used to end the static if, and then fix the indentation. :|

Adding ?w=1 to the end of the URL is your friend - https://github.com/dlang/druntime/pull/1911/files?w=1

Copy link
Member

@PetarKirov PetarKirov left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@dlang-bot dlang-bot merged commit 0c45f09 into dlang:master Sep 1, 2017
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
3 participants