Question

I’m coming back to C++ after long years spent on other technologies and i’m stuck on some weird behavior when calling some methods taking std::string as parameters :

An example of call :

LocalNodeConfiguration *LocalNodeConfiguration::ReadFromFile(std::string & path)
{
// ...
throw configuration_file_error(string("Configuration file empty"), path);
// ...
}


When I compile I get this (I cropped file names for readability) :

/usr/bin/g++    -g -I/home/shtong/Dev/OmegaNoc/build -I/usr/share/include/boost-1.41.0   -o CMakeFiles/OmegaNocInternals.dir/configuration/localNodeConfiguration.cxx.o -c /home/shtong/Dev/OmegaNoc/source/configuration/localNodeConfiguration.cxx
    .../localNodeConfiguration.cxx: In static member function ‘static OmegaNoc::LocalNodeConfiguration* OmegaNoc::LocalNodeConfiguration::ReadFromFile(std::string&)’:
    .../localNodeConfiguration.cxx:72: error: no matching function for call to ‘OmegaNoc::configuration_file_error::configuration_file_error(std::string, std::basic_string<char, std::char_traits<char>, std::allocator<char> >&)’
    .../configurationManager.hxx:25: note: candidates are: OmegaNoc::configuration_file_error::configuration_file_error(std::string&, std::string&)
    .../configurationManager.hxx:22: note:                 OmegaNoc::configuration_file_error::configuration_file_error(const OmegaNoc::configuration_file_error&)

So as I understand it, the compiler is considering that my path parameter turned into a basic_string at some point, thus not finding the constructor overload I want to use. But I don’t really get why this transformation happened.

Some search on the net suggested me to use g++ but I was already using it. So any other advice would be appreciated 🙂

Thanks

Answer 1

The problem is not basic_string, because basic_string<char, restOfMess> is equivalent to string.

The problem is the function only provides

f(string&, string&) {
//------^

but you are calling as

 f(string("blah"), path);
// ^^^^^^^^^^^^^^

This is a rvalue (temporary object), and a rvalue cannot be bound to a mutable reference. You either need to change the prototype to accept const references or just pass-by-value:

    f(const string&, string&) {
//----^^^^^^
or
    f(string, string&) {

Or provide a mutable reference (if you really need to modify the 1st argument in that function):

string s = "blah blah blah";
f(s, path);