dolfin team mailing list archive

[Anders Logg <logg@xxxxxxxxx>]

On Mon, Apr 21, 2008 at 12:32:58PM +0100, Garth N. Wells wrote:
> 
> 
> Ola Skavhaug wrote:
> > According to http://developer.kde.org/~wheeler/cpp-pitfalls.html:
> > 
> > Note: While it is not required to use the virtual keyword in our subclass
> > implementations (since if the base class implementation is virtual all
> > subclass implementations will be virtual) it is still good style to do so.
> > 
> > I don't think DOLFIN does this. 
> 
> No, it doesn't.
> 
> Should we put adding the virtual keyword to
> > virtual methods on the todo-list? After all, the virtual keyword adds
> > information to the user.

Sounds good to me.

-- 
Anders