Boost logo

Boost :

From: Graham Bennett (graham-boost_at_[hidden])
Date: 2005-09-11 15:09:22


On Tue, Sep 06, 2005 at 09:37:43PM +0200, David Abrahams wrote:

[ snip ]

> > + // fix for iterator
>
> I think you should remove that comment. Either the compiler is broken
> in this respect or it isn't; it has nothing to do with the iterator
> library.

[ snip ]

> I think you need a comment here explaining that ::type() is needed for
> Sun CC, or a maintainer might be tempted to remove it.

[ snip ]

> Other than that, please feel free to check it in!

Thanks for the feedback, attached is a better-commented patch. I'd be
grateful if someone with CVS access could check it in, if it seems
reasonable.

cheers,

Graham.

-- 
Graham Bennett



Boost list run by bdawes at acm.org, gregod at cs.rpi.edu, cpdaniel at pacbell.net, john at johnmaddock.co.uk