Re: Review of mustache starts today.

From: Peter Dimov (pdimov_at_[hidden])
Date: 2023-02-09 00:01:48

Ruben Perez wrote:
> > Should be something else, then, because the delimiters are two
> > characters long so should fit.
> 
> From my debugging, seems to be copying json::value's into the context stack
> (renderer.cpp:1093, renderer.cpp:1103).
> Is all this copying preventable with the current interface?

>From memory, the context stack only needs to refer to already extant values
at the moment, so it's probably possible to change it to the equivalent of
std::pmr::vector<json::value const*>. (However, this might impede some
future extensions that I have in mind. Either way, it's an optimization that
doesn't depend on any interface changes.)