Squirrel

Hi,

is there already a binding for std::string out there? I'm trying to bind a function that takes a std::string reference and I was wondering if there's already someone who wrote one and has it more or less tested. This would avoid having a bunch of binding-approaches out there and would focus bug-fixing, etc.

Rincewind

Ok, here's a first attempt to allow the usage of std::string in SQplus.

I'm not quite sure about how to handle unicode characters. Also, I'm not quite sure about the lifetime of the c_str() pointer in the Push-function.
My first tests show it works though.

Comments welcome.

Rincewind

inline bool    Match(TypeWrapper,HSQUIRRELVM v,int idx) {
        return sq_gettype(v,idx) == OT_STRING;
    }
    inline ::String Get(TypeWrapper,HSQUIRRELVM v,int idx)
    {
        const SQChar * s;
        SQPLUS_CHECK_GET(sq_getstring(v,idx,&s));
        return std::string(s);
    }

    inline void Push(HSQUIRRELVM v,const std::string& value) { sq_pushstring(v,value.c_str(),-1); }

To make a unicode/ascii stl string, typedef something like this:

typedef std::basic_string < SQChar > SQString;

Alberto

Hmmm, most people will probably just be using std::string or wstring. Will have to see what kind of copy-constructors are implemented. Thanks for the hint though. I will look into it.

Rincewind

Sorry to wake this old topic up, but I tried to make SqPlus work with std::string, and i encountered this problem : in sqplus.h, uncommenting "#define SQPLUS_SUPPORT_STD_STRING" isn't enough, i had to add "#include <string>" outside the SqPlus namespace.
That's almost nothing, but can be confusing (i'm a very beginner with SqPlus, and it at least confused me ;))

Hi.

I use the 20070304 snapshot, I just saw the newly one but seeing the changelog, it won't be of any use.

I use SqPlus with SQPLUS_CONST_OPT and SQPLUS_SUPPORT_STD_STRING defined. It works fine under Windows with MinGW as the compiler. However, I just discovered that it doesn't work when building it with GCC under Linux. When I run "make" in the sqplus directory it fails to compile at the definition of Push, Match and Get functions for std::string.

It seems that I'm not the only one, seeing this thread. So, I did the same as Ceacy and it worked. I think it should be included in the next snapshot. Here is the code, added right after all the defines of sqplus.h:

#ifdef SQPLUS_SUPPORT_STD_STRING
} // close SqPlus namespace
#include <string>
namespace SqPlus {
#endif

Any remarks about this ?

Thanks,
Sebastien

Thank you for your fix.
It is merged in 20070602 snapshot.