This changes the API have render take the cursor instead of having it allocate one and drop render_text and render_file in favor of the Render trait implemented on a number of types.

Please consider the following use case:

let data = HashBuilder::new();
for foo in &vec {
    let a = rustache::render_file(foo, data);
}

The above code will trigger use of moved value: 'data'.

Please could be possible to change the signature of render_file? This will obviously break other people code, but it's an easy build fix which opens to more use cases.

Thanks

By adding From implementations for all of the different data types allowed for the HashBuilder and VecBuilder APIs, we can vastly simplify those APIs down to basically a simple insert or push instead of an insert_string, insert_int, insert_float, etc.

tried

 let data = rustache::HashBuilder::new()
    .insert("name", "Bob");
let mut out = Cursor::new(Vec::new());

data.render("{{ name }}", &mut out); 

and other examples return

error[E0433]: failed to resolve. Use of undeclared type or module `Cursor`
   --> main.rs:323:15
    |
323 | let mut out = Cursor::new(Vec::new());
    |               ^^^^^^ Use of undeclared type or module `Cursor`

This API feels a bit confusing to me and it doesn't really have tests and likely has low use so remove it before the API change release that's coming up.

When parsing the template, any text in a line after a tag is dropped if there is not any whitespace before it. This allows newlines in the preceding text that is captured so that it will not be thrown away. This is an implementation of #132 that also includes a test for it.

Currently HashBuilder::insert_{vector, hash} and VecBuilder::push_{vector, hash} accept Fn for their function argument. This disallows moving out of the builder closure environment event though it is called only once in rustache code. Therefore, it is impossible, for example, to consume a vector of Strings and use its contents directly - you always need to clone all of the strings. With FnOnce these extra allocations could be avoided.

Just a typo I spotted:

the last line in the first example reads:

println!("{}", String::from_utf8(rv.into_inner()).unwrap());

It should be:

println!("{}", String::from_utf8(out.into_inner()).unwrap());

Use the error-chain machinery for generating the Error boilerplate in a standard fashion. This branch ceases the use of RustacheResult and RustacheError in favor of redefining the standard names. Additionally, it captures the actual errors behind the stream and file errors instead of just capturing the display strings. Finally, it flattens the error structure to a single level instead of having the template errors as their own type.

By allowing FnOnce instead of just Fn, the closure can move captured variables out of the closure, allowing the passed-in vector or hash to be consumed directly without having to be cloned first. Fixes #130.

Can you tag a new version and push it to crates.io? The current version doesn't build and the most recent commit in master appears to resolve the issue.

Is it possible to change the default delimiters from {{ }} to a different set of characters? For example, to template a file which uses curly braces already in it's content?

error[E0425]: cannot find function render_text in module rustache --> main.rs:323:21 | 323 | rustache::render_text(template, city); | ^^^^^^^^^^^ not found in rustache

Passing HashBuilders around is currently very difficult as HashBuilder and Data do not implement Clone. If this could be implemented, HashBuilders could be defined once and then passed to a function multiple times.

If this is already possible in a different way, please let me know.