-
Notifications
You must be signed in to change notification settings - Fork 55
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
Full std::type_index interface support #3
Comments
I'm agree with this. About |
I have been checking docs from both That said, |
Couldn't we remove the prefix to have the same name? Otherwise I agree. |
Of course we can, but it may be expensive: if(name.substr<0,8>() == "struct ")
return name.substr<8,name.size()>();
else
return name; |
Then pack an Maybe you can avoid the branch then completely, is the |
No, for classes it prepends |
Damn, then at least make the check only on MSVC. A hash that doesn't change with the platform would be great. |
ctti::type_index
should have the full interface asstd::type_index
. Currently only missing the comparison operators.Also,
name()
should really return aconst char*
but this has the null-termination issue discussed in my PR.The text was updated successfully, but these errors were encountered: