The second one – the `)` is at the same indentation level as the line that
opened the block, i.e. you write it the same way as the braces in

    if (foo) {
        bar;
    }


On Fri, Apr 5, 2013 at 3:48 PM, nmalzieu <notifications@github.com> wrote:

> In pynagiosplugins/nagios_check_sql.py:
>
> > +    """
> > +    def __init__(
> > +                self,
> > +                sql_select='COUNT(1)',
> > +                sql_from=None,
> > +                sql_where='1',
> > +                sql_query=None,
> > +                **params
> > +                ):
> > +
> > +        if sql_query == None and sql_where == None:
> > +            raise ValueError("You must specify either sql_query or sql_where")
> > +
> > +        if sql_query is None:
> > +            self.sql_query = """SELECT %s FROM %s WHERE %s""" % (
> > +                sql_select, sql_from, sql_where)
>
> Ok so just to be clear would you have this
>
>         if sql_query is None:
>             self.sql_query = """SELECT %s FROM %s WHERE %s""" % (
>                 sql_select, sql_from, sql_where
>                 )
>
> or this
>
>         if sql_query is None:
>             self.sql_query = """SELECT %s FROM %s WHERE %s""" % (
>                 sql_select, sql_from, sql_where
>             )
>
> —
> Reply to this email directly or view it on GitHub<https://github.com/Work4Labs/pynagiosplugins/pull/4/files#r3681876>
> .
>