Oops, did I do a bonehead mistake on my logic