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