Miguel Ángel <[email protected]> writes:

>>     autosprintf& operator = (autosprintf& src)
>
> First of all, const correctness.
> This would be fine with a move reference (C+11) autosprintf&& src, but a
> copy constructor must not modify the copied object.

Thanks for the explanation.  Then, how about:

autosprintf& operator = (autosprintf src)

or just providing autosprintf::swap instead of the assignment operator?

Sorry for grumbling, but I like simplicity.  At least, "autosprintf::"
prefix can be removed from your patch.

Regards,
-- 
Daiki Ueno

Reply via email to