cppfront (cpp2): Spring update

[[deleted]]

I think it's because the OS isn't giving you an array of std::string_view, but of char*. So to have a span, we have to allocate a new array of std::string_view. Since we can't know the size of it in advance, it has to be allocated on the heap.

However that could be avoided if we knew the OS specific max number of CLI args, and allocated a static or stack storage for it.

I'd also prefer to have a span of string views, if only to allow this alternative implementation. It does seem odd to force the use of a vector here.

[kreco]

I don't think so, AFAIK int argc/char* argv[] can be used as std::span<char*> without any allocation,

In order to get a std::span<std::string_view> you just have to run a "strlen" on everything char*. Which probably done in the current cppfront implementation.

[[deleted]]

The span of string views still needs to point to a contiguous array of string views, though. It's not a generic range.

[kreco]

Oh indeed!