viennagrid / viennagrid-dev

Developer repository for ViennaGrid. Visit http://viennagrid.sourceforge.net/ for the latest releases.
MIT License
15 stars 7 forks source link

Coboundary range construction from elements #40

Closed karlrupp closed 10 years ago

karlrupp commented 10 years ago

Right now one has to write

for (VertexIterator vit = ...)
{
   EdgeOnVertexContainer edges_on_vertex(domain, vit.handle());
   ....
}

I find it non-intuitive to call .handle() on the iterator. Instead,

for (VertexIterator vit = ...)
{
   EdgeOnVertexContainer edges_on_vertex(domain, *vit);
   ....
}

should also work, which is not the case at the moment.

karlrupp commented 10 years ago

Problems stems from assignment troubles in coboundary_range_wrapper and thus invalidates the correct dereference_handle() to be called

FlorianRudolf commented 10 years ago

done in 3f7c8f3