ROOT6 uses std::string, rather than std::basic_string<char>

[wmtan]

ROOT 6 uses std::string for the type name of a string, whereas ROOT5 used std::basic_string. This pull request changes the code in the ROOT6 branch to use the ROOT6 names, and thereby fixes a failing unit test.

[cmsbuild]

A new Pull Request was created by @wmtan for CMSSW_7_0_ROOT6_X.

ROOT6 uses std::string, rather than std::basic_string

It involves the following packages:

FWCore/RootAutoLibraryLoader
FWCore/Utilities

@cmsbuild, @Dr15Jones, @ktf, @nclopezo can you please review it and eventually sign? Thanks.
You can sign-off by replying to this message having '+1' in the first line of your reply.
You can reject by replying to this message having '-1' in the first line of your reply.

[davidlt]

Could it be related to the problem (ROOT-5727) that they don't expand typedefs yet? Because std::string is a typedef of std::basic_string IIRC.

[wmtan]

No, it's not related. You are correct that std::string is a typedef. However, the ROOT team has decided that the name used should be std::string. Also, in the unit test that failed, the dictionary used for the class is specified as std::basic_string, so no typedef was used.

[Dr15Jones]

+1