Anders Logg wrote: > On Sun, Aug 17, 2008 at 11:15:23PM +0200, DOLFIN wrote: >> One or more new changesets pushed to the primary dolfin repository. >> A short summary of the last three changesets is included below. >> >> changeset: 4592:b0efa4df44cec1366db459bddf093882bd1eaf1e >> tag: tip >> user: "Garth N. Wells <[EMAIL PROTECTED]>" >> date: Sun Aug 17 22:14:54 2008 +0100 >> files: dolfin/graph/Graph.cpp dolfin/graph/Graph.h >> dolfin/graph/GraphBuilder.cpp dolfin/graph/GraphBuilder.h >> dolfin/graph/GraphPartition.cpp dolfin/graph/dolfin_graph.h >> dolfin/mesh/MeshPartition.cpp dolfin/swig/dolfin_docstrings.i >> dolfin/swig/dolfin_headers.i >> description: >> Add class GraphBuilder. > > Nice, but shouldn't there be a constructor for Graph that takes a > Mesh? It can just call GraphBuilder::build(). >
Perhaps, but I'd prefer that the Graph interface be unaware of the Mesh or other object it's built from. GraphBuilder will soon build Graphs from objects other than meshes. Either way, it'll be easy to change whichever way we go. Garth > > > ------------------------------------------------------------------------ > > _______________________________________________ > DOLFIN-dev mailing list > [email protected] > http://www.fenics.org/mailman/listinfo/dolfin-dev - _______________________________________________ DOLFIN-dev mailing list [email protected] http://www.fenics.org/mailman/listinfo/dolfin-dev
