Skip to content

Commit bab1bc5

Browse files
committed
use inout conversion
- this is the situation where it is appropriate
1 parent a2b53bb commit bab1bc5

File tree

1 file changed

+9
-13
lines changed

1 file changed

+9
-13
lines changed

Tests/SystemTests/MachPortTests.swift

Lines changed: 9 additions & 13 deletions
Original file line numberDiff line numberDiff line change
@@ -20,14 +20,12 @@ import System
2020

2121
final class MachPortTests: XCTestCase {
2222
func refCountForMachPortName(name:mach_port_name_t, kind:mach_port_right_t) -> mach_port_urefs_t {
23-
var refCount:mach_port_urefs_t = 0
24-
withUnsafeMutablePointer(to: &refCount) { refCount in
25-
let kr = mach_port_get_refs(mach_task_self_, name, kind, refCount)
26-
if kr == KERN_INVALID_NAME {
27-
refCount.pointee = 0
28-
} else {
29-
XCTAssertEqual(kr, KERN_SUCCESS)
30-
}
23+
var refCount:mach_port_urefs_t = .max
24+
let kr = mach_port_get_refs(mach_task_self_, name, kind, &refCount)
25+
if kr == KERN_INVALID_NAME {
26+
refCount = 0
27+
} else {
28+
XCTAssertEqual(kr, KERN_SUCCESS)
3129
}
3230
return refCount
3331
}
@@ -39,11 +37,9 @@ final class MachPortTests: XCTestCase {
3937
}
4038

4139
func testReceiveRightDeallocation() throws {
42-
var name:mach_port_name_t = 0 // Never read
43-
withUnsafeMutablePointer(to:&name) { name in
44-
let kr = mach_port_allocate(mach_task_self_, MACH_PORT_RIGHT_RECEIVE, name)
45-
XCTAssertEqual(kr, KERN_SUCCESS)
46-
}
40+
var name: mach_port_name_t = 0xFFFFFFFF
41+
let kr = mach_port_allocate(mach_task_self_, MACH_PORT_RIGHT_RECEIVE, &name)
42+
XCTAssertEqual(kr, KERN_SUCCESS)
4743

4844
XCTAssertNotEqual(name, 0xFFFFFFFF)
4945

0 commit comments

Comments
 (0)