Newsgroups: php.internals Path: news.php.net Xref: news.php.net php.internals:92768 Return-Path: Mailing-List: contact internals-help@lists.php.net; run by ezmlm Delivered-To: mailing list internals@lists.php.net Received: (qmail 36433 invoked from network); 26 Apr 2016 08:19:10 -0000 Received: from unknown (HELO lists.php.net) (127.0.0.1) by localhost with SMTP; 26 Apr 2016 08:19:10 -0000 Authentication-Results: pb1.pair.com header.from=yohgaki@gmail.com; sender-id=pass Authentication-Results: pb1.pair.com smtp.mail=yohgaki@gmail.com; spf=pass; sender-id=pass Received-SPF: pass (pb1.pair.com: domain gmail.com designates 209.85.192.68 as permitted sender) X-PHP-List-Original-Sender: yohgaki@gmail.com X-Host-Fingerprint: 209.85.192.68 mail-qg0-f68.google.com Received: from [209.85.192.68] ([209.85.192.68:33107] helo=mail-qg0-f68.google.com) by pb1.pair.com (ecelerity 2.1.1.9-wez r(12769M)) with ESMTP id AC/CA-02401-D742F175 for ; Tue, 26 Apr 2016 04:19:09 -0400 Received: by mail-qg0-f68.google.com with SMTP id 7so529120qgj.0 for ; Tue, 26 Apr 2016 01:19:08 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=gmail.com; s=20120113; h=mime-version:sender:in-reply-to:references:from:date:message-id :subject:to:cc; bh=HGBX9ww0PG76dCGQV8GL6zrAwwB9uq0kR+QGITxuUoI=; b=IYvzgVtnZDz1j/ld6Se1DkoZc2VAG1Ci3zAYB0yx17fsU/KzOdHXYHExnMFkW0aPOk dxdD0o/cPNBuTXs8HJ8m20KlfLSM6ai2Ps4vgqKytaFYF6SfommFDCEm0edDFEvDJqRN pN16/haOhx2s31t3AWb4llWR2CJgAtbbzjWRrTPZoLK/q2wWbCy6ZmvbaEQWgK9MSnJ0 9+HueBCiK2eXiIcWmlEHNhIXoCEA0NM54u+IylJUvNORenuv4gOzhKNlgPXpKzphz7n8 NTwHkR3TzsmArbI0NZIwBf2+xSJNkKiP16uXaC9+XEC/vtFVXb4Rbtefq48zKb/aB6p3 cG0A== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20130820; h=x-gm-message-state:mime-version:sender:in-reply-to:references:from :date:message-id:subject:to:cc; bh=HGBX9ww0PG76dCGQV8GL6zrAwwB9uq0kR+QGITxuUoI=; b=MrKYh8thJmnyCt4me/Si7vbwHUJpJX00OIZAzNxKsZW4d+VogUmeEZL9OPHo2YR5wR 4zgbh46gpUmKtOyEJrJXlWmY3NyLV/qpWmynWJtAG4DTN8Ly6MI68gRBV35Hg4SUH2jQ Fvt6jfUp5F02OtXeiPj/clszeSrUodrqQFSh8D6/JhxyzS2epmCyD0OHtWbnFr1YA65D UPHzyR564FwH02SAQtDPkDV0y01J+LLVNUJgouG5UvP4S+T9JUp+ypJoVo7rF3eDk78g KgQQXpWuiI9CMnL6AjSWXp4vvpHf5I3US0eYhupjIU/pg8hF663hPWPmmv7mxW7iVIwX f9cA== X-Gm-Message-State: AOPr4FX74vr1pcj3Z8PI9SwFAOQUmBChdbifoW778JqGo3zqbOH3tgfVHVZNHW03skQxwMcd5TFT7HwkVGL5kQ== X-Received: by 10.140.92.65 with SMTP id a59mr870083qge.93.1461658746731; Tue, 26 Apr 2016 01:19:06 -0700 (PDT) MIME-Version: 1.0 Sender: yohgaki@gmail.com Received: by 10.140.27.133 with HTTP; Tue, 26 Apr 2016 01:18:27 -0700 (PDT) In-Reply-To: References: Date: Tue, 26 Apr 2016 17:18:27 +0900 X-Google-Sender-Auth: aziSqtDDob3ASPU2V2DCjFsqJzk Message-ID: To: Sara Golemon Cc: PHP internals Content-Type: text/plain; charset=UTF-8 Subject: Re: [PHP-DEV] Re: [RFC] [Discussion] Octal overflow detection From: yohgaki@ohgaki.net (Yasuo Ohgaki) Hi Sara, On Tue, Apr 19, 2016 at 8:24 AM, Sara Golemon wrote: > On Tue, Apr 12, 2016 at 7:38 PM, Sara Golemon wrote: >> https://wiki.php.net/rfc/octal.overload-checking >> Because having this expression evaluate to true makes me sad: ("\000" >> === "\400") >> > I haven't heard any responses on this and wanted to bump it back to > the top of your newsreaders. > > For the record, my inclination is to raise a warning on octal > overflow, but not change behavior (at least, not for 7.1, perhaps > later). > > Note that this warning would bubble up during the lexing phase, so an > opcache would prevent it from raising on every page load. That's > probably fine, the alternative is to signal the parser to bake it into > the ast output and that's way over-engineered. I guess everyone agrees to raise warning. ("\000" === "\400") === TRUE is wrong, simply. Regards, -- Yasuo Ohgaki yohgaki@ohgaki.net