From: Tor Didriksen
Date: January 13 2011 3:08pm
Subject: Re: bzr commit into mysql-trunk branch (tor.didriksen:3498) Bug#59241
List-Archive: http://lists.mysql.com/commits/128667
Message-Id: <4D2F156B.6010907@oracle.com>
MIME-Version: 1.0
Content-Type: text/plain; charset=ISO-8859-1; format=flowed
Content-Transfer-Encoding: 7bit

On 2011-01-13 15:45, Olav Sandstaa wrote:
> Hi Tor,
>
> The patch looks good. Thanks for adding copy constructor and 
> assignment operators and a google test for this.
>
> One minor comments that you should feel free to ignore:
>
> * In Item_func_int_div::val_int(): We have now five my_decimal 
> objects, I think tree of these are just needed for having as input 
> arguments to the various calls and are never used after the calls. So 
> the following my_decimal objects:
>
>      my_decimal value0, value1, tmp;
>
>    could probably be replaced by just on my_decimal object?

indeed.

-- didrik