[svn:dbd-oracle] r13777 - dbd-oracle/trunk

[email protected]
Newsgroups perl.dbd.oracle.changes
Message-ID <[email protected]>
Author: byterock
Date: Thu Jan 28 07:40:14 2010
New Revision: 13777

Modified:
   dbd-oracle/trunk/dbdimp.c
   dbd-oracle/trunk/oci8.c

Log:
more fixes from rc testing this time from Martin

Modified: dbd-oracle/trunk/dbdimp.c
==============================================================================
--- dbd-oracle/trunk/dbdimp.c	(original)
+++ dbd-oracle/trunk/dbdimp.c	Thu Jan 28 07:40:14 2010
@@ -1135,8 +1135,7 @@
 bufp = SvPV(source, len);
 
 	if (DBIS->debug >=3 || dbd_verbose >= 3 )
-		PerlIO_printf(DBILOGFP, " creating xml from string that is %lu long\n",len);
-
+        PerlIO_printf(DBILOGFP, " creating xml from string that is %lu long\n",(unsigned long)len);
 	if(len > MAX_OCISTRING_LEN) {
 		src_type = OCI_XMLTYPE_CREATE_CLOB;
 

Modified: dbd-oracle/trunk/oci8.c
==============================================================================
--- dbd-oracle/trunk/oci8.c	(original)
+++ dbd-oracle/trunk/oci8.c	Thu Jan 28 07:40:14 2010
@@ -983,7 +983,7 @@
 			return 0;
 	}
 /*	else {
-		/* set initial cache size by memory
+		 set initial cache size by memory
 		    [I'm not now sure why this is here - from a patch sometime ago - Tim]
 		    you are right Tim thre is no need to have this here so out it goes
 		    a very useless call to the server
@@ -1895,11 +1895,10 @@
 
 
 	if (DBIS->debug >= 3 || dbd_verbose >= 3 || oci_warn){
-		char buf[10];
-		sprintf(buf,"bytes");
-
+		char buf[11];
+		strcpy(buf,"bytes");
 		if (ftype == ORA_CLOB)
-           sprintf(buf,"characters");
+			strcpy(buf,"characters");
 
 		PerlIO_printf(DBILOGFP,
 		"		OCILobRead %s %s: csform %d (%s), LOBlen %lu(%s), LongReadLen %lu(%s), BufLen %lu(%s), Got %lu(%s)\n",
@@ -3806,7 +3805,6 @@
 			to cast it - currently only number types are handled */
 					if (fbh->req_type != 0) {
 						int sts;
-						D_imp_xxh(sth);
 						char errstr[256];
 
 						sts = DBIc_DBISTATE(imp_sth)->sql_type_cast_svpv(
@@ -4321,7 +4319,7 @@
 		phs_t *phs;
 		SV **phs_svp = hv_fetch(imp_sth->all_params_hv, fbh->name,strlen(fbh->name), 0);
 		if (!phs_svp)
-			croak("panic: LOB refetch for '%s' param (%d) - name not found",fbh->name,i+1);
+			croak("panic: LOB refetch for '%s' param (%ld) - name not found",fbh->name,(unsigned long)i+1);
 		phs = (phs_t*)(void*)SvPVX(*phs_svp);
 		fbh->special = phs;
 		if (DBIS->debug >= 3 || dbd_verbose >= 3 )
lmpx.com only provides a reader for public news (NNTP) servers. It is not affiliated with the servers or forums shown here and is not responsible for the content of articles, which is written by their respective authors.