Clean up mark()'s test for its optional argument

Commit 60bbb6b0 (v4.2.15) accidentally changed the test from "absent
or empty" to "absent".  Leave it that way, just clean it up.

diff --git a/src/lib/commands/mark.c b/src/lib/commands/mark.c
index 59045015b151f7282d882661db1aa03563a62742..1d51dde264f1412a65be8f10b6cd29b41345cbae 100644 (file)
--- a/src/lib/commands/mark.c
+++ b/src/lib/commands/mark.c
@@ -51,7 +51,7 @@ mark(void)
        return RET_FAIL;
     }
 
        return RET_FAIL;
     }
 

-    if (player->argp[1] && player->argp[1]) {
+    if (player->argp[1]) {

        p = getstarg(player->argp[1], "What commodity (or 'all')? ", buf);
        if (!p)
            return RET_SYN;
        p = getstarg(player->argp[1], "What commodity (or 'all')? ", buf);
        if (!p)
            return RET_SYN;