Newsgroups: php.internals
Path: news.php.net
Xref: news.php.net php.internals:108669
MIME-Version: 1.0
References: <CA+41xTXZBziaWG6Sp3XmYU-6+4ou3Dp-qksER5HsE-qGpr_ZRg@mail.gmail.com>
In-Reply-To: <CA+41xTXZBziaWG6Sp3XmYU-6+4ou3Dp-qksER5HsE-qGpr_ZRg@mail.gmail.com>
Date: Wed, 19 Feb 2020 12:20:00 -0300
Message-ID: <CAEsg9X0TctW+5m=u_Uj2+qHiAxJs2=94anX-SiRjoUAGWEo6YQ@mail.gmail.com>
To: Nico Oelgart <nicoswd@gmail.com>
Cc: PHP Internals <internals@lists.php.net>
Content-Type: multipart/alternative; boundary="000000000000268ec6059eef5310"
Subject: Re: [PHP-DEV] New PCRE function
From: david.proweb@gmail.com (David Rodrigues)

--000000000000268ec6059eef5310
Content-Type: text/plain; charset="UTF-8"
Content-Transfer-Encoding: quoted-printable

Maybe you can set all this messages as lowercase? That way we can use it
more easily. If we need the first letter in capital letters we can use
`ucfirst()`, because the opposite is more complicated (a `strtolower()`
would "break" the "JIT" message, for example).


Atenciosamente,
David Rodrigues


Em qua., 19 de fev. de 2020 =C3=A0s 11:37, Nico Oelgart <nicoswd@gmail.com>
escreveu:

> Hi Internals,
>
> I've submitted a small PR proposing a new PCRE function that
> returns a human-friendly string representation of the last error.
>
> https://github.com/php/php-src/pull/5185
>
> Currently there's only preg_last_error() which returns error codes,
> which isn't really helpful. Most comments in the documentation are
> about converting those to something more user friendly.
>
> https://www.php.net/preg_last_error#usernotes
>
> Besides, most extensions provide multiple functions for both
> use-cases natively, such as:
>
>  - json (json_last_error / json_last_error_msg)
>  - socket (socket_last_error / socket_strerror)
>  - sqlite (sqlite_last_error / sqlite_error_string)
>  - ldap (ldap_errno / ldap_error)
>  - etc...
>
> So I think it makes sense for this function to exist.
>
> Any thoughts?
>
> Thank you!
>

--000000000000268ec6059eef5310--