faure at kde.org
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
Apart from that, the patch looks ok to me now.
David Faure, faure at kde.org, sponsored by Trolltech to work on KDE,
Konqueror (http://www.konqueror.org), and KOffice (http://www.koffice.org).
More information about the kde-core-devel