Michael Koziarski

This patch doesn't apply cleanly right now, could you rebase it for me

However the change doesn't quite look right to me, shouldn't the check be whether the tar...

UTF-8 support isn't an 'additional feature' though, if it doesn't work with those strings, it's broken.

I'd prefer we left it as is lest people start thinking '...

heh, lighthouse doesn't scale for parallel discussions!

Yea, see if you can rejig the patch to just fix instance_eval and leave the others in place.

Of course it's possible that nothing else needs changing, in which case awesome!

Given we've decided to apply this anyhow, I feel it's important we investigate every other case where this patch introduced changes. The original is here:

http...

These changes were added for performance reasons, If you're interested in the changeset and memory savings git blame will show them to you and alexander dymo's ...