windows don't include assert.h (and afs support headers) in util_cr.c
authorMatt Benjamin <matt@linuxbox.com>
Wed, 23 Jun 2010 14:26:12 +0000 (10:26 -0400)
committerJeffrey Altman <jaltman@openafs.org>
Thu, 24 Jun 2010 01:45:50 +0000 (18:45 -0700)
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>

src/config/util_cr.c

index 4f918fd..475fc1a 100644 (file)
@@ -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;
 }