On Thu, Jul 21, 2005 at 12:58:19PM -0400, Carlos M. Gutierrez wrote:
> private:
> OptionalExceptions& assoc_;
> bool exceptions_were_enabled_;
>
> // Hidden assignment operator and copy ctor, because we should not
> // be copied.
> NoExceptions(const NoExceptions&);
> NoExceptions& operator=(const NoExceptions&) { }
>
> As defined in that last line, operator= should return a NoExceptions
> reference but it does not. I am not an expert programmer, should the "{
> }" just become ";" ?
Yes.
The latest SVN version has this "fixed", but incorrectly in my
opinion. It currently just returns *this, whereas I don't think
operator=() should be defined at all, just declared.
- Chris
