← Back to team overview

dolfin team mailing list archive

Re: Unify MeshEntityIterator and SubEntityIterator ?

 

The important point is to check that there is no overhead. I imagine
there will be an if/else check for whether a subset iterator is used
in the increment operator.

--
Anders


On Mon, Aug 22, 2011 at 07:08:05PM +0200, Andre Massing wrote:
> -----BEGIN PGP SIGNED MESSAGE-----
> Hash: SHA1
>
> Hi there!
>
> Looking at both classes, I was wondering whether they could not be
> unified. The MeshEntityIterator already provides a random access
> operator, so all we have to save/fill  is an additional
> std::vector<uint>  in case of an subiteration. And to add constructor
>
> MeshIterator(const MeshFunction<unsigned int>& labels, uint label);
> which I think is rather intuitive (when documented right)
>
> Opinions? I am working on unifying/merging stuff from Johans
> DistanceToBoundary branch right now, so I could do this along the way,
> if there aren't any objections...
>
> Andre
> -----BEGIN PGP SIGNATURE-----
> Version: GnuPG v2.0.18 (GNU/Linux)
> Comment: Using GnuPG with Mozilla - http://enigmail.mozdev.org/
>
> iQEcBAEBAgAGBQJOUoz1AAoJEA79ggnbq9dmsg4IAKtYqcewATWMiBaGIvmhdeLc
> G70vf1CJYtWx9aoxQAMcnsL6cEScXqGVHzxPUJgvnl2vrWp7xObhs7fXbKQx6CeY
> cKDGBfnK+ZK0nam2rpSLzzVJ1IufTSBMr1PseewHdbbV0YP2/TiMZQAzot3LoUOe
> r4sxeT1+xPIidUlcepceojZ1/oq3qylwd5jrS+fpQQKisURDdn1zK+iSIrZXd2Wt
> 6/xjBpiIjbtq0ZbY/h87VVF2AhGXaSvY7Ielxu2sfE3xRZ8WzHInP4N4aLxYblm3
> mPSNahas+8bSaV3lTZJz44N1SXBN5bvxf1mVv9puAHnluzpdo4y6D9aw4OWWgTA=
> =4xF+
> -----END PGP SIGNATURE-----
>
> _______________________________________________
> Mailing list: https://launchpad.net/~dolfin
> Post to     : dolfin@xxxxxxxxxxxxxxxxxxx
> Unsubscribe : https://launchpad.net/~dolfin
> More help   : https://help.launchpad.net/ListHelp


Follow ups

References