4clojure/4clojure — comment on commit fd2cc0314933469345a8233133f27a67d274be98
File: src/foreclojure/settings.clj line 83
By amalloy:

You're right about 79 itself being redundant, provided that `new-pwd` is never nil (I'm not sure if that's a safe assumption).

But we definitely do need the check here, because we are willing to accept an empty password - it signifies "no desire to change passwords". If we didn't have the `empty?` check then empty passwords would be checked against the DB, always failing; you would thus have to enter your password to get anything at all done here.