Description:
------------
The functions trader_ht_phasor and trader_ht_sine expect two parameters, the documentation lists one.  The second parameter for both is never populated with the expected data from the C TA-Lib functions.

As noted in Doc Bug #72882, the second required parameter is not listed in the documentation, but is required to execute.

In both cases, the data that should be injected into the second parameter is instead included in the return value.  The return value is a two-dimensional array instead of a single-dimensional array and a modified by-reference parameter, which would be expected given the second parameter is required.

This can be verified at http://svn.php.net/viewvc/pecl/trader/trunk/functions/trader_ht_phasor.c?view=markup for trader_ht_phasor and http://svn.php.net/viewvc/pecl/trader/trunk/functions/trader_ht_sine.c?view=markup for trader_ht_sine.

Using PHP 7.0.8 with Trader 0.4.0 on Windows Server 2012, IIS 8.5.

Test script:
---------------
<?php
$a1 = [];
$o1 = [];
for($i = 0; $i < 128; $i++){
    $a1[] = rand(1, 100);
}
$r1 = trader_ht_phasor($a1, $o1);
$r2 = trader_ht_phasor($a1, $a1);
print ($o1 == [] ? 'true' : 'false') . PHP_EOL;
print ($r1 == $r2 ? 'true' : 'false') . PHP_EOL;

//This prints the same when trader_ht_sine replaces trader_ht_phasor.


Expected result:
----------------
If the second parameter was being filled, the first print should be 'false'.
If the second parameter was an input parameter (it is not), the second print should be 'false'.  (Consider the second a sanity check.)

Actual result:
--------------
true
true