$include_dir="/home/hyper-archives/boost/include"; include("$include_dir/msg-header.inc") ?>
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