plataformatec/devise — pull request #2999 by PureSin (2014-04-23 21:18:22+00:00)
Title: =, should be ==

I think it should be == instead of =? Since (salt = authenticatable_salt) always evaluates to true.
If the intention is to set salt to be authenticatable_salt and then return salt, I don' think this is a very clear way of expressing that.