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