String move() and String(String &&rval) breaks operation of reserve()

When #if __cplusplus >= 201103L || defined(__GXX_EXPERIMENTAL_CXX0X__)
operator = uses move() to just update the buffer pointer of the destination
This ignores any reserve() the user has made to ensure the memory is not unnecessarily fragmented.
String(String &&rval) has a similar problem

move() should first check the capacity of the destination and if there is sufficient space copy the source to the destination
String(String &&rval) should use move()

A suggested move() is

```
void String::move(String &rhs) {
if (this != &rhs) {
if (capacity > rhs.size) {
copy(rhs.buffer,rhs.size);
} else {
free(buffer);
buffer = rhs.buffer;
len = rhs.len;
capacity = rhs.capacity;
}
rhs.buffer = NULL;
rhs.len = 0;
rhs.capacity = 0;
}
}
```

C++11 の String 実装を見つけ、move()、operator=、コンストラクター String(String &&rval)、および reserve() の動作を調べてください。move 中に reserve 済みの destination がどのように動作するかを検証し、そのうえでコンストラクターが意図された move path に従うこと、および moved-from string が有効なままであることを確認してください。

索引モデルが issue の本文から書いたものです。