Actually, my (very small) concern wasn't so much about line length but about getting the character entity every time before using it, when it can be found once and then used four times. Obviously it's not a significant performance improvement (if at all?), but I thought that together with the improved (?) readability it was worth the change?