Re: [PATCH] urlmatch: use hex2chr() in append_normalized_escapes()

2017-07-08 Thread René Scharfe
Am 08.07.2017 um 16:28 schrieb Kyle J. McKay: On Jul 8, 2017, at 01:59, René Scharfe wrote: Simplify the code by using hex2chr() to convert and check for invalid characters at the same time instead of doing that sequentially with one table lookup for each. I think that comment may be a bit

Re: [PATCH] urlmatch: use hex2chr() in append_normalized_escapes()

2017-07-08 Thread Kyle J. McKay
On Jul 8, 2017, at 01:59, René Scharfe wrote: Simplify the code by using hex2chr() to convert and check for invalid characters at the same time instead of doing that sequentially with one table lookup for each. I think that comment may be a bit misleading as the changes are just switching