Squirrel

I'm using SqPlus in SQUIRREL2_1_0_sqplus_19.

The DECLARE_INSTANCE_TYPE_NAME_ macro in sqplus.h and SqPlusConst.h
creates the helper functions "bool Match()" for my C++ class, but these functions don't
seem to return false.

It is because that the GetInstance() function used in the generated Match() functions
doesn't return false but throws exception when the given argument type is invalid.

It would be helpful for me to make the function not to throw exception
as follows:

1. Add MatchInstance() function in sqplus.h

// Match an instance of type T from the stack at idx (for function calls).
template<typename T>
bool MatchInstance(HSQUIRRELVM v,SQInteger idx) {
  SQUserPointer up=0;
  sq_getinstanceup(v,idx,&up,ClassType<T>::type());
  return (up != NULL);
} // MatchInstance

2. Use MatchInstance(v, idx) instead of "GetInstance<TYPE>(v,idx) != NULL"
   in DECLARE_INSTANCE_TYPE_NAME macro in sqplus.h and SqPlusConst.h.  

It works well for me.  How about this change?

Added to release 20, with appropriate changes to Match() and Get():


template<typename T,bool ExceptionOnError>
T * GetInstance(HSQUIRRELVM v,SQInteger idx) {
  SQUserPointer up=0;
  sq_getinstanceup(v,idx,&up,ClassType<T>::type());
  if (ExceptionOnError) { // This code block should be compiled out when ExceptionOnError is false. In any case, the compiler should not generate a test condition (include or exclude the enclosed code block).
    if (!up) throw SquirrelError(_T("GetInstance: Invalid argument type"));
  } // if
  return (T *)up;
} // GetInstance


http://www.brightland.com/sq/SQUIRREL2_1_0_sqplus_20.zip

Thank you!
:-)