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 samples/wc2.d so it compiles #6692

Merged
merged 1 commit into from Apr 13, 2017
Merged

Conversation

WalterBright
Copy link
Member

One of the files https://issues.dlang.org/show_bug.cgi?id=17099 that needs fixing.

@@ -109,7 +109,8 @@ int main (string[] args)

writefln("--------------------------------------");

foreach (word1; dictionary.keys.sort)
import std.algorithm;
Copy link
Member

Choose a reason for hiding this comment

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

Might as well put this at the top with the other imports

Copy link
Member

Choose a reason for hiding this comment

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

Also, you can specify it to only import sorting

@wilzbach
Copy link
Member

How about adding it to the Makefile, so that we can ensure that the samples don't break in the future?

@andralex
Copy link
Member

@wilzbach that'd be awesome! After these are in could you please do this?

@dlang-bot dlang-bot merged commit 353bc87 into dlang:master Apr 13, 2017
@WalterBright WalterBright deleted the fixwc2 branch April 13, 2017 21:50
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
5 participants