Newsgroups: php.internals Path: news.php.net Xref: news.php.net php.internals:61571 Return-Path: Mailing-List: contact internals-help@lists.php.net; run by ezmlm Delivered-To: mailing list internals@lists.php.net Received: (qmail 71090 invoked from network); 20 Jul 2012 16:39:05 -0000 Received: from unknown (HELO lists.php.net) (127.0.0.1) by localhost with SMTP; 20 Jul 2012 16:39:05 -0000 Authentication-Results: pb1.pair.com header.from=lester@lsces.co.uk; sender-id=unknown Authentication-Results: pb1.pair.com smtp.mail=lester@lsces.co.uk; spf=permerror; sender-id=unknown Received-SPF: error (pb1.pair.com: domain lsces.co.uk from 213.123.26.186 cause and error) X-PHP-List-Original-Sender: lester@lsces.co.uk X-Host-Fingerprint: 213.123.26.186 c2beaomr08.btconnect.com Received: from [213.123.26.186] ([213.123.26.186:63761] helo=mail.btconnect.com) by pb1.pair.com (ecelerity 2.1.1.9-wez r(12769M)) with ESMTP id AF/78-18983-7A989005 for ; Fri, 20 Jul 2012 12:39:04 -0400 Received: from host81-138-11-136.in-addr.btopenworld.com (EHLO _10.0.0.5_) ([81.138.11.136]) by c2beaomr08.btconnect.com with ESMTP id IGP32041; Fri, 20 Jul 2012 17:39:00 +0100 (BST) Message-ID: <500989A4.8090501@lsces.co.uk> Date: Fri, 20 Jul 2012 17:39:00 +0100 User-Agent: Mozilla/5.0 (X11; Linux x86_64; rv:13.0) Gecko/20120604 Firefox/13.0 SeaMonkey/2.10 MIME-Version: 1.0 To: PHP internals References: <5007D707.2070100@hoa-project.net> <64.87.18983.E6379005@pb1.pair.com> In-Reply-To: <64.87.18983.E6379005@pb1.pair.com> Content-Type: text/plain; charset=UTF-8; format=flowed Content-Transfer-Encoding: 7bit X-Mirapoint-IP-Reputation: reputation=Fair-1, source=Queried, refid=tid=0001.0A0B0303.500989A4.008E, actions=tag X-Junkmail-Premium-Raw: score=7/50, refid=2.7.2:2012.7.20.155117:17:7.944, ip=81.138.11.136, rules=__MOZILLA_MSGID, __HAS_MSGID, __SANE_MSGID, __HAS_FROM, __USER_AGENT, __MIME_VERSION, __TO_MALFORMED_2, __BOUNCE_CHALLENGE_SUBJ, __BOUNCE_NDR_SUBJ_EXEMPT, __SUBJ_ALPHA_END, __CT, __CT_TEXT_PLAIN, __CTE, __ANY_URI, __URI_NO_MAILTO, __URI_NO_WWW, __CP_URI_IN_BODY, BODY_ENDS_IN_URL, BODY_SIZE_1100_1199, BODYTEXTP_SIZE_3000_LESS, __MIME_TEXT_ONLY, RDNS_GENERIC_POOLED, HTML_00_01, HTML_00_10, BODY_SIZE_5000_LESS, RDNS_SUSP_GENERIC, RDNS_SUSP, BODY_SIZE_2000_LESS, BODY_SIZE_7000_LESS X-Junkmail-Status: score=10/50, host=c2beaomr08.btconnect.com X-Junkmail-Signature-Raw: score=unknown, refid=str=0001.0A0B020A.500989A4.0173:SCFSTAT14830815,ss=1,re=-4.000,fgs=0, ip=0.0.0.0, so=2011-07-25 19:15:43, dmn=2011-05-27 18:58:46, mode=multiengine X-Junkmail-IWF: false Subject: Re: [PHP-DEV] Re: Make try/catch brackets optinal From: lester@lsces.co.uk (Lester Caine) Matthew Weier O'Phinney wrote: > I've run into problems with brace-less blocks many, many times over > the years. Clearly, enough other people have as well that any serious > coding standard includes a clause requiring that all blocks use > braces. I see no reason to add another context in which braces could > be omitted. It's nice to hear that it is not only me ... Brought to mind a problem that popped up in a patch earlier this year. Not my patch, but the contributor had missed that brackets were missing, and his fix worked for him, but the following statement was now enabled in the wrong place. FORTUNATELY this was manually merged, and the mistake spotted, but with the increasing use of automatic merges ... I think I would support making brackets compulsory. -- Lester Caine - G8HFL ----------------------------- Contact - http://lsces.co.uk/wiki/?page=contact L.S.Caine Electronic Services - http://lsces.co.uk EnquirySolve - http://enquirysolve.com/ Model Engineers Digital Workshop - http://medw.co.uk Rainbow Digital Media - http://rainbowdigitalmedia.co.uk