← Back to team overview

dolfin team mailing list archive

Re: IndexSet

 

On Wed, Feb 23, 2011 at 04:11:50PM +0000, Garth N. Wells wrote:
>
>
> On 23/02/11 16:07, Anders Logg wrote:
> > B1;2600;0cOn Wed, Feb 23, 2011 at 02:41:50PM +0000, Garth N. Wells wrote:
> >> It looks to me like IndexSet should have some assertions added to check
> >> for out of bounds indices.
> >
> > I'm not so sure. It uses std::vector so I would think it's the
> > responsibility of std::vector to assert indices.
> >
>
> std::vector::operator[] does not perform bounds checking.
> std::vector::at() does.

ok. I can add asserts but let's wait until after the release.

Are we ready to release once the buildbot is green? The linux64-exp
buildbot just failed but I assume that is the long-standing
intermittent parallel test failing.

--
Anders



References