This reverts commit d44ca09.
I have some confidence that this function is correct now, but helgrind still doesn't like it because it's not lock-protected.
O(1) rather than O(string len)
…bytes with find[_from]
… of chars; rename find to find_chars; add fixmes to delete byte_index, byte_index_from