KStyle changes
    David Faure 
    faure at kde.org
       
    Wed May  9 17:53:51 BST 2007
    
    
  
On Wednesday 09 May 2007, Alex Merry wrote:
>          /// Represent as an int to store as a property
> -        operator int() const
> -        {
> -            return int(role) | int(mode);
> -        }
> +        operator int() const;
This could be replaced with a proper Q_DECLARE_FLAGS and Q_DECLARE_OPERATORS_FOR_FLAGS, I think?
Overall, I wonder if that ColorMode thing (just a glorified enum, right) is worth a Private class.
-- 
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
mailing list