Fix comments after module reference.

	ok # foo!
Stop on comma and EOL, too
Tests for commas, too
Output variables come first
Add comment for fr_canonicalize_error
Convert tmpl_afrom_attr_substr to use a similar signature as other parsing functions
Move fr_canonicalize_error to log.c

I still don't like the name, partly because now I have to remember how to spell canonicalize, and it seems like it should have double n's to me
Convert the majority of tmpl functions to use a similar signature as the parse functions

This means condition parsing error markers now extend into attribute references
I hate doxygen
Indent condition error markers
Fix invalid request/list errors
Formatting
They're indexes not references...
We should use strtol to parse tag and array indexes as strtoul still parses negative numbers into very large ones (which is confusing for users)
Fix typo
Un-tagged attributes can *never* have tags
Reorder radius_exec_program arguments to be consistent with the rest of the server
Fix argument order so ctx is always first
Fix editor error
Rename map functions to be consistent with tmpl functions