Changeset 1905

Timestamp:

08/21/10 00:19:57 (14 years ago)

Author:

dsimcha

Message:

Bug 2958: std.getopt RangeError? on missing arg

Files:

• trunk/docsrc/changelog.dd (modified) (1 diff)
• trunk/phobos/std/getopt.d (modified) (1 diff)

trunk/docsrc/changelog.dd

@@ -15 +15 @@
-    $(LI std.typetuple:  Added anySatisfy.) 
-    )
-    $(BUGSFIXED
-    $(LI Unlisted Bug:  std.math.pow doesn't work on immutable numbers.)
-    $(LI Unlisted Bug:  std.math.pow floating point overload expects both arguments to be exact same type.)
-    $(LI Unlisted Bug:  std.path.join("", "foo") returns "/foo" instead of "foo" on Posix.)
-    $(LI Unlisted Bug:  std.range.iota() random access primitives inconsistent after popFront on floating point version)
-    $(LI Several unlisted bugs in std.range.chain)
-    $(LI $(BUGZILLA 2903): Splitter should be bi-dir if the input range is bi-dir.)
-    $(LI $(BUGZILLA 2951): std.random.dice() should be templated on proportions.)
+    $(LI $(BUGZILLA 2958): std.getopt RangeError on missing arg)
-    $(LI $(BUGZILLA 3123): std.algorithm.zip fails on 'lazy' ranges)
-    $(LI $(BUGZILLA 3294): forward reference to inferred return type of function call)
-    $(LI $(BUGZILLA 3312): std.string.count should use const(char)[], not immutable.)
-    $(LI $(BUGZILLA 3348): Documentation for many std.process functions has disappeared)
-    $(LI $(BUGZILLA 3361): code in std.zlib concatenates void[] arrays )
-    $(LI $(BUGZILLA 3877): std.range.chain do not manage infinite ranges correctly)
-    $(LI $(BUGZILLA 3894): std.range.Stride!R requires R.front() and R.back() to return by reference)
-    $(LI $(BUGZILLA 3946): schwartzSort - SwapStrategy always unstable)
-    $(LI $(BUGZILLA 4402): std.range.Zip doesn't work w/ non-lvalue ranges.)
-    $(LI $(BUGZILLA 4408): Ambiguity when using std.algorithm.splitter with generic ranges.)

trunk/phobos/std/getopt.d

@@ -421 +421 @@
-        static if (is(typeof(*receiver) == bool)) {
-            *receiver = true;
-            break;
-        } else {
-            // non-boolean option, which might include an argument
-            //enum isDelegateWithOneParameter = is(typeof(receiver("")) : void);
-            enum isDelegateWithLessThanTwoParameters =
-                is(typeof(receiver) == delegate) &&
-                !is(typeof(receiver("", "")));
-            if (!isDelegateWithLessThanTwoParameters && !val && !incremental) {
-
+
+
+
+
-                val = args[i];
-                args = args[0 .. i] ~ args[i + 1 .. $];
-            }
-            static if (is(typeof(*receiver) : real))
-            {
-                // numeric receiver
-                if (incremental) ++*receiver;
-                else *receiver = to!(typeof(*receiver))(val);
-            }
-            else static if (is(typeof(*receiver) == string))