If it's only one commit, send a link exactly to that commit, so people can click on it and see the diffs right in their web browser:<div><br></div><div><a href="http://github.com/daviddoria/daviddoria-vtk/commit/43efaf1bc09bd8c21e9d15c9366b86b83288f99a">http://github.com/daviddoria/daviddoria-vtk/commit/43efaf1bc09bd8c21e9d15c9366b86b83288f99a</a></div>
<div><br></div><div>If it's a series of commits, be sure you ask for people to look at "the last N commits" or whatever is appropriate.</div><div><br></div><div>I'll let the math experts chime in and review the actual content of the diffs.</div>
<div><br></div><div>A review like this would be easier if you would separate out the "white space only" changes into a separate commit, so the people interested in reviewing the actual diff content don't have to wade through the 80% to find the 20%.... :-)</div>
<div><br></div><div><br></div><div>David C.</div><div><br><br><div class="gmail_quote">On Sun, Sep 12, 2010 at 9:18 AM, David Doria <span dir="ltr"><<a href="mailto:daviddoria%2Bvtk@gmail.com">daviddoria+vtk@gmail.com</a>></span> wrote:<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex;">I added a function called ProjectVector to vtkMath along with a test<br>
for it in TestMath.cxx.<br>
<br>
repo: git@github.com:daviddoria/daviddoria-vtk.git<br>
branch: VTK-VectorProjection<br>
<br>
Any comments/objections?<br>
<br>
David<br>
_______________________________________________<br>
Powered by <a href="http://www.kitware.com" target="_blank">www.kitware.com</a><br>
<br>
Visit other Kitware open-source projects at <a href="http://www.kitware.com/opensource/opensource.html" target="_blank">http://www.kitware.com/opensource/opensource.html</a><br>
<br>
Follow this link to subscribe/unsubscribe:<br>
<a href="http://www.vtk.org/mailman/listinfo/vtk-developers" target="_blank">http://www.vtk.org/mailman/listinfo/vtk-developers</a><br>
<br>
</blockquote></div><br></div>