[vtkusers] CVS bug??

[Andy Cedilnik]

Hi Charlie,

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.

			Andy Cedilnik


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 lines:
> #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?