Michael Stefaniuc : avifil32: Remove some redundant "not NULL" checks ( coccicheck).
Alexandre Julliard
julliard at winehq.org
Tue May 18 12:32:57 CDT 2010
Module: wine
Branch: master
Commit: 6178e86a7415507219e541a2c78b22ed9b3dad71
URL: http://source.winehq.org/git/wine.git/?a=commit;h=6178e86a7415507219e541a2c78b22ed9b3dad71
Author: Michael Stefaniuc <mstefani at redhat.de>
Date: Tue May 18 01:31:01 2010 +0200
avifil32: Remove some redundant "not NULL" checks (coccicheck).
---
dlls/avifil32/api.c | 6 +++---
1 files changed, 3 insertions(+), 3 deletions(-)
diff --git a/dlls/avifil32/api.c b/dlls/avifil32/api.c
index ac3c35e..8ef75d3 100644
--- a/dlls/avifil32/api.c
+++ b/dlls/avifil32/api.c
@@ -1447,7 +1447,7 @@ BOOL WINAPI AVISaveOptions(HWND hWnd, UINT uFlags, INT nStreams,
return AVIERR_BADPARAM;
/* save options in case the user presses cancel */
- if (ppOptions != NULL && nStreams > 1) {
+ if (nStreams > 1) {
pSavedOptions = HeapAlloc(GetProcessHeap(), 0, nStreams * sizeof(AVICOMPRESSOPTIONS));
if (pSavedOptions == NULL)
return FALSE;
@@ -1629,7 +1629,7 @@ HRESULT WINAPI AVISaveVW(LPCWSTR szFile, CLSID *pclsidHandler,
/* remember first video stream -- needed for interleaving */
if (lFirstVideo < 0)
lFirstVideo = curStream;
- } else if (!dwInterleave && plpOptions != NULL) {
+ } else if (!dwInterleave) {
/* check if any non-video stream wants to be interleaved */
WARN("options.flags=0x%X options.dwInterleave=%u\n",plpOptions[curStream]->dwFlags,plpOptions[curStream]->dwInterleaveEvery);
if (plpOptions[curStream] != NULL &&
@@ -1639,7 +1639,7 @@ HRESULT WINAPI AVISaveVW(LPCWSTR szFile, CLSID *pclsidHandler,
/* create de-/compressed stream interface if needed */
pInStreams[curStream] = NULL;
- if (plpOptions != NULL && plpOptions[curStream] != NULL) {
+ if (plpOptions[curStream] != NULL) {
if (plpOptions[curStream]->fccHandler ||
plpOptions[curStream]->lpFormat != NULL) {
DWORD dwKeySave = plpOptions[curStream]->dwKeyFrameEvery;
More information about the wine-cvs
mailing list