quartz: Free two assert calls from having side effects.

Michael Stefaniuc mstefani at redhat.de
Fri Oct 7 06:26:51 CDT 2011


---
 dlls/quartz/mpegsplit.c |    7 +++++--
 1 files changed, 5 insertions(+), 2 deletions(-)

diff --git a/dlls/quartz/mpegsplit.c b/dlls/quartz/mpegsplit.c
index 059a854..68348a3 100644
--- a/dlls/quartz/mpegsplit.c
+++ b/dlls/quartz/mpegsplit.c
@@ -182,7 +182,8 @@ static HRESULT FillBuffer(MPEGSplitterImpl *This, IMediaSample *pCurrentSample)
     IMediaSample_GetPointer(pCurrentSample, &fbuf);
 
     /* Find the next valid header.. it <SHOULD> be right here */
-    assert(parse_header(fbuf, &length, &This->position) == S_OK);
+    hr = parse_header(fbuf, &length, &This->position);
+    assert(hr == S_OK);
     IMediaSample_SetActualDataLength(pCurrentSample, length);
 
     /* Queue the next sample */
@@ -739,7 +740,9 @@ static HRESULT MPEGSplitter_first_request(LPVOID iface)
     IMediaSample *sample;
 
     TRACE("Seeking? %d\n", This->seek);
-    assert(parse_header(This->header, &length, NULL) == S_OK);
+
+    hr = parse_header(This->header, &length, NULL);
+    assert(hr == S_OK);
 
     if (pin->rtCurrent >= pin->rtStop)
     {
-- 
1.7.4.4



More information about the wine-patches mailing list