|
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