No, this is ok. The code is doing the same thing for `rm` below because we need to check for specific values explicitly. This makes it easier to have tri-state values (missing, false, true) for this in the future, just like I've changed `--rm` to be tri-state (missing with default to `true`, false and true).