parseopt: remove unreachable code

At this point in the code after running skip_prefix() on the
variable and receiving the result in the same variable, the contents
of the variable can never be NULL.  The function either (1) updates
the variable to point at a later part of the string it originally
pointed at, or (2) leaves it intact if the string does not have the
prefix.  (1) will never make the variable NULL, and (2) cannot be
the source of NULL, because the variable cannot be NULL before
calling skip_prefix(), which would die immediately by dereferencing
the NULL pointer in that case.

Helped-by: Phillip Wood <phillip.wood@dunelm.org.uk>
Signed-off-by: Junio C Hamano <gitster@pobox.com>

diff --git a/parse-options.c b/parse-options.c
index 27c1e75d5377dc7c159ae9fadb5428cf5a3f256e..97a55300e80d6a5f6be2134a2769dd977016d038 100644 (file)
--- a/parse-options.c
+++ b/parse-options.c
@@ -223,8 +223,6 @@ static enum parse_opt_result do_get_value(struct parse_opt_ctx_t *p,
                        return 0;
 
                is_optional = skip_prefix(value, ":(optional)", &value);
-               if (!value)
-                       is_optional = false;
                value = fix_filename(p->prefix, value);
                if (is_optional && is_missing_file(value)) {
                        free((char *)value);