On 31.12.2016 19:53, Philipp Stephani wrote:
> Agreed; converted to defun. I've only used defsubst because some other
> helper functions also used defsubst.
Thanks. Those others can probably be changed as well.
Yes, but rather not in this commit.
> No, the below case is more general and therefore has to come last.
Makes sense.
> It's not 100% related to the patch, but I think it can be included for
> symmetry reasons (testing encoding as well as decoding).
Of course. These are testing utf-8 encoding, though, right? It would be
better if you split them to a separate commit, I think.
OK, I've removed it from this patch and pushed it as 93be35e038.