``` + + if (FAILED(hr)) hr = NS_E_NO_MORE_SAMPLES; ```
This doesn't feel like the most idiomatic place to put this. (It also doesn't match the style of the rest of the module.) Likely better would be to replace 'return hr' with 'return NS_E_NO_MORE_SAMPLES' in wm_reader_read_stream_sample().