Unnecessary field name string copy? about reflect-cpp HOT 3 CLOSED

I'm pretty sure it is. It's not only static, it is constexpr static.

It is necessary to support this syntax:

rfl::Field<std::string, "firstName">

So at compile time, the string "firstName" is copied into the class itself. I do not think that "firstName" would be persisted so I am pretty sure you couldn't just create a view.

Also, it is very important that this is all happening at compile time. So there really is no runtime overhead either way.

from reflect-cpp.

Looks like https://godbolt.org/ tricked me - string_view used to work once, but not any more.

Maybe in the future C++ versions it will work.

After translation phase 6, a string-literal that does not begin with an encoding-prefix is an ordinary string
literal. An ordinary string literal has type “array of n const char” where n is the size of the string as defined
below, has static storage duration (6.7.5), and is initialized with the given characters

from reflect-cpp.

Yes, I gave it a try myself and couldn't make it work either.

from reflect-cpp.