diff options
author | Giampaolo Rodola <g.rodola@gmail.com> | 2017-04-27 21:19:45 +0200 |
---|---|---|
committer | Giampaolo Rodola <g.rodola@gmail.com> | 2017-04-27 21:19:45 +0200 |
commit | c2702117881eaff7bdd3dc3357db11107458e4de (patch) | |
tree | 8e3336b698df5c45bb3b76bfad115ecb6131cae0 /psutil/arch/windows | |
parent | 62fa5070735f77b14d707c63bc1b4ecc0e169ebf (diff) | |
download | psutil-c2702117881eaff7bdd3dc3357db11107458e4de.tar.gz |
windows / c: small refactoring
Diffstat (limited to 'psutil/arch/windows')
-rw-r--r-- | psutil/arch/windows/process_info.c | 15 |
1 files changed, 5 insertions, 10 deletions
diff --git a/psutil/arch/windows/process_info.c b/psutil/arch/windows/process_info.c index bfcbe624..a871282c 100644 --- a/psutil/arch/windows/process_info.c +++ b/psutil/arch/windows/process_info.c @@ -93,8 +93,7 @@ psutil_get_pids(DWORD *numberOfReturnedPIDs) { } if (! EnumProcesses(procArray, procArrayByteSz, &enumReturnSz)) { free(procArray); - PyErr_SetFromWindowsErr(0); - return NULL; + return PyErr_SetFromWindowsErr(0); } } while (enumReturnSz == procArraySz * sizeof(DWORD)); @@ -138,8 +137,7 @@ psutil_pid_is_running(DWORD pid) { // Be strict and raise an exception; the caller is supposed // to take -1 into account. else { - PyErr_SetFromWindowsErr(0); - return -1; + return PyErr_SetFromWindowsErr(err); } } @@ -154,13 +152,10 @@ psutil_pid_is_running(DWORD pid) { CloseHandle(hProcess); // Same as for OpenProcess, assume access denied means there's // a process to deny access to. - if (err == ERROR_ACCESS_DENIED) { + if (err == ERROR_ACCESS_DENIED) return 1; - } - else { - PyErr_SetFromWindowsErr(err); - return -1; - } + else + return PyErr_SetFromWindowsErr(err); } } |