<div dir="ltr">On Fri, Feb 15, 2013 at 4:20 PM, Barry Smith <span dir="ltr"><<a href="mailto:bsmith@mcs.anl.gov" target="_blank">bsmith@mcs.anl.gov</a>></span> wrote:<br><div class="gmail_extra"><div class="gmail_quote">
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><br>
  BTW petscviewer.h included by petscsys.h always includes hdf5.h (when PETSc installed with hdf5) because of hsize_t and hid_t are in the public API of PETSc viewer for hdf5.<br>
<br>
   We can either have separate petscviewerhdf5.h or we could include just H5public.h and H5Ipublic.h in petscviewer.h to get these beasties defined.<br>
<br>
   I vote for the first but am fine with the second.<br></blockquote><div><br></div><div style>I also prefer the first. Very few users of PetscViewer need or want to know anything about HDF5.</div></div></div></div>