Re: Improving output of syntax errors
| From: | Joe Watkins | Date: | Sun, 28 Jun 2020 22:08:57 +0000 |
| Subject: | Re: Improving output of syntax errors | ||
| References: | 1 2 | Groups: | php.internals |
| Request: | Send a blank email to internals+get-110763@lists.php.net to get a copy of this message | ||
Nicely done.
No need for an RFC.
Thanks.
On Sunday, 28 June 2020, Stanislav Malyshev <smalyshev@gmail.com> wrote:
> Hi!
>
> > For more examples, see:
> > https://rwec.co.uk/x/php-parse-errors/comparison.html
> >
> > The patch can be reviewed at:
> > https://github.com/php/php-src/pull/5722
>
> I think this is great, thanks for implementing it!
>
> > I am happy to post a small RFC if people think this requires a vote.
> >
> > Any other feedback is welcome.
>
> I personally don't think it requires a vote, as error messages are an
> implementation detail and nobody should ever rely on it. Let's wait for
> a bit though if somebody argues that it needs an RFC, but otherwise I
> think we can merge it.
>
> > (As an aside, the other commonly requested change was to include column
> > numbers; this appears to be feasible, but definitely more complex, and
> > with potential performance trade-offs. I hope to re-visit this later.)
>
> Column numbers would be awesome too.
>
> --
> Stas Malyshev
> smalyshev@gmail.com
>
> --
> PHP Internals - PHP Runtime Development Mailing List
> To unsubscribe, visit: https://www.php.net/unsub.php
>
>