Author: byterock
Date: Tue Jun 24 08:07:58 2008
New Revision: 11450

Modified:
   dbd-oracle/trunk/Changes
   dbd-oracle/trunk/MANIFEST
   dbd-oracle/trunk/Makefile.PL
   dbd-oracle/trunk/dbdimp.c
   dbd-oracle/trunk/oci8.c

Log:
 Added patch to fix a number of harmless, but annoying, GCC warnings from Eric 
Simon

Modified: dbd-oracle/trunk/Changes
==============================================================================
--- dbd-oracle/trunk/Changes    (original)
+++ dbd-oracle/trunk/Changes    Tue Jun 24 08:07:58 2008
@@ -1,4 +1,5 @@
 =head1 Changes in DBD-Oracle 1.22(svn rev xxxx)  2008 
+  Added patch to fix a number of harmless, but annoying, GCC warnings from 
Eric Simon
   Added (finally) ora_verbose for DBD only tracking from John Scoles and 
thanks to H.Merijn Brand
   Fix for rt.cpan.org Ticket #=32396 from John Scoles
   Fix for memory leak that snuck into 1.21 from John Scoles

Modified: dbd-oracle/trunk/MANIFEST
==============================================================================
--- dbd-oracle/trunk/MANIFEST   (original)
+++ dbd-oracle/trunk/MANIFEST   Tue Jun 24 08:07:58 2008
@@ -1,8 +1,22 @@
 Changes
+dbdimp.c
+dbdimp.h
+dbivport.h
+hints/dgux.pl
+hints/macos_bundle.syms
+hints/macos_lib.syms
+hints/macos_syms.pl
+hints/svr4.pl
+lib/DBD/Oracle/GetInfo.pm
+Makefile.PL
 MANIFEST
 META.yml                       Module meta-data (added by MakeMaker)
-Makefile.PL
-Oracle.ex/README
+mk.pm
+mkta.pl
+oci.def                                OCI.DLL export declarations
+oci8.c
+ocitrace.h
+ora_explain.PL
 Oracle.ex/bind.pl
 Oracle.ex/commit.pl
 Oracle.ex/curref.pl
@@ -11,11 +25,13 @@
 Oracle.ex/mktable.pl
 Oracle.ex/oradump.pl
 Oracle.ex/proc.pl
+Oracle.ex/README
 Oracle.ex/sql
 Oracle.ex/tabinfo.pl
 Oracle.h
 Oracle.pm
 Oracle.xs
+oraperl.ph Old oraperl file included for completeness of emulation
 Oraperl.pm
 README
 README-files/hpux/Conf-Lincoln-1.06
@@ -32,8 +48,9 @@
 README-files/hpux/Conf-Merijn-588-11.23-gcc64
 README-files/hpux/Conf-Mike
 README-files/hpux/Conf-Roger
-README-files/hpux/Makefile-Lincoln
 README-files/hpux/libjava.eml
+README-files/hpux/Makefile-Lincoln
+README.64bit.txt
 README.aix.txt
 README.clients.txt
 README.explain.txt
@@ -48,21 +65,6 @@
 README.vms.txt
 README.win32.txt
 README.wingcc.txt                      Notes about building with mingw32 and 
cygwin32
-Todo
-dbdimp.c
-dbdimp.h
-dbivport.h
-hints/dgux.pl
-hints/macos_bundle.syms
-hints/macos_lib.syms
-hints/macos_syms.pl
-hints/svr4.pl
-lib/DBD/Oracle/GetInfo.pm
-oci.def                                OCI.DLL export declarations
-oci8.c
-ocitrace.h
-ora_explain.PL
-oraperl.ph                     Old oraperl file included for completeness of 
emulation
 t/01base.t
 t/10general.t
 t/12impdata.t
@@ -93,4 +95,5 @@
 t/80ora_charset.t
 t/nchar_test_lib.pl
 test.pl
+Todo
 typemap

Modified: dbd-oracle/trunk/Makefile.PL
==============================================================================
--- dbd-oracle/trunk/Makefile.PL        (original)
+++ dbd-oracle/trunk/Makefile.PL        Tue Jun 24 08:07:58 2008
@@ -683,7 +683,7 @@
     if ($Config{gccversion}) {
        $opts{DEFINE} .= ' -Wall -Wcast-align -Wpointer-arith';
        $opts{DEFINE} .= ' -Wbad-function-cast -Wcast-qual';
-       $opts{DEFINE} .= ' -Wconversion'; # noisy
+       #$opts{DEFINE} .= ' -Wconversion'; # very noisy so remove to see what 
people say
        $opts{DEFINE} .= ' -Wimplicit  -Wimplicit-int 
-Wimplicit-function-declaration -Werror-implicit-function-declaration -Wimport  
-Winline -Wlong-long -Wmissing-braces -Wmissing-format-attribute  
-Wmissing-noreturn -Wmultichar  -Wpacked -Wparentheses  -Wpointer-arith  
-Wreturn-type  -Wsequence-point -Wsign-compare  -Wswitch  -Wtrigraphs  -Wundef 
-Wuninitialized -Wunreachable-code -Wunused  -Wunused-function  -Wunused-label  
-Wunused-parameter -Wunused-value  -Wunused-variable  -Wwrite-strings 
-Wbad-function-cast  -Wmissing-declarations -Wnested-externs'
                if $::opt_w;
     }

Modified: dbd-oracle/trunk/dbdimp.c
==============================================================================
--- dbd-oracle/trunk/dbdimp.c   (original)
+++ dbd-oracle/trunk/dbdimp.c   Tue Jun 24 08:07:58 2008
@@ -776,7 +776,7 @@
 #if defined(USE_ITHREADS) && defined(PERL_MAGIC_shared_scalar)
     if (shared_dbh_ssv && !shared_dbh) {
        /* much of this could be replaced with a single sv_setpvn() */
-               SvUPGRADE(shared_dbh_priv_sv, SVt_PV) ;
+               (void)SvUPGRADE(shared_dbh_priv_sv, SVt_PV);
                SvGROW(shared_dbh_priv_sv, sizeof(imp_dbh_t) + 1) ;
                SvCUR (shared_dbh_priv_sv) = sizeof(imp_dbh_t) ;
                imp_dbh->refcnt = 1 ;

Modified: dbd-oracle/trunk/oci8.c
==============================================================================
--- dbd-oracle/trunk/oci8.c     (original)
+++ dbd-oracle/trunk/oci8.c     Tue Jun 24 08:07:58 2008
@@ -1483,8 +1483,8 @@
             }
 
             sprintf(s_tz_min,":%02d", tz_minute);
-            strcat(str_buf, s_tz_hour);
-            strcat(str_buf, s_tz_min);
+            strcat((signed char*)str_buf, s_tz_hour);
+            strcat((signed char*)str_buf, s_tz_min);
             str_buf[str_len+7] = '\0';
 
                } else {
@@ -1506,7 +1506,9 @@
      case OCI_TYPECODE_BLOB:
         case OCI_TYPECODE_BFILE:
                raw_sv = newSV(0);
-               fetch_lob(sth, fbh->imp_sth,*(OCILobLocator**)attr_value, 
typecode, raw_sv, name);
+               fetch_lob(sth, fbh->imp_sth,*(OCILobLocator**)attr_value, 
typecode, raw_sv, (signed char*)name);
+
+
                av_push(list, raw_sv);
                break;
 
@@ -1667,7 +1669,7 @@
                                case OCI_TYPECODE_VARRAY :                    
/* variable array */
                                fld = &obj->fields[0]; /*get the field */
                        OCIIterCreate_log_stat(fbh->imp_sth->envhp, 
fbh->imp_sth->errhp,
-                         (CONST OCIColl*) value, &itr,status);
+                         (OCIColl*) value, &itr,status);
 
                                        if (status != OCI_SUCCESS) {
                                                /*not really an error just no 
data
@@ -2790,8 +2792,8 @@
                                if (!fbh->fetch_func) {
                                    /* Copy the truncated value anyway, it may 
be of use,       */
                                    /* but it'll only be accessible via prior 
bind_column()     */
-                                   sv_setpvn(sv, 
row_data,fb_ary->arlen[imp_sth->rs_array_idx]);
-                                   if ((CSFORM_IMPLIES_UTF8(fbh->csform)) && 
(fbh->ftype != SQLT_BIN)){
+                                   sv_setpvn(sv, (char 
*)row_data,fb_ary->arlen[imp_sth->rs_array_idx]);
+                                   if ((CSFORM_IMPLIES_UTF8(fbh->csform)) && 
(fbh->ftype != SQLT_BIN)){
                                                SvUTF8_on(sv);
                                        }
                                }

Reply via email to