String relational operators are members.

Member relational operators are no good because they cause the lhs and rhs to be treated differently. This causes problems such as making == appear non-commutative. Such as:

```c++
void setup() {
String("hi") == "world";
"world" == String("hi");
}
```

```
sketch_jan19a:3: error: no match for 'operator==' (operand types are 'const char [6]' and 'String')
"world" == String("hi");
^
exit status 1
```

It would be less user-surprising to bring these operators out of the class.

I propose defining them at namespace scope as symmetric pairs of operators, between a String on one side, and everything String accepts as an implicit conversion on the other side, per:
http://en.cppreference.com/w/cpp/string/basic_string/operator_cmp

Issue にある 2 つの C++ 式を最初の再現として使用し、次に ArduinoCore-API 内の String の関係演算子宣言を見つけます。両方のオペランド順を比較し、既存の String の動作を確認します。完了の条件は、同等の比較がコンパイル可能で対称的に動作し、報告されたケースを対象とするカバレッジがあることです。

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