[vtk-developers] New class: vtkGraphLargestConnectedComponent

[David Doria]

On Fri, May 21, 2010 at 11:20 AM, pat marion <pat.marion at kitware.com> wrote:
> Looks cool!  Hope you don't mind if I make a comment- you should always have
>
>   #include "vtkGraphLargestConnectedComponent.h"
>
> as the first include in your cxx file.  If you don't, it's possible that
> your header has a compiler error that you won't notice.  For example, the
> header might reference a class without using a forward declaration 'class
> vtkDataSet;' but you don't notice the error because your cxx file includes
> other headers that do have the forward declaration before it includes the
> header with the error.  Then someone else comes along and tries to include
> your header and they get the error.
>
> Pat

Pat,

Absolutely! The more comments/suggestions/criticisms the better.

That is definitely good point. I'm pretty sure everything I have ever
written has this problem - I'll be sure to go back and correct
everything.

Any other reviews?

Thanks,

David