<div dir="ltr">Also, most of the red and yellow on the dashboard should be cleaned up once that topic lands. There were some issues that gerrit didn't catch, and the holidays have delayed these fixes going in.<div><br></div><div>Bear with me, folks, we'll have it green again soon ;-)</div></div><div class="gmail_extra"><br><div class="gmail_quote">On Tue, Jan 6, 2015 at 1:48 PM, David Lonie <span dir="ltr"><<a href="mailto:david.lonie@kitware.com" target="_blank">david.lonie@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 dir="ltr">This should do the trick:<div><br></div><div><a href="http://review.source.kitware.com/#/c/18681/" target="_blank">http://review.source.kitware.com/#/c/18681/</a><br></div><div><br></div><div>Dave</div></div><div class="HOEnZb"><div class="h5"><div class="gmail_extra"><br><div class="gmail_quote">On Tue, Jan 6, 2015 at 1:40 PM, David Lonie <span dir="ltr"><<a href="mailto:david.lonie@kitware.com" target="_blank">david.lonie@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 dir="ltr">Thanks for the heads up -- I'm working on a cleanup branch for this topic now, and will add these fixes to it.<div><br></div><div>Dave</div></div><div><div><div class="gmail_extra"><br><div class="gmail_quote">On Tue, Jan 6, 2015 at 1:39 PM, David Gobbi <span dir="ltr"><<a href="mailto:david.gobbi@gmail.com" target="_blank">david.gobbi@gmail.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div dir="ltr"><div>The mingw dashboard is showing tons of warnings about the inline</div><div>methods in vtkStructuredData.h</div><div><br></div><a href="https://open.cdash.org/viewBuildError.php?type=1&buildid=3641101" target="_blank">https://open.cdash.org/viewBuildError.php?type=1&buildid=3641101</a><br><div><br></div><div>I believe that this is due to the order in which the inlined methods</div><div>are defined.  Specifically, inline methods should not be used prior to</div><div>their definition, otherwise the mingw compiler complains.</div><div><br></div><div>For example, this:</div><div><br></div><div><div>  inline vtkIdType vtkStructuredData::GetNumberOfCells(int ext[6], int)</div><div>  {</div><div>    int cellDims[3];</div><div>    vtkStructuredData::GetCellDimensionsFromExtent(ext,cellDims);</div></div><div>    ...</div><div>  }</div><div><br></div><div>appears in the header file before this:</div><div><br></div><div><div>  inline void vtkStructuredData::GetCellDimensionsFromExtent(</div><div>      int ext[6], int celldims[3], int)</div><div>  {</div><div>    celldims[0] = vtkStructuredData::Max(ext[1] - ext[0], 0);</div><div>    celldims[1] = vtkStructuredData::Max(ext[3] - ext[2], 0);</div><div>    celldims[2] = vtkStructuredData::Max(ext[5] - ext[4], 0);</div><div>  }</div></div><div><br></div><div>If the definition of GetCellDimensionsFromExtent() appeared before the</div><div>definition of GetNumberOfCells(), then mingw wouldn't complain.</div><span><font color="#888888"><div><br></div><div>  - David</div><div><br></div></font></span></div>
</blockquote></div><br></div>
</div></div></blockquote></div><br></div>
</div></div></blockquote></div><br></div>