dolfin team mailing list archive

[Anders Logg <logg@xxxxxxxxx>]

On Wed, Oct 07, 2009 at 09:03:51PM +0100, Garth N. Wells wrote:
> Is there a reason why Data::update doesn't invalidate the things that
> are not updated? We presently have
>
>    void Data::update(const ufc::cell& ufc_cell, const double* x)
>    {
>      _ufc_cell = &ufc_cell;
>      this->x = x;
>    }
>
> but should this be
>
>    void Data::update(const ufc::cell& ufc_cell, const double* x)
>    {
>      _dolfin_cell = 0;
>      _ufc_cell = &ufc_cell;
>      _facet = -1;
>      this->x = x;
>    }

Yes, I guess it wouldn't hurt. The simplest option is to add a call to
invalidate first. I'll add it.

--
Anders

Attachment: signature.asc
Description: Digital signature