Skip to content

Commit e405bab

Browse files
committed
swift-api-digester: separate migration kind from the other node annotation kind. NFC
1 parent bceaab3 commit e405bab

File tree

3 files changed

+27
-39
lines changed

3 files changed

+27
-39
lines changed

include/swift/IDE/DigesterEnums.def

Lines changed: 24 additions & 21 deletions
Original file line numberDiff line numberDiff line change
@@ -6,6 +6,10 @@
66
#define NODE_ANNOTATION(NAME)
77
#endif
88

9+
#ifndef NODE_ANNOTATION_CHANGE_KIND
10+
#define NODE_ANNOTATION_CHANGE_KIND(NAME) NODE_ANNOTATION(NAME)
11+
#endif
12+
913
#ifndef DECL_ATTR
1014
#define DECL_ATTR(NAME)
1115
#endif
@@ -49,37 +53,35 @@ NODE_KIND(DeclTypeAlias, TypeAlias)
4953
NODE_ANNOTATION(Added)
5054
NODE_ANNOTATION(Removed)
5155
NODE_ANNOTATION(Updated)
52-
NODE_ANNOTATION(RemovingErrorParam)
53-
NODE_ANNOTATION(ImplicitOptionalToOptional)
54-
NODE_ANNOTATION(OptionalToImplicitOptional)
55-
NODE_ANNOTATION(WrapOptional)
56-
NODE_ANNOTATION(WrapImplicitOptional)
57-
NODE_ANNOTATION(UnwrapOptional)
58-
NODE_ANNOTATION(GenericParamUpCast)
59-
NODE_ANNOTATION(GenericParamDownCast)
60-
NODE_ANNOTATION(TypeAliasChangeFromInt)
61-
NODE_ANNOTATION(GetterToProperty)
62-
NODE_ANNOTATION(SetterToProperty)
6356
NODE_ANNOTATION(PropertyName)
64-
NODE_ANNOTATION(TypeRewritten)
6557
NODE_ANNOTATION(TypeRewrittenLeft)
6658
NODE_ANNOTATION(TypeRewrittenRight)
67-
NODE_ANNOTATION(ModernizeEnum)
68-
NODE_ANNOTATION(UnwrapUnmanaged)
6959
NODE_ANNOTATION(RemovedDecl)
70-
NODE_ANNOTATION(Rename)
7160
NODE_ANNOTATION(RenameOldName)
7261
NODE_ANNOTATION(RenameNewName)
7362
NODE_ANNOTATION(NowThrowing)
7463
NODE_ANNOTATION(NowMutating)
7564
NODE_ANNOTATION(StaticChange)
7665
NODE_ANNOTATION(OwnershipChange)
77-
NODE_ANNOTATION(DictionaryKeyUpdate)
78-
NODE_ANNOTATION(OptionalDictionaryKeyUpdate)
79-
NODE_ANNOTATION(ArrayMemberUpdate)
80-
NODE_ANNOTATION(OptionalArrayMemberUpdate)
81-
NODE_ANNOTATION(SimpleStringRepresentableUpdate)
82-
NODE_ANNOTATION(SimpleOptionalStringRepresentableUpdate)
66+
67+
NODE_ANNOTATION_CHANGE_KIND(ImplicitOptionalToOptional)
68+
NODE_ANNOTATION_CHANGE_KIND(OptionalToImplicitOptional)
69+
NODE_ANNOTATION_CHANGE_KIND(WrapOptional)
70+
NODE_ANNOTATION_CHANGE_KIND(WrapImplicitOptional)
71+
NODE_ANNOTATION_CHANGE_KIND(UnwrapOptional)
72+
NODE_ANNOTATION_CHANGE_KIND(GetterToProperty)
73+
NODE_ANNOTATION_CHANGE_KIND(SetterToProperty)
74+
NODE_ANNOTATION_CHANGE_KIND(TypeRewritten)
75+
NODE_ANNOTATION_CHANGE_KIND(ModernizeEnum)
76+
NODE_ANNOTATION_CHANGE_KIND(UnwrapUnmanaged)
77+
NODE_ANNOTATION_CHANGE_KIND(Rename)
78+
NODE_ANNOTATION_CHANGE_KIND(DictionaryKeyUpdate)
79+
NODE_ANNOTATION_CHANGE_KIND(OptionalDictionaryKeyUpdate)
80+
NODE_ANNOTATION_CHANGE_KIND(ArrayMemberUpdate)
81+
NODE_ANNOTATION_CHANGE_KIND(OptionalArrayMemberUpdate)
82+
NODE_ANNOTATION_CHANGE_KIND(SimpleStringRepresentableUpdate)
83+
NODE_ANNOTATION_CHANGE_KIND(SimpleOptionalStringRepresentableUpdate)
84+
8385

8486
DECL_ATTR(deprecated)
8587
DECL_ATTR(fixedLayout)
@@ -167,5 +169,6 @@ SPECIAL_CASE_ID(ToUIntMax)
167169
#undef KNOWN_PROTOCOL
168170
#undef KEY
169171
#undef DECL_ATTR
172+
#undef NODE_ANNOTATION_CHANGE_KIND
170173
#undef NODE_ANNOTATION
171174
#undef NODE_KIND

lib/IDE/APIDigesterData.cpp

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -46,7 +46,7 @@ SDKNodeKind swift::ide::api::parseSDKNodeKind(StringRef Content) {
4646

4747
NodeAnnotation swift::ide::api::parseSDKNodeAnnotation(StringRef Content) {
4848
return llvm::StringSwitch<NodeAnnotation>(Content)
49-
#define NODE_ANNOTATION(NAME) .Case(#NAME, NodeAnnotation::NAME)
49+
#define NODE_ANNOTATION_CHANGE_KIND(NAME) .Case(#NAME, NodeAnnotation::NAME)
5050
#include "swift/IDE/DigesterEnums.def"
5151
;
5252
}

tools/swift-api-digester/swift-api-digester.cpp

Lines changed: 2 additions & 17 deletions
Original file line numberDiff line numberDiff line change
@@ -3017,23 +3017,8 @@ class DiffItemEmitter : public SDKNodeVisitor {
30173017
handleAnnotations(Node, Parent,
30183018
isa<SDKNodeType>(Node) ? getIndexString(Node) : "0",
30193019
{
3020-
NodeAnnotation::WrapOptional,
3021-
NodeAnnotation::UnwrapOptional,
3022-
NodeAnnotation::ImplicitOptionalToOptional,
3023-
NodeAnnotation::OptionalToImplicitOptional,
3024-
NodeAnnotation::UnwrapUnmanaged,
3025-
NodeAnnotation::TypeRewritten,
3026-
NodeAnnotation::SetterToProperty,
3027-
NodeAnnotation::GetterToProperty,
3028-
NodeAnnotation::ModernizeEnum,
3029-
NodeAnnotation::Rename,
3030-
NodeAnnotation::NowThrowing,
3031-
NodeAnnotation::DictionaryKeyUpdate,
3032-
NodeAnnotation::OptionalDictionaryKeyUpdate,
3033-
NodeAnnotation::ArrayMemberUpdate,
3034-
NodeAnnotation::OptionalArrayMemberUpdate,
3035-
NodeAnnotation::SimpleStringRepresentableUpdate,
3036-
NodeAnnotation::SimpleOptionalStringRepresentableUpdate,
3020+
#define NODE_ANNOTATION_CHANGE_KIND(NAME) NodeAnnotation::NAME,
3021+
#include "swift/IDE/DigesterEnums.def"
30373022
});
30383023
}
30393024

0 commit comments

Comments
 (0)