Newsgroups: php.internals
Path: news.php.net
Xref: news.php.net php.internals:113076
MIME-Version: 1.0
References: <DM6PR07MB66184CC05829642BD4E16E3AF9BF9@DM6PR07MB6618.namprd07.prod.outlook.com>
 <DM6PR07MB66189F7D092F98F05BE7AEA2F9B39@DM6PR07MB6618.namprd07.prod.outlook.com>
In-Reply-To: <DM6PR07MB66189F7D092F98F05BE7AEA2F9B39@DM6PR07MB6618.namprd07.prod.outlook.com>
Date: Thu, 4 Feb 2021 16:04:37 +0100
Message-ID: <CAC5V6gjtPcBNiCV6X1CRd_fzX_B9zyRgqh_PReQ4JQqeQzF1eA@mail.gmail.com>
To: tyson andre <tysonandre775@hotmail.com>
Cc: "internals@lists.php.net" <internals@lists.php.net>
Content-Type: multipart/alternative; boundary="00000000000061178705ba8406f5"
Subject: Re: [PHP-DEV] Re: [RFC] var_representation() : readable alternative
 to var_export()
From: guilliam.xavier@gmail.com (Guilliam Xavier)

--00000000000061178705ba8406f5
Content-Type: text/plain; charset="UTF-8"

On Thu, Feb 4, 2021 at 3:36 PM tyson andre <tysonandre775@hotmail.com>
wrote:

> Hi internals,
>
> > I've created https://wiki.php.net/rfc/readable_var_representation based
> on
> > my original proposal in https://externals.io/message/112924
> >
> > [...]
>
> A reminder that voting on the var_representation RFC starts in a day.
> This RFC proposes adding a new function `var_representation(mixed $value,
> int $flags=0): string` with multiple improvements on `var_export()`.
>
> Any other feedback?
>

Hi,

I think the "though strings with embedded newlines will still cause
newlines in the output" part is obsolete (since `\r` and `\n` are escaped
now).

Apart from that, since var_export (and var_dump) can't really be "fixed"
for BC reasons, I'm +1 for the new function.

Thanks

-- 
Guilliam Xavier

--00000000000061178705ba8406f5--