# [PATCH] Unicode number changes

- Thread: https://externals.io/message/26827
- Original archive: https://news-web.php.net/php.internals/26827
- Started by Matt Wilmas on 2006-12-07T10:55:22+00:00
- Messages: 2
- Reply by email to internals@lists.php.net
- More for agents: search with https://externals.io/search.md?q={query} · JSON API at https://externals.io/api · documentation at https://externals.io/llms.txt

---

## #26827 — Matt Wilmas — 2006-12-07T10:55:22+00:00

URL: https://externals.io/message/26827#26827

Hi Andrei, Antony, ...

Attached is a patch for zend_u_strtol() and HANDLE_U_NUMERIC() to only allow
ASCII digits and not use u_digit().  (Also switched to hex values for sign
characters, etc.)  I tested the changes quickly and all appears fine.
Provides a pretty good performance increase too. :-)


Matt

---

## #26843 — Andrei Zmievski — 2006-12-07T18:56:40+00:00

URL: https://externals.io/message/26827#26843 · In reply to: #26827

Applied.

-Andrei



> Hi Andrei, Antony, ...
>
> Attached is a patch for zend_u_strtol() and HANDLE_U_NUMERIC() to only 
> allow
> ASCII digits and not use u_digit().  (Also switched to hex values for 
> sign
> characters, etc.)  I tested the changes quickly and all appears fine.
> Provides a pretty good performance increase too. :-)
>
>
> Matt
> <unicode_numbers.diff.txt