Skip to content

Commit f0d3542

Browse files
committed
fix Swift 5.8 compilation as it doesn't implement if and switch expressions (introduced Swift 5.9)
1 parent bed8eb0 commit f0d3542

File tree

1 file changed

+9
-9
lines changed

1 file changed

+9
-9
lines changed

Sources/protoc-gen-swift/GeneratorOptions.swift

Lines changed: 9 additions & 9 deletions
Original file line numberDiff line numberDiff line change
@@ -47,19 +47,19 @@ class GeneratorOptions {
4747

4848
var isAccessLevel: Bool {
4949
switch self {
50-
case .accessLevel: true
51-
default: false
50+
case .accessLevel: return true
51+
default: return false
5252
}
5353
}
5454

5555
var snippet: String {
5656
switch self {
5757
case let .accessLevel(visibility):
58-
"\(visibility) import"
58+
return "\(visibility) import"
5959
case .plain:
60-
"import"
60+
return "import"
6161
case .implementationOnly:
62-
"@_implementationOnly import"
62+
return "@_implementationOnly import"
6363
}
6464
}
6565
}
@@ -169,10 +169,10 @@ class GeneratorOptions {
169169

170170
self.experimentalStripNonfunctionalCodegen = experimentalStripNonfunctionalCodegen
171171

172-
self.importDirective = switch (implementationOnlyImports, useAccessLevelOnImports) {
173-
case (false, false): .plain
174-
case (false, true): .accessLevel(visibility)
175-
case (true, false): .implementationOnly
172+
switch (implementationOnlyImports, useAccessLevelOnImports) {
173+
case (false, false): self.importDirective = .plain
174+
case (false, true): self.importDirective = .accessLevel(visibility)
175+
case (true, false): self.importDirective = .implementationOnly
176176
case (true, true): throw GenerationError.message(message: """
177177
When using access levels on imports the @_implementationOnly option is unnecessary.
178178
Disable @_implementationOnly imports.

0 commit comments

Comments
 (0)