StringData should have a const_iterator type and begin() and end() methods.

This would make it compatible with various standard algorithms and programming practices already common in the codebase.

Proposed implementation:

class StringData {
public:
typedef const char* const_iterator;
...
const_iterator begin()

const_iterator end()

...
};