DirichletBC copy constructor broken in Python interface

[Patrick Farrell]

According to the docstring for DirichletBC.__init__, there should be a copy constructor for DirichletBC objects. However, the following code:

from dolfin import *
mesh = UnitSquare(4, 4)
V = FunctionSpace(mesh, "CG", 1)
bc = DirichletBC(V, 1.0, "on_boundary")
newbc = DirichletBC(bc)

gives an exception:

Traceback (most recent call last):
  File "/tmp/py", line 5, in <module>
    newbc = DirichletBC(bc)
  File "/usr/lib/python2.7/dist-packages/dolfin/fem/bcs.py", line 89, in __init__
    if isinstance(args[1], cpp.Expression):
IndexError: tuple index out of range

instead of copying the boundary condition as expected.

[Johan Hake (johan-hake)]

Thanks for reporting. Should be fixed now.

Johan

On Monday January 9 2012 11:19:53 Patrick Farrell wrote:
> Public bug reported:
>
> According to the docstring for DirichletBC.__init__, there should be a
> copy constructor for DirichletBC objects. However, the following code:
>
> from dolfin import *
> mesh = UnitSquare(4, 4)
> V = FunctionSpace(mesh, "CG", 1)
> bc = DirichletBC(V, 1.0, "on_boundary")
> newbc = DirichletBC(bc)
>
> gives an exception:
>
> Traceback (most recent call last):
> File "/tmp/py", line 5, in <module>
> newbc = DirichletBC(bc)
> File "/usr/lib/python2.7/dist-packages/dolfin/fem/bcs.py", line 89, in
> __init__ if isinstance(args[1], cpp.Expression):
> IndexError: tuple index out of range
>
> instead of copying the boundary condition as expected.
>
> ** Affects: dolfin
> Importance: Undecided
> Status: New