<div dir="ltr"><div class="gmail_extra"><div class="gmail_quote">On Thu, Feb 19, 2015 at 3:54 PM, Barry Smith <span dir="ltr"><<a href="mailto:bsmith@mcs.anl.gov" target="_blank">bsmith@mcs.anl.gov</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><br>
  Is it intended for endusers -> petschash.h (requires decent documentation and error checking) or for use by PETSc -> petsc-private/hashimpl.h</blockquote><div><br></div><div>I am fine with the impl version. I will move all the includes to source files and cleanup.</div><div><br></div><div>  Matt</div><div> </div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div class="HOEnZb"><div class="h5">
> On Feb 19, 2015, at 3:47 PM, Matthew Knepley <<a href="mailto:knepley@gmail.com">knepley@gmail.com</a>> wrote:<br>
><br>
> On Thu, Feb 19, 2015 at 2:39 PM, Jed Brown <<a href="mailto:jed@jedbrown.org">jed@jedbrown.org</a>> wrote:<br>
> Matthew Knepley <<a href="mailto:knepley@gmail.com">knepley@gmail.com</a>> writes:<br>
><br>
> > On Thu, Feb 19, 2015 at 1:42 AM, Lisandro Dalcin <<a href="mailto:dalcinl@gmail.com">dalcinl@gmail.com</a>> wrote:<br>
> ><br>
> >> Please take look to the last line of the following output:<br>
> >><br>
> >> $ grep include $PETSC_DIR/include/petsc-private/dmpleximpl.h<br>
> >> #include <petscmat.h>       /*I      "petscmat.h"          I*/<br>
> >> #include <petscdmplex.h> /*I      "petscdmplex.h"    I*/<br>
> >> #include <petscbt.h><br>
> >> #include <petscsf.h><br>
> >> #include <petsc-private/dmimpl.h><br>
> >> #include <../src/sys/utils/hash.h><br>
> >><br>
> ><br>
> > It does not look like anything else uses hash.h. Can we just move it to<br>
> > include/petschash.h?<br>
><br>
> Why does it need to be included by a header?<br>
><br>
> I thought something else was using this, but it looks like I am the only one. We<br>
> should put it in a header and clear out the cruft no one uses anymore:<br>
><br>
> knepley/feature-harmonic-advection *$:/PETSc3/petsc/petsc-dev$ find ./src -type f | xargs grep hash.h<br>
> find ./src -type f | xargs grep hash.h<br>
> ./src/dm/impls/plex/plexinterpolate.c:#include <../src/sys/utils/hash.h><br>
> ./src/dm/impls/plex/plexinterpolate.c~:#include <../src/sys/utils/hash.h><br>
> ./src/dm/impls/plex/plextree.c:#include <../src/sys/utils/hash.h><br>
> ./src/sys/examples/tests/ex26.c:#include <../src/sys/utils/hash.h><br>
> ./src/sys/utils/hash.h:#include <khash.h><br>
>  1knepley/feature-harmonic-advection *$:/PETSc3/petsc/petsc-dev$ find ./include -type f | xargs grep hash.h<br>
> find ./include -type f | xargs grep hash.h<br>
> ./include/petsc-private/dmpleximpl.h:#include <../src/sys/utils/hash.h><br>
><br>
>   Matt<br>
><br>
> --<br>
> What most experimenters take for granted before they begin their experiments is infinitely more interesting than any results to which their experiments lead.<br>
> -- Norbert Wiener<br>
<br>
</div></div></blockquote></div><br><br clear="all"><div><br></div>-- <br><div class="gmail_signature">What most experimenters take for granted before they begin their experiments is infinitely more interesting than any results to which their experiments lead.<br>-- Norbert Wiener</div>
</div></div>