On Thu, 23 Jan 2014 21:46:43 GMT, Uplink_Coder wrote:

I played abit with the vibe.d codebase,
and I have to say it is really nice!

Greetings Uplink_Coder

Sorry for the late answer, I'm still busy with some important bug hunting.

In your pull request, you have modified the language to accept stringinclude - I'd keep the change to a pure API change and instead automatically append ".dt" to the string_include_name, so that include can be used. Other than that, I was wondering if it would be possible to support multiple string based "files" in compileStringIncludeDietFile. Maybe using an alias template parameter that points to either a, let's say, CustomDietFile struct, or to an array of them, where this struct contains the name and contents of the virtual file.

I'd also like to put some thought into the naming scheme of the public functions (BTW, I'm thinking about renaming render!() to renderDiet!(), too). But other than that I think this is a valuable feature to add, and I seem to remember that this has been asked for before.

Regards,
Sönke