String::operator= should return a reference to "this"

Avoids copying and I am pretty sure a user would expect a and b of const String& a = b to have the same address.