« first day (4547 days earlier)      last day (386 days later) » 

12:04 AM
@kelunik thank you so much. I appreciate it
 
12:45 AM
@FĆ©lixAdriyelGagnon-Grenier I'd throw in Web Components, which is lovely but people still keep trying to add compilation steps to.
 
 
11 hours later…
11:42 AM
Morning!
 
12:02 PM
o/
 
\o
 
 
1 hour later…
1:24 PM
@ramsey 8.1.18RC1 is your, right ?
 
1:36 PM
And it is late, @ramsey !
 
 
1 hour later…
2:40 PM
> vs ketchup
 
A person is trying to get a package published to my PECL. They tried contacting the mailing list but received silence. Can anyone help? github.com/open-telemetry/opentelemetry-php/issues/846
 
@Tiffany Is anyone in touch with CMB? He normally approves accounts...
Though this also kind of shows how thin on the ground the maintainers have become...
 
Speaking on RC releases, github.com/php/php-src/commit/… the release date looks incorrect?
 
2:57 PM
I can help with pecl stuff.
@Tiffany Done.
 
Cheers!
 
Does anyone know where the CSS is located for the docs? web-php?
 
I think it's shared... some sort of static thing?
erm, no
~/dev/php/php-web-php$ find . | grep css
./styles/i-love-markdown.css
./styles/mirror.css
./styles/home.css
 
Right, I think our current CSS is overwriting the style for the KBD tag
 
is that the actual name of the tag, as it has no mention in the style sheets
 
$ grep -ri kbd
derick@gargleblaster:~/dev/php/php-web-php/styles$
not mentioned...
 
Huuum, might just be an issue while generating a local doc thing
 
3:23 PM
Oh I thought the styling was a default browser thing but maybe we need to add it to our CSS
 
Yeah
It just preformats it without styles
I don't know how to link within the page from mobile but if you ctrl-f "custom styles" it gives an example
These are the styles they have for the thing at the beginning
kbd {
    background-color: #eee;
    border-radius: 3px;
    border: 1px solid #b4b4b4;
    box-shadow: 0 1px 1px rgba(0, 0, 0, 0.2), 0 2px 0 0 rgba(255, 255, 255, 0.7) inset;
    color: #333;
    display: inline-block;
    font-size: 0.85em;
    font-weight: 700;
    line-height: 1;
    padding: 2px 4px;
    white-space: nowrap;
}
 
Well first need to get the PhD thing merged and then can look at the CSS :D
 
3:42 PM
šŸ‘
 
@MarkR out of interest, did you talk to Jordi about making an implementation for externals.io/message/113545#113976 to be used by composer?
 
3:55 PM
Gosh, if only PHP had function autoloading. It could really make the language be less class centric, which would probably make it easier (eventually) to add functional features to the language, as people get used to using 'just' functions.
gazes off to the horizon, wistfully
 
How weird of you saying this, I have something just for you: github.com/Girgias/core-autoloading-rfc/blob/master/… :D
3
(i.e. please have a look at the draft RFC before we announce it on internals)
 
What fortuitous timing! I shall read it immediately!
 
4:10 PM
@Girgias out of curiosity, why did you go with the separate autoloader mechanics rather than a unified system across classes and functions?
 
@ircmaxell What do you mean? One autoloader for classes and functions?
 
@ircmaxell there's a faq item on that - github.com/Girgias/core-autoloading-rfc/blob/master/…
 
What I was proposing ~8 years ago (sidenote, holy crap, that was 8 years ago???) was to have a unified system and indicate via a flag argument whether it was a class, function, or constant. That way the system was extensible and would gracefully degrade to older versions wrather than copying: wiki.php.net/rfc/function_autoloading
 
Main concern is if a new autoading "thing" needs a different signature
Also, I personally find that sort of API kinda crap :p
 
@Danack I don't fully buy that. What I mean, is if you can register autoloaders and say what type they are (Any, class, function), then the performance argument is a bit moot isn't it, since classes would only get called by classes?
Fair enough, though copy/pasting the full api set for each "autoload type" also doesn't seem clean
 
4:15 PM
something something duplication cheaper wrong abstraction.
 
@Danack in abstract totally get that. In this case I'm not so sure that applies. What counter-example would you have to show it's a wrong abstraction? I get "sure a new thing needs a different signature", but shouldn't there be some ideas of what that looks like?
 
Are you asking about the suggested api from your RFC, or "if you can register autoloaders and say what type they are" which appears to me to be different?
 
More the second than the first, though it is more seeing so much symmetry while having duplication and parallel implementations feels off
 
> More the second than the first
Okay, I'll try to answer, but it is difficult to answer vague questions....
 
Not to mention the under the hood implementation of auto loading was so fubar that I found it needed to be ripped out and redone from scratch
 
4:23 PM
@ircmaxell Well I've kinda wrote it from scratch again, also found long standing bugs with trampolines (that was fun, not)
 
I have strong negative feelings against this type of API autoload_register_thing($type, $callback) where the signature required for $callback depends on the value of type. Although they are not absolutely wrong, they are far easier to get confused about. To me, there needs to be a strong reason not to use separate autoload_register_class + autoload_register_function.
 
why does the signature depend on the type?
@Girgias nice :)
 
@ircmaxell Well, there's a faq item on that,. But even for just:
function my_class_loader(string $class_name) {}
function my_function_loader(string $function_name) {}
Although those two callbacks appear to have the same signature, that's only due to the lack of expressiveness in the PHP type system. Adding in the PHPStan annotations that will be used.
/**
 * @param class-string $className
 */
function my_class_loader(string $class_name) {}

/**
 * @param function-string $className
 */
function my_function_loader(string $function_name) {}
And it's revealed that you would need to match up 'class_loader_type' with a 'class-string' callback, and 'function_loader_type' with a 'function-string' callback. aka, the code is possible to get wrong.
/away to pubbery.
 
4:38 PM
@Danack easily resolved by naming it "symbol_name" instead of class/function_name.
Though, even still I don't think this gets to the root of my issue
since I'm not actually against having autoload_register_class/autoload_register_function. It's more the total severability of the two systems that feels weird
as an example, another approach could be to use interfaces to segregate the two: interface Autoloader {} and then have interface classAutoloader extends Autoloader { public function autoload_class($string); } and interface functionAutoloader extends Autoloader { public function autoload_function($string); }
that way, you can have a single register function which determines internally which queue to place it on based upon the interface. And at that point autoload_register_class would simply proxy to register using a class: class CallableClassAutoloader implements ClassAutoloader { private $callable; public function autoload_class($string) { return ($this->callable)($string); } } then: function autoload_register_class($callable) { autoload_register(new CallableClassAutoloader($callable)); }
:56154225 isn't that far easier to solve by just supporting the syntax directly: `autoload("GenericStack<int>")`? And I say that because otherwise for non-trivial examples it'll become *really* hard to infer type. Take an example:

class GenericStack<T<U>, V> {}
class Foo<T> {}
class Bar<T> {}

$a = new GenericStack<Foo<Bar<int>>, Bar<float>>();

In that case, what would the call signature be? `autoload("GenericStack", "Foo<Bar<int>>", "Bar<float>")` or would it be `autoload("GenericStack", "Foo", "Bar<int>", "Bar<float>")` ? In either case you still need to parse to get inner types if you 
 
5:07 PM
@Girgias I'm wondering if the return type for the 'list' functions should be iterable, instead of array, even if you actually return an array. From thinking about this for half a minute it seems to be a good thing to make as little guarantees as possible.
 
@Danack I had it working, there's probably a PR sitting on the php-src repo somewhere, the performance was around 5% in a purely-autoloading test, I didn't move ahead with the project because Girgias looked to be making a much broader element including function autoloading which would likely have rendered it obsolete from an API standpoint.
 
@Girgias And from a purely "copy-edit" PoV you're currently inconsistently using '' and <php> for inline snippets. It would probably be good to always use the latter.
@Girgias And the hypothetical example with the 'Type' enum should probably not use a backed enum, because there is no need for it to be a backed enum.
 
 
2 hours later…
7:29 PM
@MarkR actually, I meant the "I think this RFC really needs an implementation in composer" not just a php-src PR...
 
 
2 hours later…
9:07 PM
@Danack Adding to composer would have been trivial. We discussed it in a thread over in the composer github issues somewhere.
 
Do you have a link?
 
github.com/composer/composer/discussions/9906 I can't remember if it got so far as adding the chained list to the C because I dropped the project
 
9:22 PM
thanks.
 

« first day (4547 days earlier)      last day (386 days later) »