<br><br><div class="gmail_quote">2013/1/9 Johannes Kroll <span dir="ltr"><<a href="mailto:jkroll@lavabit.com" target="_blank">jkroll@lavabit.com</a>></span><br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
On Wed, 9 Jan 2013 16:10:18 +0100<br>
<div><div class="h5">Christian Costa <<a href="mailto:titan.costa@gmail.com">titan.costa@gmail.com</a>> wrote:<br>
<br>
> 2013/1/9 Johannes Kroll <<a href="mailto:jkroll@lavabit.com">jkroll@lavabit.com</a>><br>
><br>
> > On Wed, 09 Jan 2013 10:19:54 +0100<br>
> > Christian Costa <<a href="mailto:titan.costa@gmail.com">titan.costa@gmail.com</a>> wrote:<br>
> ><br>
> > > Le 09/01/2013 04:26, Johannes Kroll a écrit :<br>
> > > > Hi,<br>
> > > ><br>
> > > > On Wed, 09 Jan 2013 03:55:25 +0100<br>
> > > > Christian Costa <<a href="mailto:titan.costa@gmail.com">titan.costa@gmail.com</a>> wrote:<br>
> > > ><br>
> > > >> Hi,<br>
> > > >><br>
> > > >> Please be consistent when using space. 1 space before and after<br>
> > > >> operators (<, ==, =, ...).<br>
> > > > spaces inserted. HTH.<br>
> > > ><br>
> > > > J.<br>
> > > That's better. You forgot some occurences in the for loop though.<br>
> ><br>
> > Is it OK if I leave the rest of the formatting to you? Any style is<br>
> > fine with me, whatever makes you happy and hopefully gets the patch<br>
> > included.<br>
> ><br>
> > Formatting only patch are not allowed. Clean patch and consistency in code<br>
> style help your patch get in.<br>
> Alexandre Julliard is the only judge not me. If he is ok with your patch<br>
> that's fine.<br>
> Anyway having the for loop consistent with rest should not be that hard.<br>
><br>
> for(i= 0; i<lpMidiOutHdr->dwBufferLength; i++)<br>
<br>
</div></div>The original patch was consistent with my coding style. To make<br>
assignments stick out visually, I use no space to the left of the<br>
assignment operator, one space to the right. On comparison operators, I<br>
use space either on both sides, or none. You will see this in all code<br>
I write.<br>
<br>
I attached another version of the patch. I added 3 more spaces and one<br>
line break in a long comment line. You or Alexandre can choose the<br>
version they like best. If you still see style issues, please consider<br>
changing the style to your liking. If that's not possible, the patch<br>
will have to be rejected.<br>
<br></blockquote><div><br></div><div>I would say consistency would be 2 spaces or none.</div><div>Using only 1 space for assignment does not seem very common in wine AFAIK.</div><div>In general it's better to follow the file style or the function to avoid mixing styles</div>
<div>except if you work heavyly on these files.</div><div><br></div><div>  </div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
If anybody sees non-style-related issues with the patch, please let me<br>
know.<br></blockquote><div><br></div><div>dwBytesRecorded is supposed to tell how many bytes are valid. What's wrong with it ?</div><div>  </div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">

<br>
Please keep any replies on the list(s).<br>
<br></blockquote><div><br></div><div>Good mail client which sucks.</div><div>wine-patches is only for patch submission.</div><div> </div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">

Have a nice day.<br>
<span class="HOEnZb"><font color="#888888">Johannes<br>
<br>
<br>
</font></span><br><br>
<br></blockquote></div><br>