Newsgroups: php.internals
Path: news.php.net
Xref: news.php.net php.internals:70918
Mailing-List: contact internals-help@lists.php.net; run by ezmlm
Received-SPF: pass (pb1.pair.com: domain gmail.com designates 209.85.215.52 as permitted sender)
MIME-Version: 1.0
Sender: yohgaki@gmail.com
In-Reply-To: <CAGa2bXZAGf7-ijr3CZaJXKCUweSe9HOcgauQoWLCOkpTusyOOA@mail.gmail.com>
References: <CAGa2bXarJDxdS0NPhFaC+20WK29bDknvcJnyWVcysXWK4o8A9A@mail.gmail.com>
 <CAGa2bXa00M4qjMM7X6bcTvcs+5-DSg2k2jpn1xH6CaJ6H++d4w@mail.gmail.com>
 <CAHMUw2Gb7GOBx=dUW4i7Mspp5dkw-EMBwUBK9TOFV9zmd2BdUg@mail.gmail.com>
 <CAGa2bXY=62y2Z_XUFuOn9gVYnmcW_qcEAO3xrTUaS2FayHm1rw@mail.gmail.com>
 <CAGa2bXayj44fmR==_2GXos-tK0ps0xNUgEQVqL=swGp1ZJkzEQ@mail.gmail.com> <CAGa2bXZAGf7-ijr3CZaJXKCUweSe9HOcgauQoWLCOkpTusyOOA@mail.gmail.com>
Date: Mon, 30 Dec 2013 19:36:26 +0900
Message-ID: <CAGa2bXYZfLM83Xkx5m5kqWBXxLX-BhV6hJcWzW1LRznH3yW0VA@mail.gmail.com>
To: Tjerk Meesters <tjerk.meesters@gmail.com>
Cc: "internals@lists.php.net" <internals@lists.php.net>
Content-Type: multipart/alternative; boundary=001a11c2ba7e7f5a3f04eebe06c4
Subject: Re: [PHP-DEV] Re: GMP and empty()
From: yohgaki@ohgaki.net (Yasuo Ohgaki)

--001a11c2ba7e7f5a3f04eebe06c4
Content-Type: text/plain; charset=UTF-8

Hi all,

On Mon, Dec 30, 2013 at 7:23 PM, Yasuo Ohgaki <yohgaki@ohgaki.net> wrote:

> On Mon, Dec 30, 2013 at 7:02 PM, Yasuo Ohgaki <yohgaki@ohgaki.net> wrote:
>
>> Apparently, we need type check and/or float support in GMP.
>>
>
> We need error instead of type check.
> Possible patch is
>
> [yohgaki@dev php-src]$ git diff ext/gmp/
> diff --git a/ext/gmp/gmp.c b/ext/gmp/gmp.c
> index 8835f05..a64e486 100644
> --- a/ext/gmp/gmp.c
> +++ b/ext/gmp/gmp.c
> @@ -1036,6 +1036,7 @@ ZEND_FUNCTION(gmp_init)
>
>         INIT_GMP_RETVAL(gmpnumber);
>         if (convert_to_gmp(gmpnumber, number_arg, base TSRMLS_CC) ==
> FAILURE) {
> +               php_error_docref(NULL TSRMLS_CC, E_WARNING, "Bad number
> supplied");
>                 zval_dtor(return_value);
>                 RETURN_FALSE;
>         }
>
> Any comments?
>

I would like to have float support rather than this dirty patch.
We may implement GMP float support. It would be easier with object.

Regards,

--
Yasuo Ohgaki
yohgaki@ohgaki.net

--001a11c2ba7e7f5a3f04eebe06c4--