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

Should Http.Proto support -fno-rtti? #89

Open
cmazakas opened this issue Jun 4, 2024 · 0 comments
Open

Should Http.Proto support -fno-rtti? #89

cmazakas opened this issue Jun 4, 2024 · 0 comments

Comments

@cmazakas
Copy link
Collaborator

cmazakas commented Jun 4, 2024

The service registry has a dependency on type_index which requires RTTI, so the code currently polyfills this via: https://github.com/cppalliance/http_proto/blob/develop/include/boost/http_proto/detail/type_index.hpp

This incurs extra maintenance burden not to mention the introduction of a downcast helper, which was created to facilitate the use-case where a derived service object uses virtual inheritance.

It'd be good to compare our implementation against the one in Asio, which seems to only return void* in the case of -fno-rtti.

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

1 participant