Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

feat: accept type arguments in function calls #923

Open
wants to merge 4 commits into
base: main
Choose a base branch
from

Conversation

JohelEGP
Copy link
Contributor

@JohelEGP JohelEGP commented Jan 8, 2024

Resolves #628 (comment).

source/parse.h Outdated Show resolved Hide resolved
@hsutter
Copy link
Owner

hsutter commented Jan 9, 2024

From #628:

But what about offsetof (https://en.cppreference.com/w/cpp/types/offsetof)?
And more generally the compiler built-ins?
Unfortunately, generally supporting function type arguments seems like it would require typename.

Can you elaborate with examples where this is used? I'm not seeing the use cases in the PR code shown so far, unless the answer to my first question about sizeof(type ...) is that the type alters the meaning somehow.

source/parse.h Outdated Show resolved Hide resolved
regression-tests/pure2-print.cpp2 Outdated Show resolved Hide resolved
source/parse.h Show resolved Hide resolved
@JohelEGP

This comment was marked as resolved.

@hsutter
Copy link
Owner

hsutter commented Sep 25, 2024

Thanks! Sorry for the lag -- in the meantime I've added some support for function types so this PR would need to be rebased to align with and extend that.

Which would be your preference: Do you want to rebase this, or close this for now?

@JohelEGP JohelEGP force-pushed the function_type_argument branch from 1a260f7 to f6c399d Compare October 3, 2024 21:59
@JohelEGP JohelEGP changed the title feat: accept function type argument feat: accept type arguments in function calls Oct 3, 2024
@JohelEGP
Copy link
Contributor Author

JohelEGP commented Oct 3, 2024

Rebased.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants