Skip to content

Commit 84410fe

Browse files
j-piaseckimeta-codesync[bot]
authored andcommitted
Restore a single method for text parsing (#55909)
Summary: Pull Request resolved: #55909 Changelog: [Internal] Restores the single path for doxygen ref parsing that was lost in refactor in D94072940 Reviewed By: cipolleschi Differential Revision: D95206764 fbshipit-source-id: c2028872d8ff904187b41baf113dfe828796edcc
1 parent c45c5cf commit 84410fe

3 files changed

Lines changed: 4 additions & 34 deletions

File tree

scripts/cxx-api/parser/builders.py

Lines changed: 4 additions & 10 deletions
Original file line numberDiff line numberDiff line change
@@ -26,12 +26,7 @@
2626
TypedefMember,
2727
VariableMember,
2828
)
29-
from .scope import (
30-
CategoryScopeKind,
31-
InterfaceScopeKind,
32-
ProtocolScopeKind,
33-
StructLikeScopeKind,
34-
)
29+
from .scope import InterfaceScopeKind, ProtocolScopeKind, StructLikeScopeKind
3530
from .snapshot import Snapshot
3631
from .template import Template
3732
from .utils import (
@@ -40,7 +35,6 @@
4035
InitializerType,
4136
parse_qualified_path,
4237
resolve_linked_text_name,
43-
resolve_ref_text_name,
4438
)
4539

4640

@@ -281,7 +275,7 @@ def get_typedef_member(
281275
typedef_def: compound.memberdefType, visibility: str
282276
) -> TypedefMember:
283277
typedef_name = typedef_def.get_name()
284-
typedef_type = resolve_ref_text_name(typedef_def.get_type())
278+
typedef_type = resolve_linked_text_name(typedef_def.get_type())[0]
285279
typedef_argstring = typedef_def.get_argsstring()
286280
typedef_definition = typedef_def.definition
287281

@@ -338,7 +332,7 @@ def get_property_member(
338332
Get the property member from a member definition.
339333
"""
340334
property_name = member_def.get_name()
341-
property_type = resolve_ref_text_name(member_def.get_type()).strip()
335+
property_type = resolve_linked_text_name(member_def.get_type())[0].strip()
342336
accessor = member_def.accessor if hasattr(member_def, "accessor") else None
343337
is_readable = getattr(member_def, "readable", "no") == "yes"
344338
is_writable = getattr(member_def, "writable", "no") == "yes"
@@ -364,7 +358,7 @@ def create_enum_scope(snapshot: Snapshot, enum_def: compound.EnumdefType) -> Non
364358
Create an enum scope in the snapshot.
365359
"""
366360
scope = snapshot.create_enum(enum_def.qualifiedname)
367-
scope.kind.type = resolve_ref_text_name(enum_def.get_type())
361+
scope.kind.type = resolve_linked_text_name(enum_def.get_type())[0]
368362
scope.location = enum_def.location.file
369363

370364
for enum_value_def in enum_def.enumvalue:

scripts/cxx-api/parser/utils/__init__.py

Lines changed: 0 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -19,7 +19,6 @@
1919
InitializerType,
2020
normalize_angle_brackets,
2121
resolve_linked_text_name,
22-
resolve_ref_text_name,
2322
)
2423
from .type_qualification import qualify_arguments, qualify_parsed_type, qualify_type_str
2524

@@ -40,5 +39,4 @@
4039
"qualify_parsed_type",
4140
"qualify_type_str",
4241
"resolve_linked_text_name",
43-
"resolve_ref_text_name",
4442
]

scripts/cxx-api/parser/utils/text_resolution.py

Lines changed: 0 additions & 22 deletions
Original file line numberDiff line numberDiff line change
@@ -90,28 +90,6 @@ def normalize_angle_brackets(text: str) -> str:
9090
return text
9191

9292

93-
def resolve_ref_text_name(type_def: compound.refTextType) -> str:
94-
"""Resolve the text content of a refTextType."""
95-
if hasattr(type_def, "content_") and type_def.content_:
96-
name = ""
97-
for part in type_def.content_:
98-
if part.category == 1: # MixedContainer.CategoryText
99-
name += part.value
100-
elif part.category == 3: # MixedContainer.CategoryComplex (ref element)
101-
if hasattr(part.value, "get_valueOf_"):
102-
name += part.value.get_valueOf_()
103-
elif hasattr(part.value, "valueOf_"):
104-
name += part.value.valueOf_
105-
else:
106-
name += str(part.value)
107-
return normalize_angle_brackets(name)
108-
109-
if type_def.ref:
110-
return normalize_angle_brackets(type_def.ref[0].get_valueOf_())
111-
112-
return normalize_angle_brackets(type_def.get_valueOf_())
113-
114-
11593
class InitializerType(Enum):
11694
NONE = (0,)
11795
ASSIGNMENT = (1,)

0 commit comments

Comments
 (0)