Newsgroups: php.internals
Path: news.php.net
Xref: news.php.net php.internals:130445
Precedence: list
MIME-Version: 1.0
References: <3f4f6959.eaf.19cf0276cd8.Coremail.lamentxu@163.com>
 <4bf704c.b746.19d20d429ac.Coremail.lamentxu@163.com> <CAAKU0umogV26u-Y9jc76XfU4cF4CN22Hs72bCYs1YQtYpac1qA@mail.gmail.com>
In-Reply-To: <CAAKU0umogV26u-Y9jc76XfU4cF4CN22Hs72bCYs1YQtYpac1qA@mail.gmail.com>
Date: Wed, 25 Mar 2026 11:02:53 +0100
Message-ID: <CADyq6s+5zM4RCT02_VDQG01RvxYpwYvJRK5BSppszPkMZ5Hd6A@mail.gmail.com>
Subject: Re: [PHP-DEV] [RFC] Remove \0 from default trim() character mask
To: Levi Morrison <levi.morrison@datadoghq.com>
Cc: LamentXU <lamentxu@163.com>, "1372449351@qq.com" <1372449351@qq.com>, 
	"internals@lists.php.net" <internals@lists.php.net>
Content-Type: multipart/alternative; boundary="000000000000c0242b064dd65fde"
From: ocramius@gmail.com (Marco Pivetta)

--000000000000c0242b064dd65fde
Content-Type: text/plain; charset="UTF-8"

On Tue, 24 Mar 2026 at 19:29, Levi Morrison <levi.morrison@datadoghq.com>
wrote:

> There hasn't been much discussion, so dear internals: are simply busy,
> un-opinionated, or what?
>

I'd vote against this proposal: `\0` being considered one of the stripped
characters is now a downstream assumption, and this ends up being a BC
break with little to no advantages.

From a semantic perspective, `\n`, `\t` and `\r` are also "control
characters" in other contexts (not the C world).


Marco Pivetta

https://mastodon.social/@ocramius

https://ocramius.github.io/

--000000000000c0242b064dd65fde
Content-Type: text/html; charset="UTF-8"
Content-Transfer-Encoding: quoted-printable

<div dir=3D"ltr"><br><div class=3D"gmail_quote gmail_quote_container"><div =
dir=3D"ltr" class=3D"gmail_attr">On Tue, 24 Mar 2026 at 19:29, Levi Morriso=
n &lt;<a href=3D"mailto:levi.morrison@datadoghq.com">levi.morrison@datadogh=
q.com</a>&gt; wrote:<br></div><blockquote class=3D"gmail_quote" style=3D"ma=
rgin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:=
1ex">
There hasn&#39;t been much discussion, so dear internals: are simply busy,<=
br>
un-opinionated, or what?<br></blockquote><div><br></div><div>I&#39;d vote a=
gainst this proposal: `\0` being considered one of the stripped characters =
is now a downstream assumption, and this ends up being a BC break with litt=
le to no advantages.</div><div><br></div><div>From a semantic perspective, =
`\n`, `\t` and `\r` are also &quot;control characters&quot; in other contex=
ts (not the C world).</div><br><div><div><br clear=3D"all"></div><div><div =
dir=3D"ltr" class=3D"gmail_signature"><div dir=3D"ltr">Marco Pivetta <br><b=
r><a href=3D"https://mastodon.social/@ocramius" target=3D"_blank">https://m=
astodon.social/@ocramius</a><br><br><a href=3D"https://ocramius.github.io/"=
 target=3D"_blank">https://ocramius.github.io/</a><br></div></div></div>=C2=
=A0</div></div></div>

--000000000000c0242b064dd65fde--