Welcome! Log In Create A New Profile

Advanced

[PHP-DEV] [VOTE] JSON_THROW_ON_ERROR

Posted by Andrea Faulds 
Andrea Faulds
[PHP-DEV] [VOTE] JSON_THROW_ON_ERROR
September 25, 2017 11:30PM
Hi everyone,

https://wiki.php.net/rfc/json_throw_on_error

Voting opens today, 2017-09-25, and ends in 10 days' time, 2017-10-05.
Under the terms of the voting RFC this only needs 50%+1 but I'll feel
guilty if it's not 2/3 so it's 2/3.

Happy voting, please ask if there's anything needing clarifying.

Thanks.
--
Andrea Faulds
https://ajf.me/

--
PHP Internals - PHP Runtime Development Mailing List
To unsubscribe, visit: http://www.php.net/unsub.php
Derick Rethans
Re: [PHP-DEV] [VOTE] JSON_THROW_ON_ERROR
September 26, 2017 12:10PM
On Mon, 25 Sep 2017, Andrea Faulds wrote:

> Hi everyone,
>
> https://wiki.php.net/rfc/json_throw_on_error
>
> Voting opens today, 2017-09-25, and ends in 10 days' time, 2017-10-05. Under
> the terms of the voting RFC this only needs 50%+1 but I'll feel guilty if it's
> not 2/3 so it's 2/3.
>
> Happy voting, please ask if there's anything needing clarifying.

I'm going to be in favour, but I was wondering whether the name of the
exception "JsonException" is a good thing? I would still introduce it,
but probably subclass a "JsonCodingException" to signal just an
exceptional code while encoding/decoding. I can see future exceptions
being thrown from the extension, where you then get confusion (or
conflicts with error codes).

cheers,
Derick

--
https://derickrethans.nl | https://xdebug.org | https://dram.io
Like Xdebug? Consider a donation: https://xdebug.org/donate.php
twitter: @derickr and @xdebug

--
PHP Internals - PHP Runtime Development Mailing List
To unsubscribe, visit: http://www.php.net/unsub.php
Niklas Keller
Re: [PHP-DEV] [VOTE] JSON_THROW_ON_ERROR
September 26, 2017 01:50PM
2017-09-26 12:01 GMT+02:00 Derick Rethans <[email protected]>:

> On Mon, 25 Sep 2017, Andrea Faulds wrote:
>
> > Hi everyone,
> >
> > https://wiki.php.net/rfc/json_throw_on_error
> >
> > Voting opens today, 2017-09-25, and ends in 10 days' time, 2017-10-05.
> Under
> > the terms of the voting RFC this only needs 50%+1 but I'll feel guilty
> if it's
> > not 2/3 so it's 2/3.
> >
> > Happy voting, please ask if there's anything needing clarifying.
>
> I'm going to be in favour, but I was wondering whether the name of the
> exception "JsonException" is a good thing? I would still introduce it,
> but probably subclass a "JsonCodingException" to signal just an
> exceptional code while encoding/decoding. I can see future exceptions
> being thrown from the extension, where you then get confusion (or
> conflicts with error codes).


Which future exceptions can you imagine?

Regards, Niklas
Andrea Faulds
[PHP-DEV] Re: [VOTE] JSON_THROW_ON_ERROR
October 06, 2017 12:10AM
Hi again,

I'm happy to report JSON_THROW_ON_ERROR was unanimously accepted with 23
votes.

Hopefully it can be merged soon. Now would be a good time to review the
pull request if desired.

Thanks for discussing and voting!

--
Andrea Faulds
https://ajf.me/

--
PHP Internals - PHP Runtime Development Mailing List
To unsubscribe, visit: http://www.php.net/unsub.php
Sorry, only registered users may post in this forum.

Click here to login