← Back to team overview

dolfin team mailing list archive

[HG dolfin] Merge: Note comments in previous changeset on fix of nasty bug

 

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

changeset:   2989:cda90b64005f8f50d57748664bf38aab6f8f171a
tag:         tip
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.


changeset:   2987:746cb32d2addefb70ea1e4bb4997641708597536
parent:      2986:a870c3ae41f0be74a3588c0fd4498cafd3671124
parent:      2985:e339e105a474c9cc5b90a7415d95e509941c7652
user:        "Garth N. Wells <g.n.wells@xxxxxxxxxx>"
date:        Thu May 03 21:38:29 2007 -0400
files:       
description:
merge.


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