this isn't correct, you're comparing two chars with one char