usr.bin/units: use else..if to avoid calling the next branch

Even if the first branch succeeds, next time it will still check for
the second branch (which will be false) as the first one was true.
Add an else..if statement to address this.

Signed-off-by: rilysh <[email protected]>
Reviewed by: imp
Pull Request: https://github.com/freebsd/freebsd-src/pull/1165

usr.bin/units/units.c

@@ -497,8 +497,7 @@ lookupunit(const char *unit)
 			}
 		}
 		free(copy);
-	}
-	if (unit[strlen(unit) - 1] == 's') {
+	} else if (unit[strlen(unit) - 1] == 's') {
 		copy = dupstr(unit);
 		copy[strlen(copy) - 1] = 0;
 		for (i = 0; i < unitcount; i++) {