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
Make more modules publicly runnable on dlang.org #4966
Conversation
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.
remarks
@@ -3934,7 +3934,6 @@ private T octal(T)(const string num) | |||
return value; | |||
} | |||
|
|||
/// | |||
@system unittest |
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.
This is a private method - there's no point in exposing in trying to expose the unittest
* Returns: the maximum value | ||
*/ | ||
T max (T) (T[] arr) | ||
{ |
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.
this was superseded by maxElement
@@ -1953,7 +1922,6 @@ ElementType!(A) pop (A) (ref A a) if (isDynamicArray!(A) && !isNarrowString!(A) | |||
return e; | |||
} | |||
|
|||
/// | |||
unittest | |||
{ | |||
auto array = [0, 1, 2, 3]; |
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.
Seems like there is already shift within Phobos ;-)
(though this one is private)
@@ -1846,7 +1818,6 @@ T[] substr (T) (T[] str, ptrdiff_t start = 0, ptrdiff_t length = ptrdiff_t.min) | |||
return str[start .. end]; | |||
} | |||
|
|||
/// | |||
unittest | |||
{ |
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.
ditto: private method
@@ -1898,7 +1869,6 @@ int compareFirstN (alias pred = "a < b", S1, S2) (S1 s1, S2 s2, size_t length, b | |||
return caseInsensitive ? slice1.icmp(slice2) : slice1.cmp(slice2); | |||
} | |||
|
|||
/// | |||
unittest | |||
{ |
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.
ditto: private method
@@ -1929,7 +1899,6 @@ auto grep (Range, Regex) (Range input, Regex pattern, bool invert = false) | |||
return filter!(dg)(input); | |||
} | |||
|
|||
/// | |||
unittest |
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.
ditto: private method
602b23e
to
ff1f2d2
Compare
So if someone is interested in the motivation/rationale, please have a look at e.g. https://dlang.org/phobos-prerelease/std_algorithm_searching.html#.minElement |
ff1f2d2
to
18ecb6b
Compare
rebased |
Follow-up to #4943: this PR reduces the module exclusion for modules that can be run on dlang.org
For this all tests within a module need to pass separately without the module.