<div dir="ltr">Hi!<div>Thanks David! That really helped!</div><div>I added small comments along with the headers, and it worked.</div><div><br></div><div>@Marcus: I accidentally provided the link to an old repository. The updated ones have forward declarations as required, and the rest have been moved to the cxx file. Aashish sir had pointed out those changes earlier itself.</div>
<div><br></div><div>Thanks marcus and david!</div><div>Jatin.</div></div><div class="gmail_extra"><br><br><div class="gmail_quote">On Thu, Jun 26, 2014 at 10:47 PM, Marcus D. Hanwell <span dir="ltr"><<a href="mailto:marcus.hanwell@kitware.com" target="_blank">marcus.hanwell@kitware.com</a>></span> wrote:<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div class="">On Thu, Jun 26, 2014 at 11:49 AM, David E DeMarle<br>
<<a href="mailto:dave.demarle@kitware.com">dave.demarle@kitware.com</a>> wrote:<br>
><br>
> On Thu, Jun 26, 2014 at 10:47 AM, Jatin Parekh <<a href="mailto:jatinparekh93@gmail.com">jatinparekh93@gmail.com</a>><br>
> wrote:<br>
>><br>
>> VTKIOGEOJSON_EXPORT<br>
><br>
><br>
> Means that you need to put this macro (defined by vtk's cmake scripts) in<br>
> your class definitions. It makes declspecs for windows shared lib builds. As<br>
> Aashish says, see other vtk headers for examples. You will see how they all<br>
> include a configured header that defines it.<br>
><br>
>> has 2 includes<br>
><br>
><br>
> The multiple headers is a vtk style guideline. To keep compilation quick and<br>
> declarations contained the guidelines recommend a single include per header<br>
> file. In practice we need two since modularization for the modern export<br>
> macros but that is beside the point. If you need more than three includes in<br>
> your header then you should rethink the design. If you have two or three<br>
> then you can get through this style check by adding a short comment after<br>
> each include which describes why you need to include more than one header.<br>
><br>
</div>Looking at some of your headers it is clear you have a lot of includes<br>
that should just be in the implementation, and you should forward<br>
declare classes that you just need as pointers/references in the API.<br>
<a href="https://github.com/jatin3893/vtkDataSetReaders/blob/master/vtkGeoJSONReader.h" target="_blank">https://github.com/jatin3893/vtkDataSetReaders/blob/master/vtkGeoJSONReader.h</a><br>
contains many things that I don't see referenced in the header. I hope<br>
you can keep the JSON include to the implementation.<br>
<span class="HOEnZb"><font color="#888888"><br>
Marcus<br>
</font></span></blockquote></div><br></div>