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 mi

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 from

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

2017-07-08 Thread René Scharfe
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. Signed-off-by: Rene Scharfe --- urlmatch.c | 10 +- 1 file changed, 5 insertions(+), 5 deletions(-) diff --git a/urlmatch.