Skip to content

Conversation

mingodad
Copy link
Contributor

@mingodad mingodad commented Nov 4, 2014

@@ -1020,6 +1021,27 @@ void def(ref int x)
int z = 3;
def(z);
// z is now 4

//T is a placeholder identifier that can be any accepted type.
//A template function is usefull for generic code, instead of writing
Copy link
Contributor

Choose a reason for hiding this comment

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

Typo (useful)

@quickfur
Copy link
Member

ping
Any updates based on reviewer comments?

@quickfur
Copy link
Member

quickfur commented Dec 5, 2014

ping

@andralex
Copy link
Member

andralex commented Jan 8, 2015

@quickfur could you please take this over?

@quickfur
Copy link
Member

quickfur commented Jan 8, 2015

Whoa, one thing at a time! I'm still wading through std.algorithm's docs...

Speaking of which, (when) are we gonna split up std.algorithm into more manageable chunks? I already have to manually delete half the file in order to run its unittests when locally testing PR's, because otherwise it soaks up all RAM and dies. Or maybe I should start doing that as part of the ddoc improvements...

@ntrel
Copy link
Contributor

ntrel commented Jan 9, 2015

I've now implemented the suggestions here in #751.

@9rnsr 9rnsr merged commit fe188ab into dlang:master Jan 9, 2015
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.

7 participants