Hi Andrey

2009/5/20 Andrey Hristov <p...@hristov.com>:
>  Hi,
>
>
> -               if (rc == (my_ulonglong)-1) {
> +               if (rc == (long)-1) {
>
> This is not correct. Maybe you have 64bit box, but 32bit machines are still
> out there and this will be bad. my_ulonglong is 8 byte and -1, will be the
> max value 2^64-1, long is 4 byte and even when it is 8 byte, (long) -1 will
> be -1, not 2^64-1, so I think it would be better to revert these changes!
> Especially when the come after RC2
>
> This is also wrong:
> -       row_count = mysql_stmt_affected_rows(S->stmt);
> -       if (row_count != (my_ulonglong)-1) {
> +       row_count = (long) mysql_stmt_affected_rows(S->stmt);
> +       if (row_count != (long)-1) {
>
> There are a few places where you have done the same mistake and they should
> be reverted.

I will review and commit fixes tonight or tomorrow, thanks for the
notice. btw. mysqlnd is emitting many warnings too, if non of you
mysql'ers wants to deal with it, I'll have a look at it aswell.

>
> Andrey
>
> --
> PHP CVS Mailing List (http://www.php.net/)
> To unsubscribe, visit: http://www.php.net/unsub.php
>
>



-- 
Kalle Sommer Nielsen
ka...@php.net

--
PHP CVS Mailing List (http://www.php.net/)
To unsubscribe, visit: http://www.php.net/unsub.php

Reply via email to