<div dir="ltr"><br><div class="gmail_extra"><br><div class="gmail_quote">On Thu, Jan 15, 2015 at 1:01 PM, Eric Anholt <span dir="ltr"><<a href="mailto:eric@anholt.net" target="_blank">eric@anholt.net</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><span class="">Jason Ekstrand <<a href="mailto:jason@jlekstrand.net">jason@jlekstrand.net</a>> writes:<br>
<br>
> ---<br>
>  src/glsl/nir/nir.h                    |   2 +-<br>
>  src/mesa/Makefile.sources             |   2 -<br>
>  src/mesa/drivers/dri/i965/intel_fbo.c |   2 +-<br>
>  src/mesa/main/set.c                   | 346 ----------------------------------<br>
>  src/mesa/main/set.h                   |  94 ---------<br>
>  src/mesa/main/shared.c                |   2 +-<br>
>  src/mesa/main/syncobj.c               |   2 +-<br>
>  src/mesa/main/vdpau.c                 |   2 +-<br>
>  src/util/Makefile.sources             |   2 +<br>
>  src/util/hash_set.c                   | 346 ++++++++++++++++++++++++++++++++++<br>
>  src/util/hash_set.h                   |  94 +++++++++<br>
>  11 files changed, 447 insertions(+), 447 deletions(-)<br>
>  delete mode 100644 src/mesa/main/set.c<br>
>  delete mode 100644 src/mesa/main/set.h<br>
>  create mode 100644 src/util/hash_set.c<br>
>  create mode 100644 src/util/hash_set.h<br>
<br>
</span>I'm not a fan of renaming the files to hash_set.[ch].  The name of the<br>
datatype is "set", the function names are "set", and people call these<br>
things "sets".<br>
<br>
I'm not going to block based on that, but I'd rather see the filenames<br>
retained.  Other than this, the series is:<br>
<br>
Reviewed-by: Eric Anholt <<a href="mailto:eric@anholt.net">eric@anholt.net</a>><br></blockquote><div><br></div><div>Sure, I can can the rename.  I thought about renaming the whole thing to hash_set but didn't want to deal with the churn.  I'll just switch it to set.h and set.c before pushing. <br></div></div><br></div></div>