On Tue, Dec 9, 2008 at 2:35 AM, Michael Karcher <karcher(a)physik.fu-berlin.de> wrote:
Am Dienstag, den 09.12.2008, 00:39 -0600 schrieb Austin English:
- MSSTYLES_GetNextInteger(lpValue, lpValueEnd, &lpValue, &g); - if(MSSTYLES_GetNextInteger(lpValue, lpValueEnd, &lpValue, &b)) { + if(MSSTYLES_GetNextInteger(lpValue, lpValueEnd, &lpValue, &r) && + MSSTYLES_GetNextInteger(lpValue, lpValueEnd, &lpValue, &g) && + MSSTYLES_GetNextInteger(lpValue, lpValueEnd, &lpValue, &b)) { I would like to see the last two lines indented more than the "if" line. Take a look at line 748 in msstyles.c to find a precedence case in the file you are patching. In short, that would mean
+ if(MSSTYLES_GetNextInteger(...) && + MSSTYLES_GetNextInteger(...) && + MSSTYLES_GetNextInteger(...)) + {
Regards, Michael Karcher
Patch was already committed. If I fix something else in this file, I'll update this as well. Otherwise, feel free to do so. -- -Austin