Re: dlls/winmm/winealsa/midi.c: Bug in MIDI_CMD_BENDER fixed

2005-07-27 Thread Felix Nawothnig
Johannes Koch wrote: No, you don't. The bitwise shifting operator has a higher precedence than the bitwise or does. In case you didn't know... $ man 7 operator is your friend. :-) Felix

Re: dlls/winmm/winealsa/midi.c: Bug in MIDI_CMD_BENDER fixed

2005-07-27 Thread Johannes Koch
No, you don't. The bitwise shifting operator has a higher precedence than the bitwise or does. You may test it with this programm: #include int main(int agrc, char** argv) { int iTest = 0x42 << 7 | 0x02; int iTest1 = (0x42 << 7) | 0x02; int iTest2 = 0x42 << (7 | 0x02); printf("iTest

Re: dlls/winmm/winealsa/midi.c: Bug in MIDI_CMD_BENDER fixed

2005-07-26 Thread Marcus Meissner
On Tue, Jul 26, 2005 at 11:52:55PM +0200, Johannes Koch wrote: > The pitch value was wrongly converted and has to be centered around zero. > > Changelog >Fixed pitch bending in the alsa midi driver. > > ? patch.diff > Index: dlls/winmm/winealsa/midi.c > ==