Truncate MIDI SysEx messages after termination byte

[Johannes Kroll]

On Wed, 9 Jan 2013 16:10:18 +0100
Christian Costa <titan.costa at gmail.com> wrote:

> 2013/1/9 Johannes Kroll <jkroll at lavabit.com>
> 
> > On Wed, 09 Jan 2013 10:19:54 +0100
> > Christian Costa <titan.costa at gmail.com> wrote:
> >
> > > Le 09/01/2013 04:26, Johannes Kroll a écrit :
> > > > Hi,
> > > >
> > > > On Wed, 09 Jan 2013 03:55:25 +0100
> > > > Christian Costa <titan.costa at gmail.com> wrote:
> > > >
> > > >> Hi,
> > > >>
> > > >> Please be consistent when using space. 1 space before and after
> > > >> operators (<, ==, =, ...).
> > > > spaces inserted. HTH.
> > > >
> > > > J.
> > > That's better. You forgot some occurences in the for loop though.
> >
> > Is it OK if I leave the rest of the formatting to you? Any style is
> > fine with me, whatever makes you happy and hopefully gets the patch
> > included.
> >
> > Formatting only patch are not allowed. Clean patch and consistency in code
> style help your patch get in.
> Alexandre Julliard is the only judge not me. If he is ok with your patch
> that's fine.
> Anyway having the for loop consistent with rest should not be that hard.
> 
> for(i= 0; i<lpMidiOutHdr->dwBufferLength; i++)

The original patch was consistent with my coding style. To make
assignments stick out visually, I use no space to the left of the
assignment operator, one space to the right. On comparison operators, I
use space either on both sides, or none. You will see this in all code
I write.

I attached another version of the patch. I added 3 more spaces and one
line break in a long comment line. You or Alexandre can choose the
version they like best. If you still see style issues, please consider
changing the style to your liking. If that's not possible, the patch
will have to be rejected.

If anybody sees non-style-related issues with the patch, please let me
know.

Please keep any replies on the list(s).

Have a nice day.
Johannes


-------------- next part --------------
An embedded and charset-unspecified text was scrubbed...
Name: 0001-terminate-sysex-messages.txt
URL: <http://www.winehq.org/pipermail/wine-patches/attachments/20130109/c26de9be/attachment.txt>