[Insight-developers] GDCM empty cxx files

[Bill Lorensen]

Mathieu,

We still support VS7.1. I would like to submit a patch that removes these
files from the lib and see if it works.

Unfortunately, the VS generator does not work with the modularized ITK. We
may not be able to thoroughly test it until the patched cmake is released.

Thanks,

Bill

On Fri, Mar 25, 2011 at 2:13 PM, Mathieu Malaterre <
mathieu.malaterre at gmail.com> wrote:

> From the top of my head, this was V7.1 with /MT as compilation flag
> (instead of the default /MD used by Cmake). Can't remember if this was
> using shared or static lib.
>
> Does anyone still have this compiler ?
>
> HTH
>
> On Fri, Mar 25, 2011 at 7:06 PM, Bill Lorensen <bill.lorensen at gmail.com>
> wrote:
> > Mathieu,
> >
> > Several files including gdcmObject.cxx have no code in them. Your comment
> in
> > gdcmObject.cxx says:
> >   // Don't ask why, but this is EXTREMELY important on Win32
> >   // Apparently the compiler is doing something special the first time it
> > compiles
> >   // this instanciation unit
> >   // If this fake file is not present I get an unresolved symbol for each
> > function
> >   // of the gdcm::Object class
> >
> > When linking these "empty" files, VS10 reports several warnings:
> >  warning LNK4221: This object file does not define any previously
> undefined
> > public symbols, so it will not be used by any link operation that
> consumes
> > this library
> >
> > I have removed this file from the lib as well as
> > gdcmProgressEvent.cxx, gdcmString.cxx, gdcmException.cxx
> > gdcmDeflateStream.cxx and gdcmByteSwap.cxx
> >
> > When I build the modified lib, I do not get the warnoings, nor do I get
> any
> > errors.
> >
> > What compiler was giving the errors you mention in the comments?
> >
> > Bill
> >
>
>
>
> --
> Mathieu
>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://www.itk.org/mailman/private/insight-developers/attachments/20110325/3b788246/attachment.htm>