David Faure faure at
Tue Jun 8 13:03:07 BST 2004

On Tuesday 08 June 2004 02:07, Benjamin Meyer wrote:
> Also Added All to the enum.
Is that a good idea, in terms of extensibility? If you add another feature
to the method later, you will be faced with two bad possibilities:
* adding the new feature to All, which would change existing behavior,
* or having a "All" that doesn't really mean "All", since one feature will
be missing...

Apart from that, the patch looks ok to me now.

David Faure, faure at, sponsored by Trolltech to work on KDE,
Konqueror (, and KOffice (

More information about the kde-core-devel mailing list