[xmlutils] adds helper GetAttribute for retrieving a std::string version of an attribute
[xmlutils] use XMLUtils::GetAttribute() when assigning attributes directly to CStdString/std::string, as they may be NULL
[xml] fix potential null pointer dereference when reading attributes
[addons] make the 'type' attribute of addon settings consistently compared case-sensitive
[addons] read the id attribute into a std::string where comparison is made case-sensitive
[xml] use GetAttribute() where it makes the code clearer (and where used enough to warrant the include)
[xml] use TiXmlNode::Attribute(foo, int|double) method where it tidies things up
[xml] read attributes into const char* where they're re-used immediately
[skinsettings] setting type of skin settings needn't be compared case-insensitive (we write them, we read them)
[addons] have the 'subsetting' attribute and '' values attribute be case-sensitive to drop some strcmpi()
[settings] make the advancedsetting 'bydate' attribute for tvshow regexps case-sensitive (i.e. 'true')
[addons] drop duplicate if clause
[cosmetic] align assignment block
Merge pull request #4971 from jmarshallnz/xml_attribute

Xml attribute tidy up
Merge pull request #4968 from jmarshallnz/stdstring_removal_part0

Stdstring removal part 0
[stdstring] get rid of CStdString from SaveFileStateJob
[stdstring] get rid of CStdString from a bunch of URIUtils methods that return a string reference.
[stdstring] get rid of CStdString from utils/md5.cpp|h
[stdstring] get rid of CStdString in xbmc/XBDateTime

NOTE: RFC1123 means Month matching can be case-sensitive
[stdstring] get rid of CStdString in CDVDDemux::GetStreamCodecName