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
Sprinkle VERIFY_MAP in the map functions