This line bothered me more than it should :)

    typedef std::string String;

when

    using std::string;

would have done just as well.

Yes, I noticed the capitalization change. But why do you want to change the name of a fundamental type?