"Aric Stewart" <aric at codeweavers.com> wrote: > - wszPatternString++; /* skip ',' */ > + if (hr == S_OK) > + wszPatternString++; /* skip ',' */ > > if (hr == S_OK) > { Why not just move it into the below block? -- Dmitry.