<div dir="ltr">So, should I resend it with `git mv` or we will leave this file's name as it is?<br></div><div class="gmail_extra"><br><br><div class="gmail_quote">On Thu, Jun 6, 2013 at 3:23 AM, Brian Paul <span dir="ltr"><<a href="mailto:brianp@vmware.com" target="_blank">brianp@vmware.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="im">On 06/05/2013 03:25 PM, Kenneth Graunke wrote:<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
On 06/05/2013 12:09 PM, Arnas Milasevicius wrote:<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
---<br>
  src/mesa/Makefile.sources     |   2 +-<br>
  src/mesa/SConscript           |   2 +-<br>
  src/mesa/main/draw_validate.c | 745<br>
++++++++++++++++++++++++++++++<u></u>++++++++++++<br>
  3 files changed, 747 insertions(+), 2 deletions(-)<br>
  create mode 100644 src/mesa/main/draw_validate.c<br>
</blockquote>
<br>
It looks like this patch leaves the old api_validate.c file in place, so<br>
we would have two copies of everything.  The proper way to do this is:<br>
$ cd src/mesa/main<br>
$ git mv api_validate.c draw_validate.c<br>
$ <edit Makefile.sources and SConscript><br>
$ git commit -a<br>
<br>
That said...Brian, was this one of your ideas?  I don't see much point<br>
to renaming this file, and renaming files makes it harder to go back in<br>
history with git blame and such.  So unless there's a good reason, I'd<br>
rather leave it be.<br>
</blockquote>
<br></div>
Yes, it's from my personal Mesa to-do list.  Your point about git-blame is well taken so if you'd rather not have the file renamed we can leave it as-is.  It's just another one of those little things that I've always found annoying.<span class="HOEnZb"><font color="#888888"><br>

<br>
-Brian<br>
<br>
</font></span></blockquote></div><br></div>