Code review comment for lp:~swift/swift/swift_xml_subdir_changes

Might want to have Mike look at this too, since I think the attribute instead of body name was because he wanted to put items within the subdir element later. Kind of like how the container element has a name attribute and not a subelement of name.