Not necessary because we control the LightSong's life cycle.

Zero means "unknown".

Reduce bloat.  We never used the set lookup.

Make this non-fatal.  Just stop reading more objects.

Passing this to csvToStrings() will do the same.

Fixes crash when there's no SearchCaps element.

Fixes problems with some UPnP servers.

Avoid appending to an obsolete string object.

Not a fatal error.  Fixes assertion failure.

Reduce overhead.

Reduce overhead.

Reduce bloat.

Expat can call CharacterData() multiple times if the CDATA contains
entity references.  We need to collect all of them in one large
string.