SF.net SVN: frontierkernel: [1341] Frontier/branches/Conversant/Common/source/conversantverbs.c

[<set...@us...>]

Revision: 1341
Author:   sethdill
Date:     2006-05-02 22:10:37 -0700 (Tue, 02 May 2006)
ViewCVS:  http://svn.sourceforge.net/frontierkernel/?rev=1341&view=rev

Log Message:
-----------
Bug in function conv_tableToStringVerb(). Now it works on a copy of each tyvaluerecord when marching through the table, rather than on the original. It's less efficient than I had hoped, but it doesn't crash... I think I like it better this way. (Fix suggested by Andre.)

Modified Paths:
--------------
    Frontier/branches/Conversant/Common/source/conversantverbs.c
Modified: Frontier/branches/Conversant/Common/source/conversantverbs.c
===================================================================
--- Frontier/branches/Conversant/Common/source/conversantverbs.c	2006-05-02 20:32:33 UTC (rev 1340)
+++ Frontier/branches/Conversant/Common/source/conversantverbs.c	2006-05-03 05:10:37 UTC (rev 1341)
@@ -1022,7 +1022,7 @@
 {
 	handlestream	hs;			/* this is how we handle writing to the string */
 	hdlhashnode		nomad;		/* trip-trip-tripping merrily through the strings */
-	tyvaluerecord *	pVal;		/* the value record associated with the table cell */
+	tyvaluerecord	val;		/* the value record associated with the table cell */
 	long			ix;			/* position in the handle */
 	
 	newemptyhandle( hOutString );
@@ -1035,12 +1035,12 @@
 	
 	while ( nomad != NULL )
 	{
-		pVal = &(**nomad).val;
+		val = (**nomad).val;
 		
-		if ( ! conv_getValueStringLength( pVal, &ix ) )
+		if ( ! conv_getValueStringLength( &val, &ix ) )
 			return ( false );
 		
-		if ( ! writehandlestreamhandle( &hs, (*pVal).data.stringvalue ) )
+		if ( ! writehandlestreamhandle( &hs, val.data.stringvalue ) )
 		{
 			closehandlestream( &hs );
 			return ( false );


This was sent by the SourceForge.net collaborative development platform, the world's largest Open Source development site.