Newsgroups: php.internals
Path: news.php.net
Xref: news.php.net php.internals:123120
Precedence: bulk
MIME-Version: 1.0
References: <CAEPPVa3oV_e6gHAojWqfb9eW7QPP3371r4jsL6a7thcjTFVZ2g@mail.gmail.com>
 <733732ea-7b2a-473c-9f65-6ef975404543@bastelstu.be> <CAEPPVa0Z55xX49LPKmRyLq_U8odUWH+D9THYGYgVbSi5UC3mXw@mail.gmail.com>
In-Reply-To: <CAEPPVa0Z55xX49LPKmRyLq_U8odUWH+D9THYGYgVbSi5UC3mXw@mail.gmail.com>
Date: Fri, 12 Apr 2024 11:32:29 +0200
Message-ID: <CAOWwgp=iX-=VvMh_mpRxFQEHPKFNw-Ypun7ReNrZ0ekYNx4REg@mail.gmail.com>
Subject: Re: [PHP-DEV][DISCUSSION] Fix mb_trim inaccurate $character
To: youkidearitai <youkidearitai@gmail.com>
Cc: php internals <internals@lists.php.net>
Content-Type: multipart/alternative; boundary="0000000000000b9fe40615e2f473"
From: nicolas.grekas+php@gmail.com (Nicolas Grekas)

--0000000000000b9fe40615e2f473
Content-Type: text/plain; charset="UTF-8"
Content-Transfer-Encoding: quoted-printable

Hi

Le jeu. 4 avr. 2024 =C3=A0 07:41, youkidearitai <youkidearitai@gmail.com> a
=C3=A9crit :

> 2024=E5=B9=B44=E6=9C=884=E6=97=A5(=E6=9C=A8) 6:30 Tim D=C3=BCsterhus <tim=
@bastelstu.be>:
> >
> > Hi
> >
> > On 4/3/24 10:02, youkidearitai wrote:
> > > Therefore, I think require an RFC, I have written a draft an RFC that
> > > fixes these issues.
> > > https://wiki.php.net/rfc/mb_trim_change_characters
> >
> > I don't think this (widening the type and changing the default value to
> > obtain the *intended* behavior) requires an RFC. It's a bugfix, a bugfi=
x
> > with a slightly larger observable impact than other bugfixes.
> >
> > Best regards
> > Tim D=C3=BCsterhus
>
> Hi
> Thank you very much for comment.
>
> I would like to discuss this RFC with all of you, as we have not yet
> decided on the correct revision policy.
> I created the RFC for that purpose.
>
> Still waiting for your comments!
> Thank you.
>
>
I also don't think this requires an RFC. This is a minor design issue that
has only one possible solution, so there is little to discuss.

Cheers,
Nicolas

--0000000000000b9fe40615e2f473
Content-Type: text/html; charset="UTF-8"
Content-Transfer-Encoding: quoted-printable

<div dir=3D"ltr"><div dir=3D"ltr">Hi<br></div><br><div class=3D"gmail_quote=
"><div dir=3D"ltr" class=3D"gmail_attr">Le=C2=A0jeu. 4 avr. 2024 =C3=A0=C2=
=A007:41, youkidearitai &lt;<a href=3D"mailto:youkidearitai@gmail.com">youk=
idearitai@gmail.com</a>&gt; a =C3=A9crit=C2=A0:<br></div><blockquote class=
=3D"gmail_quote" style=3D"margin:0px 0px 0px 0.8ex;border-left:1px solid rg=
b(204,204,204);padding-left:1ex">2024=E5=B9=B44=E6=9C=884=E6=97=A5(=E6=9C=
=A8) 6:30 Tim D=C3=BCsterhus &lt;<a href=3D"mailto:tim@bastelstu.be" target=
=3D"_blank">tim@bastelstu.be</a>&gt;:<br>
&gt;<br>
&gt; Hi<br>
&gt;<br>
&gt; On 4/3/24 10:02, youkidearitai wrote:<br>
&gt; &gt; Therefore, I think require an RFC, I have written a draft an RFC =
that<br>
&gt; &gt; fixes these issues.<br>
&gt; &gt; <a href=3D"https://wiki.php.net/rfc/mb_trim_change_characters" re=
l=3D"noreferrer" target=3D"_blank">https://wiki.php.net/rfc/mb_trim_change_=
characters</a><br>
&gt;<br>
&gt; I don&#39;t think this (widening the type and changing the default val=
ue to<br>
&gt; obtain the *intended* behavior) requires an RFC. It&#39;s a bugfix, a =
bugfix<br>
&gt; with a slightly larger observable impact than other bugfixes.<br>
&gt;<br>
&gt; Best regards<br>
&gt; Tim D=C3=BCsterhus<br>
<br>
Hi<br>
Thank you very much for comment.<br>
<br>
I would like to discuss this RFC with all of you, as we have not yet<br>
decided on the correct revision policy.<br>
I created the RFC for that purpose.<br>
<br>
Still waiting for your comments!<br>
Thank you.<br>
<br></blockquote><div><br></div><div>I also don&#39;t think this requires a=
n RFC. This is a minor design issue that has only one possible solution, so=
 there is little to discuss.</div><div><br></div><div>Cheers,<br></div><div=
>Nicolas<br></div></div></div>

--0000000000000b9fe40615e2f473--