Patch from oliver: In message.c in the escape & unescape functions the loops are not optimal, doing a strlen() for each character in the message string. valgrind/callgrind identified especially msim_unescape() as a expensive because of that (guess there is more unescaping going on than escaping). The attached patch moves the strlen() out of the loop header for both functions. Closes #4790.