void ALE::move(...) checks the dimension incorrectly

[Charles]

In ALE.cpp

//-----------------------------------------------------------------------------
void ALE::move(Mesh& mesh, const Function& displacement)
{
  // Check dimensions
  dolfin_assert(displacement.function_space()->element());
  const FiniteElement& element = *displacement.function_space()->element();
  const std::size_t gdim = mesh.geometry().dim();
  if (!((element.value_rank() == 0 && gdim == 0) ||
        (element.value_rank() == 1 && gdim == element.value_dimension(0))))
  {
    dolfin_error("ALE.cpp",
                 "move mesh using mesh smoothing",
                 "Illegal value dimension of displacement function");
  }

"This is a bug. The first line should check gdim == 1..."
https://answers.launchpad.net/dolfin/+question/221688

Thank you,

Charles