quicknir/wise_enum

Allow usage of string_view for C++17

Closed this issue · 2 comments

It's more efficient (verified by looking at generated assembly) to do this directly in the library than to force users to wrap it.

Note that the default, even in C++17 will be const char*. Since const char* and string_view have quite different interfaces, if the 17 default were string_view, then this could break code in surprising ways for clients supporting both 14 and 17, or for people upgrading from 14 to 17. Instead, string_view will be opt-in only.

This was added some time ago, but I did not get around to closing the issue yet.