[petsc-dev] characteristic

[Jed Brown]

I noticed the shared library fixes go in the other day (along with it
moving out of contrib a while back).  As a purely organizational matter,
shouldn't this thing be at src/characteristic?  There are some dubious
unreachable, #if 0, and debugging (example below) code fragments hanging
around.

        printf("[%d]Remote point (%d) at n+1/2 to neighbor %d: (i:%d, j:%d) (x:%g, y:%g)\n", rank, (int)c->queueSize+1, Qi.proc, Qi.i, Qi.j, Qi.x, Qi.y);

There is also DA-specific code in the interface despite appearances that
the DA-specific part is isolated to impls/da.  It would appear that this
package could use some love if it's going to be widely used (which is
one possible interpretation of the significance of moving out of
contrib).

Jed