ReceiveTempSMS
/
Sweden-Number
1
0
Fork 0
Code Issues Pull Requests Releases Wiki Activity

msiexec: Invert the return values of the boolean functions so that 

their names make more sense.
This commit is contained in:
Alexandre Julliard 2006-07-14 12:38:20 +02:00
parent 8c3fc3a889
commit 4cf14960ba
1 changed files with 36 additions and 36 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

72
programs/msiexec/msiexec.c
View File

@ -219,21 +219,21 @@ static BOOL msi_strequal(LPCWSTR str1, LPCSTR str2)
len = MultiByteToWideChar( CP_ACP, 0, str2, -1, NULL, 0);
if( !len )
return TRUE;
return FALSE;
if( lstrlenW(str1) != (len-1) )
return TRUE;
return FALSE;
strW = HeapAlloc(GetProcessHeap(), 0, sizeof(WCHAR)*len);
MultiByteToWideChar( CP_ACP, 0, str2, -1, strW, len);
ret = CompareStringW(GetThreadLocale(), NORM_IGNORECASE, str1, len, strW, len);
HeapFree(GetProcessHeap(), 0, strW);
return (ret != CSTR_EQUAL);
return (ret == CSTR_EQUAL);
}
/* prefix is hyphen or dash, and str1 is the same as str2, ignoring case */
static BOOL msi_option_equal(LPCWSTR str1, LPCSTR str2)
{
if (str1[0] != '/' && str1[0] != '-')
return TRUE;
return FALSE;
/* skip over the hyphen or slash */
return msi_strequal(str1 + 1, str2);
@ -247,21 +247,21 @@ static BOOL msi_strprefix(LPCWSTR str1, LPCSTR str2)
len = MultiByteToWideChar( CP_ACP, 0, str2, -1, NULL, 0);
if( !len )
return TRUE;
return FALSE;
if( lstrlenW(str1) < (len-1) )
return TRUE;
return FALSE;
strW = HeapAlloc(GetProcessHeap(), 0, sizeof(WCHAR)*len);
MultiByteToWideChar( CP_ACP, 0, str2, -1, strW, len);
ret = CompareStringW(GetThreadLocale(), NORM_IGNORECASE, str1, len-1, strW, len-1);
HeapFree(GetProcessHeap(), 0, strW);
return (ret != CSTR_EQUAL);
return (ret == CSTR_EQUAL);
}
/* prefix is hyphen or dash, and str2 is at the beginning of str1, ignoring case */
static BOOL msi_option_prefix(LPCWSTR str1, LPCSTR str2)
{
if (str1[0] != '/' && str1[0] != '-')
return TRUE;
return FALSE;
/* skip over the hyphen or slash */
return msi_strprefix(str1 + 1, str2);
@ -401,7 +401,7 @@ int main(int argc, char **argv)
* We do that before starting to process the real commandline,
* then overwrite the commandline again.
*/
if(!msi_option_equal(argvW[1], "@"))
if(msi_option_equal(argvW[1], "@"))
{
if(!process_args_from_reg( argvW[2], &argc, &argvW ))
return 1;
@ -411,15 +411,15 @@ int main(int argc, char **argv)
{
WINE_TRACE("argvW[%d] = %s\n", i, wine_dbgstr_w(argvW[i]));
if (!msi_option_equal(argvW[i], "regserver"))
if (msi_option_equal(argvW[i], "regserver"))
{
FunctionRegServer = TRUE;
}
else if (!msi_option_equal(argvW[i], "unregserver") || !msi_option_equal(argvW[i], "unregister"))
else if (msi_option_equal(argvW[i], "unregserver") || msi_option_equal(argvW[i], "unregister"))
{
FunctionUnregServer = TRUE;
}
else if(!msi_option_prefix(argvW[i], "i"))
else if(msi_option_prefix(argvW[i], "i"))
{
LPWSTR argvWi = argvW[i];
FunctionInstall = TRUE;
@ -435,7 +435,7 @@ int main(int argc, char **argv)
}
PackageName = argvWi;
}
else if(!msi_option_equal(argvW[i], "a"))
else if(msi_option_equal(argvW[i], "a"))
{
FunctionInstall = TRUE;
FunctionInstallAdmin = TRUE;
@ -447,7 +447,7 @@ int main(int argc, char **argv)
PackageName = argvW[i];
StringListAppend(&property_list, ActionAdmin);
}
else if(!msi_option_prefix(argvW[i], "f"))
else if(msi_option_prefix(argvW[i], "f"))
{
int j;
int len = lstrlenW(argvW[i]);
@ -515,7 +515,7 @@ int main(int argc, char **argv)
WINE_TRACE("argvW[%d] = %s\n", i, wine_dbgstr_w(argvW[i]));
PackageName = argvW[i];
}
else if(!msi_option_equal(argvW[i], "x"))
else if(msi_option_equal(argvW[i], "x"))
{
FunctionInstall = TRUE;
i++;
@ -525,7 +525,7 @@ int main(int argc, char **argv)
PackageName = argvW[i];
StringListAppend(&property_list, RemoveAll);
}
else if(!msi_option_prefix(argvW[i], "j"))
else if(msi_option_prefix(argvW[i], "j"))
{
int j;
int len = lstrlenW(argvW[i]);
@ -553,7 +553,7 @@ int main(int argc, char **argv)
WINE_TRACE("argvW[%d] = %s\n", i, wine_dbgstr_w(argvW[i]));
PackageName = argvW[i];
}
else if(!msi_strequal(argvW[i], "u"))
else if(msi_strequal(argvW[i], "u"))
{
FunctionAdvertise = TRUE;
AdvertiseMode = ADVERTISEFLAGS_USERASSIGN;
@ -563,7 +563,7 @@ int main(int argc, char **argv)
WINE_TRACE("argvW[%d] = %s\n", i, wine_dbgstr_w(argvW[i]));
PackageName = argvW[i];
}
else if(!msi_strequal(argvW[i], "m"))
else if(msi_strequal(argvW[i], "m"))
{
FunctionAdvertise = TRUE;
AdvertiseMode = ADVERTISEFLAGS_MACHINEASSIGN;
@ -573,7 +573,7 @@ int main(int argc, char **argv)
WINE_TRACE("argvW[%d] = %s\n", i, wine_dbgstr_w(argvW[i]));
PackageName = argvW[i];
}
else if(!msi_option_equal(argvW[i], "t"))
else if(msi_option_equal(argvW[i], "t"))
{
i++;
if(i >= argc)
@ -581,7 +581,7 @@ int main(int argc, char **argv)
WINE_TRACE("argvW[%d] = %s\n", i, wine_dbgstr_w(argvW[i]));
StringListAppend(&transform_list, argvW[i]);
}
else if(!msi_option_equal(argvW[i], "g"))
else if(msi_option_equal(argvW[i], "g"))
{
i++;
if(i >= argc)
@ -589,7 +589,7 @@ int main(int argc, char **argv)
WINE_TRACE("argvW[%d] = %s\n", i, wine_dbgstr_w(argvW[i]));
Language = msi_atou(argvW[i]);
}
else if(!msi_option_prefix(argvW[i], "l"))
else if(msi_option_prefix(argvW[i], "l"))
{
int j;
int len = lstrlenW(argvW[i]);
@ -680,7 +680,7 @@ int main(int argc, char **argv)
ExitProcess(1);
}
}
else if(!msi_option_equal(argvW[i], "p"))
else if(msi_option_equal(argvW[i], "p"))
{
FunctionPatch = TRUE;
i++;
@ -689,37 +689,37 @@ int main(int argc, char **argv)
WINE_TRACE("argvW[%d] = %s\n", i, wine_dbgstr_w(argvW[i]));
PatchFileName = argvW[i];
}
else if(!msi_option_prefix(argvW[i], "q"))
else if(msi_option_prefix(argvW[i], "q"))
{
if(lstrlenW(argvW[i]) == 2 || !msi_strequal(argvW[i]+2, "n"))
if(lstrlenW(argvW[i]) == 2 || msi_strequal(argvW[i]+2, "n"))
{
InstallUILevel = INSTALLUILEVEL_NONE;
}
else if(!msi_strequal(argvW[i]+2, "b"))
else if(msi_strequal(argvW[i]+2, "b"))
{
InstallUILevel = INSTALLUILEVEL_BASIC;
}
else if(!msi_strequal(argvW[i]+2, "r"))
else if(msi_strequal(argvW[i]+2, "r"))
{
InstallUILevel = INSTALLUILEVEL_REDUCED;
}
else if(!msi_strequal(argvW[i]+2, "f"))
else if(msi_strequal(argvW[i]+2, "f"))
{
InstallUILevel = INSTALLUILEVEL_FULL|INSTALLUILEVEL_ENDDIALOG;
}
else if(!msi_strequal(argvW[i]+2, "n+"))
else if(msi_strequal(argvW[i]+2, "n+"))
{
InstallUILevel = INSTALLUILEVEL_NONE|INSTALLUILEVEL_ENDDIALOG;
}
else if(!msi_strequal(argvW[i]+2, "b+"))
else if(msi_strequal(argvW[i]+2, "b+"))
{
InstallUILevel = INSTALLUILEVEL_BASIC|INSTALLUILEVEL_ENDDIALOG;
}
else if(!msi_strequal(argvW[i]+2, "b-"))
else if(msi_strequal(argvW[i]+2, "b-"))
{
InstallUILevel = INSTALLUILEVEL_BASIC|INSTALLUILEVEL_PROGRESSONLY;
}
else if(!msi_strequal(argvW[i]+2, "b+!"))
else if(msi_strequal(argvW[i]+2, "b+!"))
{
InstallUILevel = INSTALLUILEVEL_BASIC|INSTALLUILEVEL_ENDDIALOG;
WINE_FIXME("Unknown modifier: !\n");
@ -730,7 +730,7 @@ int main(int argc, char **argv)
wine_dbgstr_w(argvW[i]+2));
}
}
else if(!msi_option_equal(argvW[i], "y"))
else if(msi_option_equal(argvW[i], "y"))
{
FunctionDllRegisterServer = TRUE;
i++;
@ -739,7 +739,7 @@ int main(int argc, char **argv)
WINE_TRACE("argvW[%d] = %s\n", i, wine_dbgstr_w(argvW[i]));
DllName = argvW[i];
}
else if(!msi_option_equal(argvW[i], "z"))
else if(msi_option_equal(argvW[i], "z"))
{
FunctionDllUnregisterServer = TRUE;
i++;
@ -748,16 +748,16 @@ int main(int argc, char **argv)
WINE_TRACE("argvW[%d] = %s\n", i, wine_dbgstr_w(argvW[i]));
DllName = argvW[i];
}
else if(!msi_option_equal(argvW[i], "h") || !msi_option_equal(argvW[i], "?"))
else if(msi_option_equal(argvW[i], "h") || msi_option_equal(argvW[i], "?"))
{
ShowUsage(0);
}
else if(!msi_option_equal(argvW[i], "m"))
else if(msi_option_equal(argvW[i], "m"))
{
FunctionUnknown = TRUE;
WINE_FIXME("Unknown parameter /m\n");
}
else if(!msi_option_equal(argvW[i], "D"))
else if(msi_option_equal(argvW[i], "D"))
{
FunctionUnknown = TRUE;
WINE_FIXME("Unknown parameter /D\n");