On 21.07.2024 at 13:28, Tim Düsterhus wrote:

> On 7/20/24 20:31, Christoph M. Becker wrote:
>
>> If the tokenizer would tokenize a whole file as a single token, would
>> that also be correct?  Of course, I'm exaggerating, but
>
> The function is documented as:
>
>> token_get_all() parses the given code string into PHP language tokens
>> using the Zend engine's lexical scanner.
>
> So if Zend engine's lexical scanner would tokenize a complete PHP file
> as a single token, then the output would be correct, yes.

Apparently we cannot even agree that there is a bug, and as such we
cannot fix it in PHP 8.3 according to the release process documentation.

>> <https://3v4l.org/qIf2c> doesn't look correct to me – "yield /* comment
>> */ from" shouldn't be a single token.
>
> Looking at this from a user expectation perspective and not a technical
> perspective, neither should `yield from` be a single token in:
>
>     var_dump(token_get_all('<?php yield from $foo;'));

ACK

Cheers,
Christoph

Reply via email to