Drop CStdStringArray, SplitString, JoinString

[jmarshallnz]

Mostly cleanup, though there are a few fixes here as well.

The ones I'm really not sure about are designated with the REVIEW subscript.

@davilla, @koying: Please see 0a6ae3a

@Montellese: please check d813aba, 67b10ff, and dee673f

@Karlson2k Perhaps you could review in detail - most of the changes are straightforward, but the odd one or two are a little more. I anticipate this will go in at the end of the June merge window.

[Montellese]

Gone through all the changes. Very nice work. Looks like you've found some nice code simplifications as well.

[jmarshallnz]

Thanks - have fixed up your comments - most appreciated.

[Karlson2k]

Nice and big job!
I'd use ctor initializer instead of default ctor + assignment.
And, mostly cosmetics, add std:: to string and vector.

[jmarshallnz]

Thanks - I'll fix some of the cosmetics, but won't bother with most of them when the same files are using vector already over and over. IMO using namespace std in a .cpp file is sometimes the lesser evil to all the replication that is otherwise in place. Treated carefully, ofc - if there's not much use of it, then no problem with the prefix.

[jmarshallnz]

jenkins build this please