true, that would be cleaner. I will resubmit. -aric Dmitry Timoshkov wrote: > "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? >