php / php/php-src

Exceptions thrown during yield operation fall through the generator instead of being handled within

Offen
#8,269 3 Kommentare 2 Reaktionen 0 zugewiesene Personen Auf GitHub ansehen

Dieses Issue hat noch niemand übernommen.

Bug Category: Engine Status: Verified
Vorherrschende Sprache
C
Sterne
40.4k
Forks
8.1k
Ø Merge
2 T. 13 Std.
Gemergte PRs (30 T.)
96

Beschreibung

Description

The following code:

<?php

set_error_handler(function() { throw new \Error(); });

function &gen() {
    try {
        yield 1;
    } catch (Error $e) {
        print "Caught";
    }
}

foreach (gen() as $y);

Resulted in Uncaught Error

But I expected this output instead:

Caught

return_by_ref has the same problem, it seems to me, however in generators there's the additional case of exceptions in destructors of previous key and values.

I wouldn't change the longstanding behaviour in current PHP versions, but should be fixed for PHP 8.2 possibly, for academical correctness at least.

PHP Version

master

Operating System

No response

Beitragsleitfaden

Beitragsleitfaden öffnen

Erste Schritte

  1. Lies das ganze Issue und danach den Beitragsleitfaden des Projekts.
  2. Schreib ins Issue, dass du es übernimmst — das erspart doppelte Arbeit.
  3. Forke das Repository und arbeite in einem Branch.
  4. Öffne einen Pull Request, der die Issue-Nummer nennt.

Rechercherichtung

Das bereitgestellte Generator-Beispiel auf PHP master reproduzieren und den beobachteten Fehlerablauf mit dem erwarteten catch-Verhalten vergleichen. Anschließend die Behandlung von yield in Generatoren, das Rückgabeverhalten per Referenz und die genannten Destruktorfälle untersuchen. Als abgeschlossen gilt die Aufgabe, wenn die beabsichtigte Ausnahmebehandlung für die gemeldeten Szenarien definiert und abgedeckt ist, ohne unbeabsichtigt etabliertes Verhalten zu ändern.

Vom Indexierungsmodell aus dem Issue-Text verfasst.

Bewertung

Tech-Stack
php
Bereich
compilers
Issue-Typ
Bug
Schwierigkeit
4/5
Geschätzter Aufwand
3-5 Tage
Aktivitätsstatus
Veraltet
Klarheit
Größtenteils klar
Anfängerfreundlichkeit
35/100

Neue Issues direkt in Ihr Postfach

Eine kurze Übersicht über anfängerfreundliche GitHub-Issues.