Just my two nitpicky cents, but I actually prefer spearate opening and closing tags rather than self-closing for this kind of multiline attribute listing. I know this is just setting attributes and does not require actual bracketing of any data, but the separate tags conform better to the file's usage and are easier to read, IMHO.