mewin/cxxheaderparser
1
0
Fork 0
Code Issues Pull Requests Projects Releases Wiki Activity

Merge pull request #89 from robotpy/fn-constraints

Browse Source 
Move non-template requires to the function
This commit is contained in:
Dustin Spicuzza 2023-12-03 01:04:11 -05:00 committed by GitHub
commit 64c5290318
No known key found for this signature in database
GPG Key ID: 4AEE18F83AFDEB23
3 changed files with 108 additions and 44 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

15
cxxheaderparser/parser.py
View File

@ -1912,12 +1912,10 @@ class CxxParser:
else:
rtok = self.lex.token_if("requires")
if rtok:
fn_template = fn.template
if fn_template is None:
# requires on a function must always be accompanied by a template
if fn.template is None:
raise self._parse_error(rtok)
elif isinstance(fn_template, list):
fn_template = fn_template[0]
fn_template.raw_requires_post = self._parse_requires(rtok)
fn.raw_requires = self._parse_requires(rtok)
if self.lex.token_if("ARROW"):
self._parse_trailing_return_type(fn)
@ -1983,12 +1981,7 @@ class CxxParser:
toks = self._consume_balanced_tokens(otok)[1:-1]
method.noexcept = self._create_value(toks)
elif tok_value == "requires":
method_template = method.template
if method_template is None:
raise self._parse_error(tok)
elif isinstance(method_template, list):
method_template = method_template[0]
method_template.raw_requires_post = self._parse_requires(tok)
method.raw_requires = self._parse_requires(tok)
else:
self.lex.return_token(tok)
break

10
cxxheaderparser/types.py
View File

@ -526,9 +526,6 @@ class TemplateDecl:
#: template <typename T> requires ...
raw_requires_pre: typing.Optional[Value] = None
#: template <typename T> int main() requires ...
raw_requires_post: typing.Optional[Value] = None
#: If no template, this is None. This is a TemplateDecl if this there is a single
#: declaration:
@ -730,6 +727,13 @@ class Function:
#: is the string "conversion" and the full Type is found in return_type
operator: typing.Optional[str] = None
#: A requires constraint following the function declaration. If you need the
#: prior, look at TemplateDecl.raw_requires_pre. At the moment this is just
#: a raw value, if we interpret it in the future this will change.
#:
#: template <typename T> int main() requires ...
raw_requires: typing.Optional[Value] = None
@dataclass
class Method(Function):

83
tests/test_concepts.py
View File

@ -6,6 +6,7 @@ from cxxheaderparser.types import (
Concept,
Function,
FundamentalSpecifier,
Method,
MoveReference,
NameSpecifier,
PQName,
@ -495,8 +496,9 @@ def test_requires_last_elem() -> None:
)
],
template=TemplateDecl(
params=[TemplateTypeParam(typekey="typename", name="T")],
raw_requires_post=Value(
params=[TemplateTypeParam(typekey="typename", name="T")]
),
raw_requires=Value(
tokens=[
Token(value="Eq"),
Token(value="<"),
@ -504,7 +506,6 @@ def test_requires_last_elem() -> None:
Token(value=">"),
]
),
),
)
]
)
@ -752,7 +753,8 @@ def test_requires_both() -> None:
Token(value=">"),
]
),
raw_requires_post=Value(
),
raw_requires=Value(
tokens=[
Token(value="Subtractable"),
Token(value="<"),
@ -760,7 +762,6 @@ def test_requires_both() -> None:
Token(value=">"),
]
),
),
)
]
)
@ -791,8 +792,9 @@ def test_requires_paren() -> None:
)
],
template=TemplateDecl(
params=[TemplateTypeParam(typekey="class", name="T")],
raw_requires_post=Value(
params=[TemplateTypeParam(typekey="class", name="T")]
),
raw_requires=Value(
tokens=[
Token(value="("),
Token(value="is_purrable"),
@ -804,7 +806,72 @@ def test_requires_paren() -> None:
Token(value=")"),
]
),
),
)
]
)
)
def test_non_template_requires() -> None:
content = """
// clang-format off
template <class T>
struct Payload
{
constexpr Payload(T v)
requires(std::is_pod_v<T>)
: Value(v)
{
}
};
"""
data = parse_string(content, cleandoc=True)
assert data == ParsedData(
namespace=NamespaceScope(
classes=[
ClassScope(
class_decl=ClassDecl(
typename=PQName(
segments=[NameSpecifier(name="Payload")], classkey="struct"
),
template=TemplateDecl(
params=[TemplateTypeParam(typekey="class", name="T")]
),
),
methods=[
Method(
return_type=None,
name=PQName(segments=[NameSpecifier(name="Payload")]),
parameters=[
Parameter(
type=Type(
typename=PQName(
segments=[NameSpecifier(name="T")]
)
),
name="v",
)
],
constexpr=True,
has_body=True,
raw_requires=Value(
tokens=[
Token(value="("),
Token(value="std"),
Token(value="::"),
Token(value="is_pod_v"),
Token(value="<"),
Token(value="T"),
Token(value=">"),
Token(value=")"),
]
),
access="public",
constructor=True,
)
],
)
]
)