[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
fixed: split non-template class in header/implementation
fixed: remove selfinclude from header
[UnrarXLib] drop use of URIUtils::GetExtension in favour of simple strrchr, ref dcd92b3769a0e4db0fceb568f9e3ccd71fbc4b74
[dllloader] have LibraryLoader hold std::string's
[fileitem] make the string constructor of CFileItem(List) explicit
[stdstring] drop assignment/append/construction from non-char pointers, and remove usage of assign from char
[AE/osxsink] - distinguish between passthroughnative and passthroughbitstream in the findsuitableformat method
Merge pull request #4971 from jmarshallnz/xml_attribute

Xml attribute tidy up