<html>

  <head>

    <meta content="text/html; charset=ISO-8859-1"

      http-equiv="Content-Type">

  </head>

  <body bgcolor="#FFFFFF" text="#000000">

    On 2012-05-30 05:10, Francis Giraldeau wrote:

    <blockquote cite="mid:4FC5E421.8060807@gmail.com" type="cite">

      <meta content="text/html; charset=ISO-8859-1"

        http-equiv="Content-Type">

      Le 2012-05-29 20:43, David Goulet a écrit :<br>

      <span style="white-space: pre;">> > + if

        (!config_exists(config_path)) + goto done;<br>

        ><br>

        > Please add {} even for a one liner if().</span><br>

      <br>

      I used checkpatch.pl from kernel scripts and the standard there is

      to remove them. I can add them, but then patches will not pass the

      check. IMHO I would follow the kernel style, since this makes

      easier for everyone to check their patches. Or maybe there are

      options to skip on-line-if-brace check? There is "--ignore BRACES"

      but then errors are reported, only the description is not shown.

      Will wait for your input on this.<br>

    </blockquote>

    It's not because it's the kernel style that it's a good style... ;-)<br>

    <br>

    Yannick<br>

  </body>

</html>