← Back to team overview

dolfin team mailing list archive

[HG dolfin] Bug fix in local mesh coarsening introduced by modified mesh

 

One or more new changesets pushed to the primary DOLFIN repository.
A short summary of the last three changesets is included below.

changeset:   2990:ac9e363bdb9f1a6525372d81706aece1fae37c45
tag:         tip
user:        "Anders Logg <logg@xxxxxxxxx>"
date:        Fri May 04 17:38:23 2007 +0200
files:       src/kernel/mesh/LocalMeshCoarsening.cpp
description:
Bug fix in local mesh coarsening introduced by modified mesh
iterator interface


changeset:   2989:cda90b64005f8f50d57748664bf38aab6f8f171a
parent:      2987:746cb32d2addefb70ea1e4bb4997641708597536
parent:      2988:193b22c9395605a20d218e83693e02d18bdeb6c1
user:        "Anders Logg <logg@xxxxxxxxx>"
date:        Fri May 04 17:03:42 2007 +0200
files:       src/kernel/mesh/LocalMeshCoarsening.cpp
description:
Merge: Note comments in previous changeset on fix of nasty bug
in mesh iterators which made a small interface change necessary
for how nested mesh iterators are be defined.


changeset:   2988:193b22c9395605a20d218e83693e02d18bdeb6c1
parent:      2981:89b78a82bdad888d5f83c2005ea1be6878c8ba00
user:        "Anders Logg <logg@xxxxxxxxx>"
date:        Wed May 02 23:33:10 2007 +0200
files:       src/kernel/function/DiscreteFunction.cpp src/kernel/graph/Graph.cpp src/kernel/io/GiDFile.cpp src/kernel/io/MFile.cpp src/kernel/io/OpenDXFile.cpp src/kernel/io/VTKFile.cpp src/kernel/mesh/BoundaryComputation.cpp src/kernel/mesh/LocalMeshCoarsening.cpp src/kernel/mesh/LocalMeshRefinement.cpp src/kernel/mesh/SubDomain.cpp src/kernel/mesh/TopologyComputation.cpp src/kernel/mesh/dolfin/Cell.h src/kernel/mesh/dolfin/Edge.h src/kernel/mesh/dolfin/Face.h src/kernel/mesh/dolfin/Facet.h src/kernel/mesh/dolfin/MeshEntityIterator.h src/kernel/mesh/dolfin/Vertex.h src/sandbox/graph/main.cpp
description:
Fix very nasty bug in mesh iterators. To avoid ambiguation in
choice of constructor for mesh entity iterators, constructors
taking a mesh entity iterator (which were sometimes being replaced
by default copy constructors) have been removed. It is no longer
possible to write

  for (CellIterator c(mesh); !c.end(); ++c)
    for (VertexIterator v(c); !v.end(); ++v)
      ...

One must instead write

  for (CellIterator c(mesh); !c.end(); ++c)
    for (VertexIterator v(*c); !v.end(); ++v)
      ...

thus iterating over the connected entities of an entity (not an
entity iterator) which makes sense.


----------------------------------------------------------------------
For more details, visit http://www.fenics.org/hg/dolfin