Thread: std::string declaration in header file error

> void execute_assert(bool expr, std::string str[]);
So why would you want to pass an array of strings ?

It seems to me like some hack job converted
void execute_assert(bool expr, char str[]);
to
void execute_assert(bool expr, std::string str[]);
without thinking about the consequences.

Removing the square brackets should fix the first problem.

> v_assert(s < 1, "Illegal acl size (" + deb + ")");
What is deb?
If it isn't a char array, or a char pointer, you might be SoL.

What you might need to do, to force the right thing to happen with the + (ie, string concatenation), is something like
v_assert(s < 1, string("Illegal acl size (") + deb + ")");

Removed the square brackets.

and the more complete code including deb is:

Code:

#ifdef DEBUG
    std::string deb;
#endif
    // Test parameter s
    v_assert(s >= MAX_ACL_DEPTH, "MAX_ACL_DEPTH too small");
#ifdef DEBUG
    deb=s;
    v_assert(s < 1, "Illegal acl size (" + deb + ")");

Ok, now the errors are.

Code:

Error: cannot convert ‘const char*’ to ‘std::string* {aka std::basic_string<char>*}’ for argument ‘2’ to ‘void execute_assert(bool, std::string*)’

AND

Code:

Error: cannot convert ‘std::basic_string<char>’ to ‘std::string* {aka std::basic_string<char>*}’ for argument ‘2’ to ‘void execute_assert(bool, std::string*)’

consecutively.

Code:

v_assert(s < 1, std::string("Illegal acl size (") + deb + ")");

Does not work either.

Any ideas?

> Removed the square brackets.
Then why does the error message say that you haven't?

Error: cannot convert ‘const char*’ to ‘std::string*
Error: cannot convert ‘std::basic_string<char>’ to ‘std::string*

That * at the end of std::string means the compiler still thinks there is a square bracket pair at the end of your parameter declaration.

Just in case, are you sure you ended up with:

Code:

execute_assert(s = MAX_ACL_DEPTH, "MAX_ACL_DEPTH too small");

? It looks like you have an assignment when you originally had a comparison. Perhaps you should have gone with something closer to your original:

Code:

execute_assert(s >= MAX_ACL_DEPTH, "MAX_ACL_DEPTH too small");