* jim.c, jim.h: more text formatting fixes, improved error messages

	with location of error

2 files changed, 71 insertions, 17 deletions

diff --git a/ecos/language/tcl/jim/current/include/jim.h b/ecos/language/tcl/jim/current/include/jim.h
index 056eb0c..cd53883 100644
--- a/ecos/language/tcl/jim/current/include/jim.h
+++ b/ecos/language/tcl/jim/current/include/jim.h
@@ -675,6 +675,9 @@ JIM_STATIC char * JIM_API(Jim_StrDup) (const char *s);
 

 /* evaluation */

 JIM_STATIC int JIM_API(Jim_Eval)(Jim_Interp *interp, const char *script);

+/* in C code, you can do this and get better error messages */

+/*   Jim_Eval_Named( interp, "some tcl commands", __FILE__, __LINE__ ); */

+JIM_STATIC int JIM_API(Jim_Eval_Named)(Jim_Interp *interp, const char *script,const char *filename, int lineno);

 JIM_STATIC int JIM_API(Jim_EvalGlobal)(Jim_Interp *interp, const char *script);

 JIM_STATIC int JIM_API(Jim_EvalFile)(Jim_Interp *interp, const char *filename);

 JIM_STATIC int JIM_API(Jim_EvalObj) (Jim_Interp *interp, Jim_Obj *scriptObjPtr);

@@ -733,6 +736,8 @@ JIM_STATIC Jim_Obj * JIM_API(Jim_NewStringObjNoAlloc) (Jim_Interp *interp,
         char *s, int len);

 JIM_STATIC void JIM_API(Jim_AppendString) (Jim_Interp *interp, Jim_Obj *objPtr,

         const char *str, int len);

+JIM_STATIC void JIM_API(Jim_AppendString_sprintf) (Jim_Interp *interp, Jim_Obj *objPtr,

+												   const char *fmt, ... );

 JIM_STATIC void JIM_API(Jim_AppendObj) (Jim_Interp *interp, Jim_Obj *objPtr,

         Jim_Obj *appendObjPtr);

 JIM_STATIC void JIM_API(Jim_AppendStrings) (Jim_Interp *interp,

@@ -1169,6 +1174,7 @@ static void Jim_InitExtension(Jim_Interp *interp)
   JIM_GET_API(Alloc);

   JIM_GET_API(Free);

   JIM_GET_API(Eval);

+  JIM_GET_API(Eval_Named);

   JIM_GET_API(EvalGlobal);

   JIM_GET_API(EvalFile);

   JIM_GET_API(EvalObj);

@@ -1195,6 +1201,7 @@ static void Jim_InitExtension(Jim_Interp *interp)
   JIM_GET_API(NewStringObj);

   JIM_GET_API(NewStringObjNoAlloc);

   JIM_GET_API(AppendString);

+  JIM_GET_API(AppendString_sprintf);

   JIM_GET_API(AppendObj);

   JIM_GET_API(AppendStrings);

   JIM_GET_API(StringEqObj);

diff --git a/ecos/language/tcl/jim/current/src/jim.c b/ecos/language/tcl/jim/current/src/jim.c
index cc8664e..e21679b 100644
--- a/ecos/language/tcl/jim/current/src/jim.c
+++ b/ecos/language/tcl/jim/current/src/jim.c
@@ -109,6 +109,33 @@ static Jim_HashTableType JimVariablesHashTableType;
  * Utility functions

  * ---------------------------------------------------------------------------*/

 

+static char *

+jim_vasprintf( const char *fmt, va_list ap )

+{

+#ifndef HAVE_VASPRINTF

+	/* yucky way */

+static char buf[2048];

+	vsnprintf( buf, sizeof(buf), fmt, ap );

+	/* garentee termination */

+	buf[sizeof(buf)-1] = 0;

+#else

+	char *buf;

+	vasprintf( &buf, fmt, ap );

+#endif

+	return buf;

+}

+

+static void

+jim_vasprintf_done( void *buf )

+{

+#ifndef HAVE_VASPRINTF

+	(void)(buf);

+#else

+	free(buf);

+#endif

+}

+	

+

 /*

  * Convert a string to a jim_wide INTEGER.

  * This function originates from BSD.

@@ -2017,6 +2044,22 @@ void Jim_AppendString(Jim_Interp *interp, Jim_Obj *objPtr, const char *str,
     StringAppendString(objPtr, str, len);

 }

 

+void Jim_AppendString_sprintf( Jim_Interp *interp, Jim_Obj *objPtr, const char *fmt, ... )

+{

+	char *buf;

+	va_list ap;

+

+	va_start( ap, fmt );

+	buf = jim_vasprintf( fmt, ap );

+	va_end(ap);

+

+	if( buf ){

+		Jim_AppendString( interp, objPtr, buf, -1 );

+		jim_vasprintf_done(buf);

+	}

+}

+

+

 void Jim_AppendObj(Jim_Interp *interp, Jim_Obj *objPtr,

         Jim_Obj *appendObjPtr)

 {

@@ -8724,17 +8767,31 @@ int JimCallProcedure(Jim_Interp *interp, Jim_Cmd *cmd, int argc,
     return retcode;

 }

 

-int Jim_Eval(Jim_Interp *interp, const char *script)

+int Jim_Eval_Named(Jim_Interp *interp, const char *script, const char *filename, int lineno)

 {

-    Jim_Obj *scriptObjPtr = Jim_NewStringObj(interp, script, -1);

     int retval;

+    Jim_Obj *scriptObjPtr;

 

+	scriptObjPtr = Jim_NewStringObj(interp, script, -1);

     Jim_IncrRefCount(scriptObjPtr);

+

+

+	if( filename ){

+		JimSetSourceInfo( interp, scriptObjPtr, filename, lineno );

+	}

+

     retval = Jim_EvalObj(interp, scriptObjPtr);

     Jim_DecrRefCount(interp, scriptObjPtr);

     return retval;

 }

 

+int Jim_Eval(Jim_Interp *interp, const char *script)

+{

+	return Jim_Eval_Named( interp, script, NULL, 0 );

+}

+

+

+

 /* Execute script in the scope of the global level */

 int Jim_EvalGlobal(Jim_Interp *interp, const char *script)

 {

@@ -9074,6 +9131,7 @@ void JimRegisterCoreApi(Jim_Interp *interp)
   JIM_REGISTER_API(Alloc);

   JIM_REGISTER_API(Free);

   JIM_REGISTER_API(Eval);

+  JIM_REGISTER_API(Eval_Named);

   JIM_REGISTER_API(EvalGlobal);

   JIM_REGISTER_API(EvalFile);

   JIM_REGISTER_API(EvalObj);

@@ -9100,6 +9158,7 @@ void JimRegisterCoreApi(Jim_Interp *interp)
   JIM_REGISTER_API(NewStringObj);

   JIM_REGISTER_API(NewStringObjNoAlloc);

   JIM_REGISTER_API(AppendString);

+  JIM_REGISTER_API(AppendString_sprintf);

   JIM_REGISTER_API(AppendObj);

   JIM_REGISTER_API(AppendStrings);

   JIM_REGISTER_API(StringEqObj);

@@ -12533,27 +12592,15 @@ int
 Jim_SetResult_sprintf( Jim_Interp *interp, const char *fmt,... )

 {

 	va_list ap;

-#ifndef HAVE_VASPRINTF

-	/* yucky way */

-	char buf[2048];

-

-	va_start(ap,fmt);

-	vsnprintf( buf, sizeof(buf), fmt, ap );

-	va_end(ap);

-	/* garentee termination */

-	buf[2047] = 0;

-	Jim_SetResultString( interp, buf, -1 );

-

-#else

 	char *buf;

+

 	va_start(ap,fmt);

-	vasprintf( &buf, fmt, ap );

+	buf = jim_vasprintf( fmt, ap );

 	va_end(ap);

 	if( buf ){

 		Jim_SetResultString( interp, buf, -1 );

-		free(buf);

+		jim_vasprintf_done(buf);

 	}

-#endif

 	return JIM_OK;

 }