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

[MSVC] Template resolution #60

Open
klemens-morgenstern opened this issue Oct 12, 2020 · 2 comments
Open

[MSVC] Template resolution #60

klemens-morgenstern opened this issue Oct 12, 2020 · 2 comments

Comments

@klemens-morgenstern
Copy link
Contributor

klemens-morgenstern commented Oct 12, 2020

MSVC has an issue in the current version, with the process signatures, where the process_launcher is placed at the end. The is the following code is not working on msvc, but on gcc:

proc::process proc1{target_path, {"--exit-code", "40", "--wait", "100"}, proc::default_process_launcher{}};

MSVC gives a C2440 error.

I do not know if this is an issue with MSVC or with my design, we might need to move the position of the customer initializer. I am doing the latter for some experimentation on process_groups. Either way, this should be mentioned.

Edit: I had a few more issues and it seems that the MSVC concept support is just buggy as of now.

@JeffGarland
Copy link
Owner

I had a few more issues and it seems that the MSVC concept support is just buggy as of now.

That doesn't surprise me given that the ink isn't even dry on c++20. I don't think we want to change our design. Maybe there's a channel for reporting the bugs?

@klemens-morgenstern
Copy link
Contributor Author

Given that the official description of MSVC is "partial support" I am quite certain they're aware. Wasn't too sure with the design though.

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

No branches or pull requests

2 participants