Better to use enum class instead of enum

eg. In s2shape_index.h:
```
enum InitialPosition { BEGIN, END, UNPOSITIONED };
```
Such an enum name like `BEGIN` is very common, so it is easy to have the same name with a macro or other enum, which will cause conflict.