Skip to content

Commit ead80fe

Browse files
committed
Foundation: remove overlay for testing unresovled failure
This allows working around an diagnosed failure in the compiler with canonically typed imports.
1 parent e04c57c commit ead80fe

5 files changed

Lines changed: 12 additions & 36 deletions

File tree

Sources/Foundation/FileManager+Win32.swift

Lines changed: 5 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -183,8 +183,8 @@ extension FileManager {
183183
// BACKUP_SEMANTICS are (confusingly) required in order to receive a
184184
// handle to a directory
185185
CreateFileW($0, 0,
186-
FILE_SHARE_READ | FILE_SHARE_WRITE | FILE_SHARE_DELETE,
187-
nil, OPEN_EXISTING, FILE_FLAG_BACKUP_SEMANTICS, nil)
186+
DWORD(FILE_SHARE_READ) | DWORD(FILE_SHARE_WRITE) | DWORD(FILE_SHARE_DELETE),
187+
nil, DWORD(OPEN_EXISTING), DWORD(FILE_FLAG_BACKUP_SEMANTICS), nil)
188188
}
189189
if hFile == INVALID_HANDLE_VALUE {
190190
return try FileManager.default._fileSystemRepresentation(withPath: path) {
@@ -228,8 +228,8 @@ extension FileManager {
228228

229229
var statInfo = stat()
230230
let handle =
231-
CreateFileW(_fsRep, 0, FILE_SHARE_READ, nil, OPEN_EXISTING,
232-
FILE_FLAG_OPEN_REPARSE_POINT | FILE_FLAG_BACKUP_SEMANTICS,
231+
CreateFileW(_fsRep, 0, FILE_SHARE_READ, nil, DWORD(OPEN_EXISTING),
232+
FILE_FLAG_OPEN_REPARSE_POINT | DWORD(FILE_FLAG_BACKUP_SEMANTICS),
233233
nil)
234234
if handle == INVALID_HANDLE_VALUE {
235235
throw _NSErrorWithWindowsError(GetLastError(), reading: false, paths: [path])
@@ -292,7 +292,7 @@ extension FileManager {
292292
FILETIME(from: time_t((modificationTime ?? stat.lastModificationDate).timeIntervalSince1970))
293293

294294
let hFile: HANDLE =
295-
CreateFileW(fsr, GENERIC_WRITE, FILE_SHARE_WRITE, nil, OPEN_EXISTING, 0,
295+
CreateFileW(fsr, GENERIC_WRITE, FILE_SHARE_WRITE, nil, DWORD(OPEN_EXISTING), 0,
296296
nil)
297297
if hFile == INVALID_HANDLE_VALUE {
298298
throw _NSErrorWithWindowsError(GetLastError(), reading: true, paths: [path])

Sources/Foundation/NSPathUtilities.swift

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -653,8 +653,8 @@ internal func _NSCreateTemporaryFile(_ filePath: String) throws -> (Int32, Strin
653653
}
654654
let pathResult = FileManager.default.string(withFileSystemRepresentation: String(decoding: buf, as: UTF16.self), length: wcslen(buf))
655655
guard let h = CreateFileW(buf, GENERIC_READ | GENERIC_WRITE,
656-
FILE_SHARE_READ | FILE_SHARE_WRITE | FILE_SHARE_DELETE,
657-
nil, OPEN_EXISTING, FILE_ATTRIBUTE_NORMAL, nil),
656+
DWORD(FILE_SHARE_READ) | DWORD(FILE_SHARE_WRITE) | DWORD(FILE_SHARE_DELETE),
657+
nil, DWORD(OPEN_EXISTING), DWORD(FILE_ATTRIBUTE_NORMAL), nil),
658658
h != INVALID_HANDLE_VALUE else {
659659
throw _NSErrorWithWindowsError(GetLastError(), reading: false)
660660
}

Sources/Foundation/NSURL.swift

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -926,8 +926,8 @@ extension NSURL {
926926
#if os(Windows)
927927
let hFile: HANDLE = absolutePath.withCString(encodedAs: UTF16.self) {
928928
CreateFileW($0, GENERIC_READ,
929-
FILE_SHARE_READ | FILE_SHARE_WRITE | FILE_SHARE_DELETE,
930-
nil, OPEN_EXISTING, FILE_FLAG_BACKUP_SEMANTICS, nil)
929+
DWORD(FILE_SHARE_READ) | DWORD(FILE_SHARE_WRITE) | DWORD(FILE_SHARE_DELETE),
930+
nil, DWORD(OPEN_EXISTING), DWORD(FILE_FLAG_BACKUP_SEMANTICS), nil)
931931
}
932932
guard hFile == INVALID_HANDLE_VALUE else {
933933
defer { CloseHandle(hFile) }

Sources/Foundation/WinSDK+Extensions.swift

Lines changed: 0 additions & 24 deletions
Original file line numberDiff line numberDiff line change
@@ -17,10 +17,6 @@ internal var FILE_ATTRIBUTE_DIRECTORY: DWORD {
1717
DWORD(WinSDK.FILE_ATTRIBUTE_DIRECTORY)
1818
}
1919

20-
internal var FILE_ATTRIBUTE_NORMAL: DWORD {
21-
DWORD(WinSDK.FILE_ATTRIBUTE_NORMAL)
22-
}
23-
2420
internal var FILE_ATTRIBUTE_HIDDEN: DWORD {
2521
DWORD(WinSDK.FILE_ATTRIBUTE_HIDDEN)
2622
}
@@ -33,26 +29,10 @@ internal var FILE_ATTRIBUTE_REPARSE_POINT: DWORD {
3329
DWORD(WinSDK.FILE_ATTRIBUTE_REPARSE_POINT)
3430
}
3531

36-
internal var FILE_FLAG_BACKUP_SEMANTICS: DWORD {
37-
DWORD(WinSDK.FILE_FLAG_BACKUP_SEMANTICS)
38-
}
39-
4032
internal var FILE_FLAG_OPEN_REPARSE_POINT: DWORD {
4133
DWORD(WinSDK.FILE_FLAG_OPEN_REPARSE_POINT)
4234
}
4335

44-
internal var FILE_SHARE_DELETE: DWORD {
45-
DWORD(WinSDK.FILE_SHARE_DELETE)
46-
}
47-
48-
internal var FILE_SHARE_READ: DWORD {
49-
DWORD(WinSDK.FILE_SHARE_READ)
50-
}
51-
52-
internal var FILE_SHARE_WRITE: DWORD {
53-
DWORD(WinSDK.FILE_SHARE_WRITE)
54-
}
55-
5636
internal var GENERIC_WRITE: DWORD {
5737
DWORD(WinSDK.GENERIC_WRITE)
5838
}
@@ -68,8 +48,4 @@ internal var MOVEFILE_REPLACE_EXISTING: DWORD {
6848
internal var MOVEFILE_WRITE_THROUGH: DWORD {
6949
DWORD(WinSDK.MOVEFILE_WRITE_THROUGH)
7050
}
71-
72-
internal var OPEN_EXISTING: DWORD {
73-
DWORD(WinSDK.OPEN_EXISTING)
74-
}
7551
#endif

Tests/Foundation/TestFileHandle.swift

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -135,9 +135,9 @@ class TestFileHandle : XCTestCase {
135135
// NOTE(compnerd) we need the FILE_FLAG_BACKUP_SEMANTICS so that we
136136
// can create the handle to the directory.
137137
CreateFileW($0, GENERIC_READ,
138-
FILE_SHARE_DELETE | FILE_SHARE_READ | FILE_SHARE_WRITE,
139-
nil, OPEN_EXISTING,
140-
FILE_ATTRIBUTE_NORMAL | FILE_FLAG_BACKUP_SEMANTICS, nil)
138+
DWORD(FILE_SHARE_DELETE) | DWORD(FILE_SHARE_READ) | DWORD(FILE_SHARE_WRITE),
139+
nil, DWORD(OPEN_EXISTING),
140+
DWORD(FILE_ATTRIBUTE_NORMAL) | DWORD(FILE_FLAG_BACKUP_SEMANTICS), nil)
141141
}
142142
if hDirectory == INVALID_HANDLE_VALUE {
143143
fatalError("unable to create handle to current directory")

0 commit comments

Comments
 (0)