<div dir="ltr">Yes, that works. Thanks, Jed!</div><div class="gmail_extra"><br><br><div class="gmail_quote">On Sun, Oct 6, 2013 at 1:31 PM, Jed Brown <span dir="ltr"><<a href="mailto:jedbrown@mcs.anl.gov" target="_blank">jedbrown@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">Geoff, thanks for the log.<br>
<br>
| In file included from /Users/goxberry/petsc-3.4.2/src/mat/impls/elemental/matelem.cxx:1:<br>
| In file included from /Users/goxberry/petsc-3.4.2/include/../src/mat/impls/elemental/matelemimpl.h:5:<br>
| In file included from /Users/goxberry/petsc-3.4.2/include/petsc-private/matimpl.h:5:<br>
| In file included from /Users/goxberry/petsc-3.4.2/include/petscmat.h:6:<br>
| In file included from /Users/goxberry/petsc-3.4.2/include/petscvec.h:9:<br>
| In file included from /Users/goxberry/petsc-3.4.2/include/petscis.h:7:<br>
| /Users/goxberry/petsc-3.4.2/include/petscsys.h:2325:117: error: expected ')'<br>
| PETSC_STATIC_INLINE PetscErrorCode PetscSegBufferGetInts(PetscSegBuffer seg,PetscInt count,PetscInt *PETSC_RESTRICT *slot) {return PetscSegBufferGet(seg,count,(void**)slot);}<br>
|                                                                                                                     ^<br>
<br>
<br>
Karl fixed this in 'next', but in a branch started from 'master'.  We<br>
had a discussion about the design at that point and decided that the<br>
"hack" was messy and error-prone.<br>
<br>
<a href="https://bitbucket.org/petsc/petsc/commits/0a8720703c3660d0eaeec3a29abfe2a9bfa7773f" target="_blank">https://bitbucket.org/petsc/petsc/commits/0a8720703c3660d0eaeec3a29abfe2a9bfa7773f</a><br>
<br>
<br>
I think Matt's suggestion of two separate files is overkill and not<br>
clearly better.  Here is an alternative proposal (which is consistent<br>
with the handling of PETSC_FUNCTION_NAME, which works smoothly).<br>
<br>
<a href="https://bitbucket.org/petsc/petsc/commits/ed938b00aafbc8619c94f12cc845a1c4a3fa2bcd" target="_blank">https://bitbucket.org/petsc/petsc/commits/ed938b00aafbc8619c94f12cc845a1c4a3fa2bcd</a><br>
<br>
You can test by:<br>
<br>
  $ git checkout jed/fix-restrict-cxx<br>
  $ ./configure ...<br>
<br>
(Actually, you could just delete the definitions of PETSC_RESTRICT and<br>
PETSC_STATIC_INLINE from petscconf.h, then run "make".)<br>
<br>
This branch starts from 'maint' so you won't see any interface changes<br>
in the above.<br>
</blockquote></div><br><br clear="all"><div><br></div>-- <br><div dir="ltr">Geoffrey Oxberry, Ph.D., E.I.T.<br></div>
</div>