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
