Move tmpl_attribute fields out into its own type
Formatting
Expose FR_FAULT_LOG
Consistency checks should exit with an error code
Add functions to initialise tmpls, makes it easier to switch stack allocated to talloced in debug builds

Rewrite tmpl_verify function to work with stack allocated tmpls
Use tmpl_alloc to initialise templates in xlat.c
Don't partially populate the tmpl and error out in tmpl_from_attr_substr
Add tmpl_afrom_attrstr
Change tmpl_afrom_str so that it doesn't re-use allocated tmpl structures. This was triggering the internal sanity checks in VERIFY_TMPL
Rename some of the fields in the literal member of the union in the value_pair_tmpl_t struct, and convert more things to use the tmpl_init and tmpl_alloc functions
Add macros for VERIFY_TMPL and VERIFY_MAP
Formatting
Sprinkle VERIFY_TMPL in tmpl functions (to check they produce valid output)
Fix comment
Quiet doxygen
Typo
Fix const errors
Don't bitch if we're using references and the format config item has been removed
We no longer return -2 on partial parse error
Better parsing for attributes defined in pass1

They are now ATTR_UNKNOWN, instead of literals