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

Maintenance status of poly #1172

Open
cstratopoulos opened this issue Jun 19, 2019 · 3 comments
Open

Maintenance status of poly #1172

cstratopoulos opened this issue Jun 19, 2019 · 3 comments

Comments

@cstratopoulos
Copy link
Contributor

Hello

Was wondering about whether folly/poly is being maintained and if PRs would be accepted. From what I've seen its tests are commented out in CI and its source files are unconditionally deleted on Windows builds.

I've been making some minor updates in a fork and I wanted to know if it would be possible to get them merged. In summary:

Note that with the last few bullet points some testing on my local machine indicates that compiling/test passing with recent MSVC (16, definitely, 15.9.x maybe) should be within reach.

The only issue I've encountered is with non-member functions as it seems these require a lambda in an unevaluated context which I think lands in C++20 with p0315.

@Orvid
Copy link
Contributor

Orvid commented Jun 19, 2019

Pull requests are welcome :)

The only reason it's commented out is because when Poly was first created I didn't have the time to go through and fix it to work on Windows, so I just disabled the test for the moment, and then proceeded to forget I'd done so :)

@cstratopoulos
Copy link
Contributor Author

@Orvid cool! Would you prefer single-topic PRs for the bullets I outlined above (e.g, one for noexcept, one for the NTTP macro) or should I just open all in one go?

@Orvid
Copy link
Contributor

Orvid commented Jun 19, 2019

Prefer separate PRs as it makes them easier to review and get landed.

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