diff options
| author | Peter Atashian <retep998@gmail.com> | 2014-08-07 16:40:12 -0400 |
|---|---|---|
| committer | Peter Atashian <retep998@gmail.com> | 2014-08-07 16:40:12 -0400 |
| commit | 24ebbb442014d0bca9f32b36c25f739bf18146bf (patch) | |
| tree | 3b57f5cadd35cbec3c1f983793aed1149db2116b /src/libnative | |
| parent | feb219d23fd4236fc69ec86e34c088e232289534 (diff) | |
| download | rust-24ebbb442014d0bca9f32b36c25f739bf18146bf.tar.gz rust-24ebbb442014d0bca9f32b36c25f739bf18146bf.zip | |
windows: Fix INVALID_HANDLE_VALUE
Made INVALID_HANDLE_VALUE actually a HANDLE. Removed all useless casts during INVALID_HANDLE_VALUE comparisons. Signed-off-by: Peter Atashian <retep998@gmail.com>
Diffstat (limited to 'src/libnative')
| -rw-r--r-- | src/libnative/io/file_win32.rs | 6 | ||||
| -rw-r--r-- | src/libnative/io/pipe_win32.rs | 10 | ||||
| -rw-r--r-- | src/libnative/io/process.rs | 10 |
3 files changed, 13 insertions, 13 deletions
diff --git a/src/libnative/io/file_win32.rs b/src/libnative/io/file_win32.rs index a024d498ef3..fe29c024529 100644 --- a/src/libnative/io/file_win32.rs +++ b/src/libnative/io/file_win32.rs @@ -320,7 +320,7 @@ pub fn open(path: &CString, fm: rtio::FileMode, fa: rtio::FileAccess) dwFlagsAndAttributes, ptr::mut_null()) }; - if handle == libc::INVALID_HANDLE_VALUE as libc::HANDLE { + if handle == libc::INVALID_HANDLE_VALUE { Err(super::last_error()) } else { let fd = unsafe { @@ -368,7 +368,7 @@ pub fn readdir(p: &CString) -> IoResult<Vec<CString>> { let wfd_ptr = malloc_raw(rust_list_dir_wfd_size() as uint); let find_handle = libc::FindFirstFileW(path.as_ptr(), wfd_ptr as libc::HANDLE); - if find_handle as libc::c_int != libc::INVALID_HANDLE_VALUE { + if find_handle != libc::INVALID_HANDLE_VALUE { let mut paths = vec!(); let mut more_files = 1 as libc::c_int; while more_files != 0 { @@ -440,7 +440,7 @@ pub fn readlink(p: &CString) -> IoResult<CString> { libc::FILE_ATTRIBUTE_NORMAL, ptr::mut_null()) }; - if handle as int == libc::INVALID_HANDLE_VALUE as int { + if handle == libc::INVALID_HANDLE_VALUE { return Err(super::last_error()) } // Specify (sz - 1) because the documentation states that it's the size diff --git a/src/libnative/io/pipe_win32.rs b/src/libnative/io/pipe_win32.rs index 87129bba845..717915e5d23 100644 --- a/src/libnative/io/pipe_win32.rs +++ b/src/libnative/io/pipe_win32.rs @@ -223,7 +223,7 @@ impl UnixStream { libc::FILE_FLAG_OVERLAPPED, ptr::mut_null()) }; - if result != libc::INVALID_HANDLE_VALUE as libc::HANDLE { + if result != libc::INVALID_HANDLE_VALUE { return Some(result) } @@ -238,7 +238,7 @@ impl UnixStream { libc::FILE_FLAG_OVERLAPPED, ptr::mut_null()) }; - if result != libc::INVALID_HANDLE_VALUE as libc::HANDLE { + if result != libc::INVALID_HANDLE_VALUE { return Some(result) } } @@ -253,7 +253,7 @@ impl UnixStream { libc::FILE_FLAG_OVERLAPPED, ptr::mut_null()) }; - if result != libc::INVALID_HANDLE_VALUE as libc::HANDLE { + if result != libc::INVALID_HANDLE_VALUE { return Some(result) } } @@ -565,7 +565,7 @@ impl UnixListener { // and such. let addr_v = try!(to_utf16(addr)); let ret = unsafe { pipe(addr_v.as_ptr(), true) }; - if ret == libc::INVALID_HANDLE_VALUE as libc::HANDLE { + if ret == libc::INVALID_HANDLE_VALUE { Err(super::last_error()) } else { Ok(UnixListener { handle: ret, name: addr.clone() }) @@ -680,7 +680,7 @@ impl UnixAcceptor { // create a second server pipe. If this fails, we disconnect the // connected client and return an error (see comments above). let new_handle = unsafe { pipe(name.as_ptr(), false) }; - if new_handle == libc::INVALID_HANDLE_VALUE as libc::HANDLE { + if new_handle == libc::INVALID_HANDLE_VALUE { let ret = Err(super::last_error()); // If our disconnection fails, then there's not really a whole lot // that we can do, so fail the task. diff --git a/src/libnative/io/process.rs b/src/libnative/io/process.rs index c89a40d6513..d83e36a5e2a 100644 --- a/src/libnative/io/process.rs +++ b/src/libnative/io/process.rs @@ -359,13 +359,13 @@ fn spawn_process_os(cfg: ProcessConfig, libc::OPEN_EXISTING, 0, ptr::mut_null()); - if *slot == INVALID_HANDLE_VALUE as libc::HANDLE { + if *slot == INVALID_HANDLE_VALUE { return Err(super::last_error()) } } Some(ref fd) => { let orig = get_osfhandle(fd.fd()) as HANDLE; - if orig == INVALID_HANDLE_VALUE as HANDLE { + if orig == INVALID_HANDLE_VALUE { return Err(super::last_error()) } if DuplicateHandle(cur_proc, orig, cur_proc, slot, @@ -450,9 +450,9 @@ fn zeroed_startupinfo() -> libc::types::os::arch::extra::STARTUPINFO { wShowWindow: 0, cbReserved2: 0, lpReserved2: ptr::mut_null(), - hStdInput: libc::INVALID_HANDLE_VALUE as libc::HANDLE, - hStdOutput: libc::INVALID_HANDLE_VALUE as libc::HANDLE, - hStdError: libc::INVALID_HANDLE_VALUE as libc::HANDLE, + hStdInput: libc::INVALID_HANDLE_VALUE, + hStdOutput: libc::INVALID_HANDLE_VALUE, + hStdError: libc::INVALID_HANDLE_VALUE, } } |
