[vtkusers] CVS bug??
cboehnen at nd.edu
Tue Jun 25 04:33:08 EDT 2002
You Can Increase the Heap Space (properties, add command line /Zm1000 or
whatever size you want), although from my experience it means you will
need to increase the swap space on your computer as well. I ran into
that problem with a program that was only 150K when all was said and
done. Microsoft needs to work on that.
From: vtkusers-admin at public.kitware.com
[mailto:vtkusers-admin at public.kitware.com] On Behalf Of Andy Cedilnik
Sent: Tuesday, June 25, 2002 6:36 AM
To: Charlie H. Chang
Subject: Re: [vtkusers] CVS bug??
This would be my "fault". A while ago we started to test VTK on MS
Visual Studio .NET. One of the interesting things about this compiler is
that it is extremely eager to run out of heap space. So, VTK did not
compile. The problem is that lots of header files include several header
files. This in a large hierarchy like VTK means that when you include
something from the bottom of the hierarchy (such as something from
Hybrid), you include half of the VTK. The solution is to start following
the last addition to coding standard:
* The header file of the class should include only the superclasses
header file. If you need any other includes, include comment at each one
describing why it should be included.
Because of this, some of the header files do not include the same header
files that they used to.
On Tue, 2002-06-25 at 05:07, Charlie H. Chang wrote:
> Ok, I found a way to solve the problem. Simply add the following two
> #include "vtkProperty.h"
> #include "vtkCamera.h"
> will do. However, the same piece of code didn't require to include the
> above two
> header files in previous CVS source code.(about one month ago)
> So I guess that vtkProperty and vtkCamera were included by the other
> header file
> (could be vtkActor.h??) but then they were excluded. What is the
> rationale behind
> this change?
This is the private VTK discussion list.
Please keep messages on-topic. Check the FAQ at:
Follow this link to subscribe/unsubscribe:
More information about the vtkusers