Bug #69957 [Com]: Three different ways of handling same error condition

From: Date: Mon, 29 Jun 2015 10:32:11 +0000
Subject: Bug #69957 [Com]: Three different ways of handling same error condition
References: 1  Groups: php.bugs 
Request: Send a blank email to php-bugs+get-193987@lists.php.net to get a copy of this message
Edit report at https://bugs.php.net/bug.php?id=69957&edit=1 ID: 69957 Comment by: cameron dot adam+php at gmail dot com Reported by: cameron dot adam+php at gmail dot com Summary: Three different ways of handling same error condition Status: Re-Opened Type: Bug Package: Scripting Engine problem Operating System: Windows 7 64-bit PHP Version: 7.0.0alpha2 Assigned To: bwoebi Block user comment: N Private report: N New Comment: Yeah, got that. My point was not that your not fulfilling the prescribed behaviour, it's that the behaviour you're prescribing lacks uniformity, and - in the bigger scheme of things - isn't terribly sensible or helpful. The article I link to in the original ticket details this: I did not want to duplicate it here for various reasons. Cheers. Previous Comments: ------------------------------------------------------------------------ [2015-06-29 10:24:44] dmitry@php.net The patch was reverted. Division by zero doesn't throw exception on purpose. It emits warning and returns INF. ------------------------------------------------------------------------ [2015-06-29 09:18:47] ab@php.net Hi Bob, this breaks the discussed behaviors and should be reverted. See the intdiv RFC and http://www.serverphorums.com/read.php?7,1174854 . I guess to fix this the correct behavior needs to be discussed and defined. Thanks. ------------------------------------------------------------------------ [2015-06-28 17:22:22] bwoebi@php.net yes, the Error class which extends Throwable. … All catchable ;-) ------------------------------------------------------------------------ [2015-06-28 16:41:33] cameron dot adam+php at gmail dot com OK, just to be clear... a *catchable* error, yes? Thanks for jumping on this so quickly, btw :-) -- Adam ------------------------------------------------------------------------ [2015-06-28 16:25:32] bwoebi@php.net Note that it now all is an Error. We may eventually change it to a more specific error, but Error is good enough for the moment, I think. See http://git.php.net/?p=php-src.git;a=blob;f=Zend/tests/bug69957.phpt;h=d6f441b6ea93ae50607ddbeb118aca32361691f3;hb=fb08798c9f0ea820d567668d0cea4833dc61dd8e for current behavior. ------------------------------------------------------------------------ The remainder of the comments for this report are too long. To view the rest of the comments, please view the bug report online at https://bugs.php.net/bug.php?id=69957 -- Edit this bug report at https://bugs.php.net/bug.php?id=69957&edit=1

« previous php.bugs (#193987) next »