Interesting. I didn't realize the logic was already built into smartypants, and I distinctly recall having test cases which weren't converted by smartypants, which was why the extra lines were added - I agree with you that on a quick review of the code they seem redundant, and it's possible something else was going on. I'll dig back into it when I get a chance.
|