Re: Warnings
"Martin J. Evans" <[email protected]>
| Newsgroups | gmane.comp.lang.perl.modules.dbi.sybase.devel |
|---|---|
| Message-ID | <[email protected]> |
On 17/09/2010 19:51, David E. Wheeler wrote: > Hey All, > > Just installed 1.614 on three of my boxes. Looks good, and thanks for getting it out! I did notice some warnings, though: > > /usr/local/bin/perl /usr/local/lib/perl5/5.12.2/ExtUtils/xsubpp -typemap /usr/local/lib/perl5/5.12.2/ExtUtils/typemap -typemap typemap DBI.xs> DBI.xsc&& mv DBI.xsc DBI.c > cc -c -fno-common -DPERL_DARWIN -no-cpp-precomp -fno-strict-aliasing -pipe -fstack-protector -I/usr/local/include -O3 -DVERSION=\"1.614\" -DXS_VERSION=\"1.614\" "-I/usr/local/lib/perl5/5.12.2/darwin-thread-multi-2level/CORE" -W -Wall -Wpointer-arith -Wbad-function-cast -Wno-comment -Wno-sign-compare -Wno-cast-qual -Wmissing-noreturn -Wno-unused-parameter DBI.c > DBI.xs: In function ‘dbih_setup_handle’: > DBI.xs:1280: warning: value computed is not used > DBI.xs:1318: warning: value computed is not used > DBI.xs:1319: warning: value computed is not used > DBI.xs:1324: warning: value computed is not used > DBI.xs:1327: warning: value computed is not used > DBI.xs: In function ‘dbih_set_attr_k’: > DBI.xs:2062: warning: value computed is not used > DBI.xs: In function ‘dbih_get_attr_k’: > DBI.xs:2157: warning: value computed is not used > DBI.xs:2384: warning: value computed is not used > DBI.xs: In function ‘_profile_next_node’: > DBI.xs:2590: warning: value computed is not used > DBI.xs: In function ‘XS_DBI_dispatch’: > DBI.xs:3143: warning: value computed is not used > DBI.xs:3222: warning: value computed is not used > DBI.xs: In function ‘XS_DBI__new_handle’: > DBI.xs:4243: warning: value computed is not used > DBI.xs: In function ‘XS_DBD_____st_fetchrow_hashref’: > DBI.xs:4965: warning: value computed is not used > > New DBIx::Connector now on its way to CPAN, too. > > Thanks, > > David > They are all hv_store calls and hv_store returns a value. I think they are all harmless and the only way to make them go away is to prefix each hv_store call with (void). I'm happy to sort that out unless anyone objects. Martin