« first day (3592 days earlier)      last day (1363 days later) » 

2:56 AM
stream_socket_client be called by call_user_func_array with wrong param and cau ・ Reproducible crash ・ #79979
 
 
2 hours later…
4:47 AM
Trying to build a PHP extension using CMake. I'm doing pretty well, but I'm not sure what to do here:
  "_zend_vm_set_opcode_handler", referenced from:
      _zm_activate_ddtrace in ddtrace.c.o
I get a bunch of missing symbols like this.
We don't have a libphp.so to link against, so I'm not sure how to handle it.
 
5:06 AM
Only having this problem on Mac; works on Linux.
 
5:18 AM
require_once can include a file twice via symlink loops ・ *Directory/Filesystem functions ・ #79980
 
Well, on Linux it doesn't actually work still:
> PHP Startup: Invalid library (maybe not a PHP library) 'ddtrace.so' in Unknown on line 0
Ah, was missing -DHAVE_CONFIG_H
 
 
3 hours later…
8:19 AM
stream_socket_server cause crash with call_user_func_array ・ Reproducible crash ・ #79981
stream_socket_client cause crash with call_user_func_array ・ Reproducible crash ・ #79982
 
 
5 hours later…
1:23 PM
@beberlei @Derick perhaps your RFC could highlight some of the logical fallacies that arrived at @@ i.e. claiming that <</>> will be confused with either bit shifting or generics, but somehow @@ wouldn't be confused with error silencing.
 
Morning
 
2:00 PM
I get a value from a json string using json_decode and then proceed to do some maths. But for whatever goddamn reason, php wants to round up my numbers. Printing the actual value works fine, multiplying it with 100 does not. I also tried using floatval, it didn't change anything. What is the reason and how can I fix it?
 
2:16 PM
@LeviMorrison any reason you want to port ddtrace to cmake? better CLiON support?
 
about my problem, i rebooted the server and now it works.
 
 
1 hour later…
3:26 PM
@beberlei Yes, but also that we want to expand into C++, have unit tests at the extension level (everything is run through PHP right now, want some C/C++ tests), and our existing Makefile is an overloaded mess.
I think high-quality CMake is better than high-quality autoconf. Of course both can be terrible.
 
Well we kinda have a custom thing for PHP which is loosely based of autoconf from my understanding
 
4:13 PM
Needed to add -undefined dynamic_lookup for Apple platforms.
Not sure why I don't have to do this for regular phpize builds, though.
NOOOO! I lost my progress with an errant rm -fr *
 
git reflog?
Oh no you did an actual rm :(
 
The second time you'll write it better :-) (still sucks, though)
 
4:37 PM
IntelliJ local history... the one true saviour
 
hello, I have some memory leaks - can someone tell me more about?
I'm testing because of mysqli_ but, I wish to know more - for good tests knowledge is basic need
 
Was @{Attr} considered for attribute syntax? Would it be possible? Would there be any bc breaks?
 
4:54 PM
Everything was considered, including 💩Attribute💩
and of course 💩💩Attribute
 
@gharlan It would be possible but I don't think more options is what we need. Also, { braces would look more confusing as class/function bodies already use {.
 
@gharlan @{Attr} was not really considered, mostly things sort of related to existing syntaxes are
 
@{
    Foo,
    Bar,
}
class Foo {

}
Not really very appealing (to me)
 
@beberlei Were the old votes saved before the vote was removed or are they lost to time?
 
I have them as data, benjamin as screenshot, I believe.
 
4:59 PM
Would be interested in knowing what % of people changed their mind between that one and whatever comes next
 
for me @{} has the advantage that it does not look like an array (of function calls) and that there are no bc breaks (if I'm correct).
 
it looks like a silenced scope
 
@Derick This is true, at least.
 
@gharlan Yes I believe there would be no BC breaks. Although we have many other options without BC break.
But the minimal BC break of each seems to be of lesser concern.
 
There's probably not a great deal of appetite for new options unless someone comes up with something groundbreaking. Voter fatigue is setting in pretty heavily at this point
IIRC pretty much the only style that hasn't gone through some form of vote would be a keyword such as :

__attr(JitOff)
function foo() { }
 
5:40 PM
@MarkR let's go full C style: __attribute__((jit_off))
 
Let's use the ugliest syntax possible so everybody is unhappy.
6
 
<?@@:#[__attr__(JitOff)]
 
@IluTov So @@ then? ;-)
 
Aaah, I can't put the typed class constants RFC to under discussions status because of frickin' Bison conflicts
 
6:04 PM
bison conflicts are sort of a big thing
 
@IluTov So everyone is equally unhappy.
 
@moliata Yeah sorry not in the mood for another headache tonight :P
 
@IluTov Had those myself these past 2 nights
Can't figure out where the problem is :/
 
6:27 PM
Well, as mentioned it conflicts with trait adaptations, but I can't understand how exactly.
 
7:01 PM
@moliata migraine?
 
7:18 PM
@IluTov iirc it was a lalr state merging issue
not a "real" conflict
 
regarding @Crell's email and curating user comments section in the manual... this is a comment I used to add a small contribution a long time ago: php.net/manual/en/curl.examples-basic.php#114417, but I'm hesitant to delete it in case I'm missing something in my change: svn.php.net/viewvc/phpdoc/en/trunk/reference/curl/…
 
@NikiC Oh Nikita ^^ In case you're bored: github.com/php/php-src/pull/5994 I have no clue if there's a cleaner solution that doesn't require tracking the offsets.
In case it isn't obvious, we forgot that sometimes compilation of vars/dims is delayed, and JMP_NULL would be emitted out of order.
 
7:57 PM
@Tiffany The user note can be deleted. There's no need to keep around a suggestion to add curl_error() to check for errors to the example, when the example does include curl_error() to check for errors. If you're not confident that the example is good, useful, adequate, or whatever. (aside: it's fine) that's completely orthogonal to whether that user note should be kept around.
P.S. Thanks for integrating the note into the docs proper. :-)
 
alright, thanks, deleted it
 
 
1 hour later…
9:06 PM
openssl_encrypt / openssl_decrypt fail with OCB mode ・ OpenSSL related ・ #79983
 
 
1 hour later…
user12640521
10:11 PM
Using the following code on php 8.0.0b1 I get an error.
 
user12640521
file_put_contents(
    filename: __DIR__ . '/_.txt',
    data: 'Hello world!',
);
 
user12640521
Fatal error: Uncaught Error: Unknown named parameter $data
 
user12640521
php -v
HP 8.0.0beta1 (cli) (built: Aug  6 2020 12:17:05) ( ZTS )
Copyright (c) The PHP Group
Zend Engine v4.0.0-dev, Copyright (c) Zend Technologies
    with Zend OPcache v8.0.0beta1, Copyright (c), by Zend Technologies
 
user12640521
I checked on in the docs and in the source $data is defined
 
@JellyLegend The param name seems to be correct. github.com/php/php-src/blob/… Feel free to create a bug report. bugs.php.net
Can you try it with content:?
 
10:19 PM
@JellyLegend The docs are lying
We are currently trying to normalize the names to make them consistent across all (at least most) PHP functions before we update the docs
me needs to wait a 20min CI run before knowing if my changes are finally correct with optimizations enabled
 
10:35 PM
go watch anime
 
Already did that
Man this is very frustrating to have something only segfault on CI in optimized builds
 
user12640521
@IluTov ok, content: works, thank you!
 
user12640521
@IluTov now I also know where to look :)
 
user12640521
11:02 PM
# One more question I cannot do this:
$1 = 'something';
echo $1;
echo "$1";


# But I can do this?

${1} = 'something';
echo ${1};
echo "${1}";
 
11:15 PM
Braces do something fancy
@Girgias I used to have to deal with something similar when I was deploying a mobile app to the Apple app store, it was insanity
 
@Tiffany also now that I'm trying against Azure to see if it gives me better info, it's GCC is shouting at me, when it does not locally nor on Travis
 
The most frustrating part was if I forgot to increment the sub-sub-version (whatever the word for that is), and began the process of deploying... I wouldn't be notified until like 10-20 minutes into the process that "oh, this version number already exists"
 
:|
 
WHY DIDN'T YOU TELL ME THAT AT THE START, XCODE
 
@JellyLegend You're better off searching for the corresponding stub files: github.com/php/php-src/blob/… It's easier to digest, and these files are the single source of truth by which arginfos are generated :)
 
11:26 PM
Worst part I don't know how useful what I'm trying to do is
But because it's ABI break that need to land before 8.0 gets out...
@MateKocsis as you're here, what's your opinion on something like this: github.com/Girgias/php-src/pull/4 ?
 

« first day (3592 days earlier)      last day (1363 days later) »