[prev in list] [next in list] [prev in thread] [next in thread] 

List:       kde-core-devel
Subject:    Re: KConfig XT: kcalc.kcfg, an example
From:       Nadeem Hasan <nhasan () nadmm ! com>
Date:       2003-10-02 15:22:57
[Download RAW message or body]

Quoting Waldo Bastian <bastian@kde.org>:

> > The "QColor defaultButtonColor = " in <code></code> above looks redundant
> > to me. This line can be constructed using the information already
> available
> > elsewhere. Pardon me if I missed something obvious.
> 
> Yeah, if you have multiple defaults that are all the same they could be 
> factored out automatically, in theory. I'm not sure if that's worth the 
> effort though, this is just as easy IMHO.

I think ossi has a good point that the code may be more complex than just
the example you provided. Although, it can be done by wrapping up that code
in a function by the developer, but as you said it may not be worth the 
overall effort.
--
Nadeem Hasan
nhasan@nadmm.com
http://www.nadmm.com/

---------------------------------------------------------
This mail sent through WebMail: http://webmail.nadmm.com/
[prev in list] [next in list] [prev in thread] [next in thread]