Re: [PHP-DEV] Improved logarithm of base 2|10 of standard math function

[email protected] (Marc Bennewitz)
Newsgroups php.internals
Message-ID <[email protected]>
Sara,

shouldn't it be merged into the branches PHP-5.4, PHP-5.5 as well as
PHP-5.6 and shouldn't there an info in the NEWS file? It's currently
only merged to master.

Marc

On 14.08.2014 17:29, Sara Golemon wrote:
> On Thu, Aug 14, 2014 at 2:44 AM, Marc Bennewitz <[email protected]> wrote:
>> forgotten the link:
>> https://github.com/php/php-src/pull/658
>>
>>
>> On 14.08.2014 11:43, Marc Bennewitz wrote:
>>>
>>> Hi internals,
>>>
>>> I have created a PR to improve the base 2 and base 10 of the standard
>>> math function "log" to reduce rounding errors.
>>>
>>> Internally on log(x, 2) the native C function log2(x) and on log(x, 10)
>>> the native C function log10(x) will be called.
>>>
>>> The PR is open since april without any comments!
>>> Please please please someone there to merge this.
>>>
>>> PS:
>>> Because of after this PR the following will be the same in PHP log10(x)
>>> === log10(x) the function log10 is needless and could be
>>> deprecated/removed. I will create an RFC if this PR was merged.
>>>
>>
> This seems entirely reasonable and would be covered by existing unit
> tests.   No user-impacting effect (beyond being more precise/correct
> output).  I'll push this after my Dr appointment today if nobody has
> beaten me to it.
> 
> -Sara
>
lmpx.com only provides a reader for public news (NNTP) servers. It is not affiliated with the servers or forums shown here and is not responsible for the content of articles, which is written by their respective authors.