Sure.

On Dec 14, 2006, at 9:07 AM, Ron Korving wrote:

I can redo the patch if you want. I do notice the braces aren't always used in if-statements (see preg_get_backref() for a few occasions). Do you want
me to repost it with braces?

- Ron


"Andrei Zmievski" <[EMAIL PROTECTED]> wrote in message
news:[EMAIL PROTECTED]

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

--
PHP Internals - PHP Runtime Development Mailing List
To unsubscribe, visit: http://www.php.net/unsub.php

--
PHP Internals - PHP Runtime Development Mailing List
To unsubscribe, visit: http://www.php.net/unsub.php

Reply via email to