From: Matt Benjamin <matt@linuxbox.com>
Date: Wed, 23 Jun 2010 14:26:12 +0000 (-0400)
Subject: windows  don't include assert.h (and afs support headers) in util_cr.c
X-Git-Tag: openafs-devel-1_5_75~80
X-Git-Url: https://git.michaelhowe.org/gitweb/?a=commitdiff_plain;h=a4b6ac7a4a51c5655d91b971e4cb7d69f7ffc2b7;p=packages%2Fo%2Fopenafs.git

windows  don't include assert.h (and afs support headers) in util_cr.c

Don't include afsconfig.h and afs/param.h in util_cr.c, nor
assert.h.  The assert was included but was in fact not used in
the file, and can be ommitted, per jaltman.  The OpenAFS
includes are then not needed.  An incorrect cast is also changed,
due to jaltman.

Change-Id: I3e1ad7a1d24473d6ad3ce2e401133b25b0a4c795
Reviewed-on: http://gerrit.openafs.org/2234
Reviewed-by: Jeffrey Altman <jaltman@openafs.org>
Tested-by: Jeffrey Altman <jaltman@openafs.org>
---

diff --git a/src/config/util_cr.c b/src/config/util_cr.c
index 4f918fd13..475fc1a1a 100644
--- a/src/config/util_cr.c
+++ b/src/config/util_cr.c
@@ -9,13 +9,9 @@
 
 */
 
-#include <afsconfig.h>
-#include <afs/param.h>
-
 #undef _CRTDBG_MAP_ALLOC
 #include "stdio.h"
 #include "io.h"
-#include <assert.h>
 #include "string.h"
 #include "process.h"
 #include "windows.h"
@@ -329,7 +325,7 @@ SetSysEnv(int argc, char *argv[])
 	   , argv[3]
 	);
     SendMessageTimeout(HWND_BROADCAST, WM_SETTINGCHANGE, 0,
-		       (DWORD) "Environment", SMTO_NORMAL, 1, &dwResult);
+		       (LPARAM) "Environment", SMTO_NORMAL, 1, &dwResult);
     return 0;
 }