[vtk-developers] [vtkusers] Compile Bug in VTK 5.6

[Nicolas Rannou]

Hi all, 

Just to let you know that we are also experimenting some warning with
this file, while linking it against our application.
(http://my.cdash.org/viewBuildError.php?type=1&buildid=85242)

-----------------------------------------------------------------------
Linking CXX executable ../bin/gofigure
/home/nr52/GITROOT/VTK/BUILD/RELEASE/STATIC/bin/libvtksqlite.a(vtk_sqlite3.c.o): In function `exprDup':
vtk_sqlite3.c:(.text+0x2d384): warning: memset used with constant zero length parameter; this could be due to transposed parameters
-----------------------------------------------------------------------

Did you already hear about this warning by any chance?
Thanks,

    Nicolas

On Thu, 2010-07-29 at 19:10 -0400, David Gobbi wrote:
> Interesting, I tried to push the "if" fix but I couldn't.  Git says
> the following:
> 
> ----------------------------------------------------------------------
> commit 5bb0cc15 creates blob c26da674 at path
>   Utilities/vtksqlite/vtk_sqlite3.c
> with size 3870 KB, greater than the maximum 1024 KB
> ----------------------------------------------------------------------
> 
> The file is just too darn big.
> 
>    David
> 
> 
> On Thu, Jul 29, 2010 at 3:52 PM, Sean McBride <sean at rogue-research.com> wrote:
> > On Thu, 29 Jul 2010 15:19:31 -0600, David Gobbi said:
> >
> >>Quick update.  This afternoon's clang build of VTK is clean on my 10.6
> >>system.  No compile errors, very few warnings, 100% of tests passed.
> >
> > For this warning:
> >
> > vtk_sqlite3.c:46188:3: warning: expression result unused [-Wunused-value]
> >   expandBlob(pMem);
> >   ^~~~~~~~~~~~~~~~
> > #define expandBlob(P) (((P)->flags&MEM_Zero)?vtk_sqlite3VdbeMemExpandBlob
> > (P):0)
> >
> >                                                                             ^
> > casting the 0 to void will silence the warning, like so:
> >
> > #define expandBlob(P) (((P)->flags&MEM_Zero)?vtk_sqlite3VdbeMemExpandBlob
> > (P):(void)0)
> >
> > or perhaps the macro could be transformed into an "if(foo){bar()}" construct.
> >
> > --
> > ____________________________________________________________
> > Sean McBride, B. Eng                 sean at rogue-research.com
> > Rogue Research                        www.rogue-research.com
> > Mac Software Developer              Montréal, Québec, Canada
> >
> >
> >
> _______________________________________________
> Powered by www.kitware.com
> 
> Visit other Kitware open-source projects at http://www.kitware.com/opensource/opensource.html
> 
> Follow this link to subscribe/unsubscribe:
> http://www.vtk.org/mailman/listinfo/vtk-developers
>