<html>

  <head>

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

      http-equiv="Content-Type">

  </head>

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

    <div class="moz-cite-prefix">Le 13/08/2012 09:51, Dmitry Timoshkov a

      écrit :<br>

    </div>

    <blockquote cite="mid:20120813165150.76d243e0.dmitry@baikal.ru"

      type="cite">

      <pre wrap="">Eric Pouech <a class="moz-txt-link-rfc2396E" href="mailto:eric.pouech@orange.fr"><eric.pouech@orange.fr></a> wrote:

</pre>

      <blockquote type="cite">

        <blockquote type="cite">

          <pre wrap="">It would look much better without all those 'ret' + 'else' things.

</pre>

        </blockquote>

        <pre wrap="">well, wine doesn't have coding guidelines...

so I don't know what "look much better" means (project wise)

</pre>

      </blockquote>

      <pre wrap="">

For this particalr patch it means:

1. less code

2. simpler code

3. more readable code

and has nothing to do with coding guidelines for the whole project.

P.S.

I didn't even ask about the point in adding a bunch of spaces between

'MMRESULT' and 'ret' :)

</pre>

    </blockquote>

    <font face="Helvetica, Arial, sans-serif">feel free to do it. I

      won't<br>

      A+<br>

    </font>

  </body>

</html>