"James Hawkins" <truiken(a)gmail.com> writes:
+static const WCHAR insert_fmts[][100] = +{ + {'I','N','S','E','R','T',' ','I','N','T','O',' ','`','%','s','`',' ','(',' ',0}, + {'`','%','s','`',',',' ',0}, + {' ',')',' ','V','A','L','U','E','S',' ','(',' ',0}, + {'\'','%','s','\'',',',' ',0}, + {'%','s',',',' ',0}, + {' ',')',0}, +};
It would be cleaner to use separate variables for the separate strings, there's no reason this needs to be an array. It would avoid wasting a lot of space, and make it more explicit what the strings mean, unlike something like insert_fmts[3]. Also since all you do is copy strings, it may be even cleaner to avoid sprintf and simply strcat the strings and the separators. -- Alexandre Julliard julliard(a)winehq.org