Newsgroups: php.internals Path: news.php.net Xref: news.php.net php.internals:26955 Return-Path: Mailing-List: contact internals-help@lists.php.net; run by ezmlm Delivered-To: mailing list internals@lists.php.net Received: (qmail 91283 invoked by uid 1010); 14 Dec 2006 16:27:41 -0000 Delivered-To: ezmlm-scan-internals@lists.php.net Delivered-To: ezmlm-internals@lists.php.net Received: (qmail 91262 invoked from network); 14 Dec 2006 16:27:41 -0000 Received: from unknown (HELO lists.php.net) (127.0.0.1) by localhost with SMTP; 14 Dec 2006 16:27:41 -0000 Authentication-Results: pb1.pair.com smtp.mail=andrei@gravitonic.com; spf=permerror; sender-id=unknown Authentication-Results: pb1.pair.com header.from=andrei@gravitonic.com; sender-id=unknown Received-SPF: error (pb1.pair.com: domain gravitonic.com from 204.11.219.139 cause and error) X-PHP-List-Original-Sender: andrei@gravitonic.com X-Host-Fingerprint: 204.11.219.139 lerdorf.com Linux 2.5 (sometimes 2.4) (4) Received: from [204.11.219.139] ([204.11.219.139:33650] helo=lerdorf.com) by pb1.pair.com (ecelerity 2.1.1.9-wez r(12769M)) with ESMTP id EC/0C-04864-85B71854 for ; Thu, 14 Dec 2006 11:27:39 -0500 Received: from [66.228.175.145] (borndress-lm.corp.yahoo.com [66.228.175.145]) (authenticated bits=0) by lerdorf.com (8.13.8/8.13.8/Debian-3) with ESMTP id kBEGR10B028706; Thu, 14 Dec 2006 08:27:01 -0800 In-Reply-To: <10.23.32177.CA640854@pb1.pair.com> References: <10.23.32177.CA640854@pb1.pair.com> Mime-Version: 1.0 (Apple Message framework v624) Content-Type: text/plain; charset=US-ASCII; format=flowed Message-ID: <2da4c44996eb06fb90d7141e5e5ebc02@gravitonic.com> Content-Transfer-Encoding: 7bit Cc: internals@lists.php.net Date: Thu, 14 Dec 2006 08:27:15 -0800 To: "Ron Korving" X-Mailer: Apple Mail (2.624) Subject: Re: [PHP-DEV] [PATCH] 1 small optimization and 1 cleanup From: andrei@gravitonic.com (Andrei Zmievski) On Dec 13, 2006, at 10:28 AM, Ron Korving wrote: > + > + if (pce) > + { > + if (extra) *extra = pce->extra; > + if (preg_options) *preg_options = pce->preg_options; > + if (compile_options) *compile_options = pce->compile_options; > + return pce->re; > } If you rewrite it, then please adhere to the code style that was used, i.e. with explicit braces for if() statements. -Andrei