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

Reply via email to