Andre Poenitz <[EMAIL PROTECTED]> writes:
| On Mon, Oct 13, 2003 at 12:10:06AM +0200, Lars Gullik Bjønnes wrote:
>> I even think boost::optional should be used for the Cached vars type,
>> then you can easily know if the cached var is set or not.
>
| We'd never, ever need to know whether this is set or not. You can put in
| arbitrary values in this variable at any point of time _except_ at the
| end of 'metrics'.
The you need to be pretty sure that is it never used before metrics is
called, can you vouch for that? Use of optional would enforce it.
| I'd really like to suggest promoting the 'KISS' principle to the top of
| the 'nice to have' stuff.
Sure. To me use of boost::optional actually makes code KISSier. Why?
It gets more explicit, harder to mis-use etc.
--
Lgb
