mirrors/phobos
1
0
Fork 0
mirror of https://github.com/dlang/phobos.git synced 2025-04-27 13:40:20 +03:00
Code Issues Projects Releases Packages Wiki Activity Actions

getopt public undocumented symbols and style

Browse source 
some of the symbols where clearly only used from within getopt but still where
public.

additionally some style violation fixes
This commit is contained in:
Robert burner Schadek 2015-01-21 14:48:31 +01:00
parent fb7d68882e
commit 014ff8e155
1 changed files with 10 additions and 6 deletions
Download patch file Download diff file Expand all files Collapse all files

16
std/getopt.d
View file

@ -425,7 +425,8 @@ to another program). Invoking the example above with $(D "--foo -- --bar")
parses foo but leaves "--bar" in $(D args). The double-dash itself is
removed from the argument array.
*/
GetoptResult getopt(T...)(ref string[] args, T opts) {
GetoptResult getopt(T...)(ref string[] args, T opts)
{
import std.exception : enforce;
enforce(args.length,
"Invalid arguments string passed: program name missing");
@ -500,7 +501,7 @@ struct Option {
/// an error.
}
pure Option splitAndGet(string opt) @trusted nothrow
private pure Option splitAndGet(string opt) @trusted nothrow
{
import std.array : split;
auto sp = split(opt, "|");
@ -614,7 +615,7 @@ private void getoptImpl(T...)(ref string[] args, ref configuration cfg,
}
}
bool handleOption(R)(string option, R receiver, ref string[] args,
private bool handleOption(R)(string option, R receiver, ref string[] args,
ref configuration cfg, bool incremental)
{
import std.algorithm : map, splitter;
@ -622,7 +623,8 @@ bool handleOption(R)(string option, R receiver, ref string[] args,
import std.conv : text, to;
// Scan arguments looking for a match for this option
bool ret = false;
for (size_t i = 1; i < args.length; ) {
for (size_t i = 1; i < args.length; )
{
auto a = args[i];
if (endOfOptions.length && a == endOfOptions) break;
if (cfg.stopOnFirstNonOption && (!a.length || a[0] != optionChar))
@ -685,7 +687,8 @@ bool handleOption(R)(string option, R receiver, ref string[] args,
enum isCallbackWithLessThanTwoParameters =
(is(typeof(receiver) == delegate) || is(typeof(*receiver) == function)) &&
!is(typeof(receiver("", "")));
if (!isCallbackWithLessThanTwoParameters && !(val.length) && !incremental) {
if (!isCallbackWithLessThanTwoParameters && !(val.length) && !incremental)
{
// Eat the next argument too. Check to make sure there's one
// to be eaten first, though.
enforce(i < args.length,
@ -1351,7 +1354,8 @@ message is present it will be printed after the long version of the
$(D Option).
------------
foreach(it; opt) {
foreach(it; opt)
{
writefln("%*s %*s %s", lengthOfLongestShortOption, it.optShort,
lengthOfLongestLongOption, it.optLong, it.help);
}