On Sat, Mar 06, 2010 at 07:13:06PM +0100, Steinar H. Gunderson wrote: > result = This->m_pFilter->vt->EnumPins(This->m_pFilter, &enum_pins); > if (result || !enum_pins) > { > em = "could not enumerate pins"; > - break; > + break; > } Except that this change shouldn't be here it looks ok to me.