should use .eql?