Re: [boost] Proposed changes to operators.hpp

From: Daniel Frey (d.frey_at_[hidden])
Date: 2002-07-08 17:13:52

On Mon, 2002-07-08 at 22:46, Daniel Frey wrote:
> On Mon, 2002-07-08 at 21:48, David Abrahams wrote:
> > 
> I uploaded the file, it's called boost_1_28_0.patch.tgz, not the best
> name but the file can be removed once it is applied or rejected :-/

Some more notes/questions:

- The patch concentrates on operators.hpp, but there are other
candidates for NRVO, e.g. next/prior. I suggest that we check the result
for operators.hpp first and add compiler-configs for other compilers
that have the NRVO. If this works well for the next release of boost, we
can start to apply the NRVO-scheme to other libs. OK?

- I also fixed some inconsistencies in the documentation of
operators.hpp, but all very basic stuff that shouldn't be controversial.

- The implementation of other operators use 'x' and 'y' for the
parameters, sometimes they are swapped, sometimes not. This seems
inconsistent to me. Is there a common scheme I don't see or should we
change the code to use lhs/rhs somewhere in the future?

Regards, Daniel