Skip to content

Commit 1f2e78b

Browse files
authoredDec 4, 2024··
Remove guarded declarations case in generated code (#2138)
## Motivation #2131 removed all usages of availability guards. However, the `guarded` Declaration case was not removed even though it's unused. ## Modifications This PR removes the `guarded` case since it's not used anywhere anymore. ## Result Cleaner codebase.
1 parent 962d634 commit 1f2e78b

File tree

2 files changed

+0
-9
lines changed

2 files changed

+0
-9
lines changed
 

Diff for: ‎Sources/GRPCCodeGen/Internal/Renderer/TextBasedRenderer.swift

-2
Original file line numberDiff line numberDiff line change
@@ -857,8 +857,6 @@ struct TextBasedRenderer: RendererProtocol {
857857
renderCommentableDeclaration(comment: comment, declaration: nestedDeclaration)
858858
case let .deprecated(deprecation, nestedDeclaration):
859859
renderDeprecatedDeclaration(deprecation: deprecation, declaration: nestedDeclaration)
860-
case let .guarded(availability, nestedDeclaration):
861-
renderGuardedDeclaration(availability: availability, declaration: nestedDeclaration)
862860
case .variable(let variableDescription): renderVariable(variableDescription)
863861
case .extension(let extensionDescription): renderExtension(extensionDescription)
864862
case .struct(let structDescription): renderStruct(structDescription)

Diff for: ‎Sources/GRPCCodeGen/Internal/StructuredSwiftRepresentation.swift

-7
Original file line numberDiff line numberDiff line change
@@ -791,9 +791,6 @@ indirect enum Declaration: Equatable, Codable, Sendable {
791791
/// A declaration that adds a comment on top of the provided declaration.
792792
case deprecated(DeprecationDescription, Declaration)
793793

794-
/// A declaration that adds an availability guard on top of the provided declaration.
795-
case guarded(AvailabilityDescription, Declaration)
796-
797794
/// A variable declaration.
798795
case variable(VariableDescription)
799796

@@ -1870,7 +1867,6 @@ extension Declaration {
18701867
switch self {
18711868
case .commentable(_, let declaration): return declaration.accessModifier
18721869
case .deprecated(_, let declaration): return declaration.accessModifier
1873-
case .guarded(_, let declaration): return declaration.accessModifier
18741870
case .variable(let variableDescription): return variableDescription.accessModifier
18751871
case .extension(let extensionDescription): return extensionDescription.accessModifier
18761872
case .struct(let structDescription): return structDescription.accessModifier
@@ -1889,9 +1885,6 @@ extension Declaration {
18891885
case .deprecated(let deprecationDescription, var declaration):
18901886
declaration.accessModifier = newValue
18911887
self = .deprecated(deprecationDescription, declaration)
1892-
case .guarded(let availability, var declaration):
1893-
declaration.accessModifier = newValue
1894-
self = .guarded(availability, declaration)
18951888
case .variable(var variableDescription):
18961889
variableDescription.accessModifier = newValue
18971890
self = .variable(variableDescription)

0 commit comments

Comments
 (0)
Please sign in to comment.