Remove 'source lifetime about minijinja

[mitsuhiko]

One other thing is that the engine has a few places where that basic model does not work well:

• Token::Str holds a Cow so that it can store unescaped strings
• for internal loop usage Instruction::Lookup and Instruction::GetAttr also get static strings at times
• Internal maps often hold &'source str and with that change they would need to hold a String instead since we could not adjacently borrow

I think a model that could work is to have a StrCache<'source> which is used everywhere where currently &'source str is used to hold the source and then a few StrHandle given out by the StrCache<'source>. For as long as also for release builds an internal ID is given out that could be made safe without incurring bounds checks. For the few places where a &'static str is placed there, I think the handle could be allowed to store references to static strings or specific instructions are used instead. For the the Cow case that only exists in the Token but never in Instruction so that problem is much smaller in comparison. In the instructions it's already converted into a Value which clones.

[mitsuhiko]

I made some small changes internally that address some of these issues. In d53160d I removed static strings added to instructions for loop internals and in 6e60023 I removed the Cow from the string token.

[mitsuhiko]

I had back out one of these due to performance regressions.

[mitsuhiko]

I have a new way now in which I think I want to go about this. The 'source lifetime is quite useful for multiple scenarios so it does not need outright removing. What actually however is annoying is that you don't have a convenient way to add owned strings into the environment. This runs against a bunch of lifetime issues, but that is already true for the Source feature too.

I think what I might be doing for MiniJinja 1.0 and to close off this issue is the following:

• When the source feature is enabled, the Engine always holds a Source object internally to replace the template map
• Environment::add_template stays and borrows
• Environment::add_owned_template (name TDB) is does what today is env.source_mut().add_template(...)
• Hide the Source object internally.

Since after you can add owned templates there is not a lot of reason to actually expose the Source object. The main of the Source object which at that point is internal is to set a loader:

Before:

let mut env = Environment::new();
env.set_source(Source::with_loader(|name| {
    if name == "layout.html" {
        Ok(Some("...".into()))
    } else {
        Ok(None)
    }
}));

After:

let mut env = Environment::new();
env.set_loader(|name| {
    if name == "layout.html" {
        Ok(Some("...".into()))
    } else {
        Ok(None)
    }
});

And for owned templates:

Before:

let mut env = Environment::new();
let mut source = Source::new();
source.add_template("index.html", "...").unwrap();
env.set_source(source);

After:

let mut env = Environment::new();
env.add_owned_template("index.html", "...").unwrap();