Allow catching as ParseError "Non-abstract method C::foo() must contain a body"
Dieses Issue hat noch niemand übernommen.
- Vorherrschende Sprache
- C
- Sterne
- 40.4k
- Forks
- 8.1k
- Ø Merge
- 2 T. 13 Std.
- Gemergte PRs (30 T.)
- 96
Beschreibung
Description
Since PHP 7 it is possible to catch syntax errors in PHP files during inclusion, which is a very nice feature, such as:
try {
eval("class C { function foo() }");
}
catch (\Throwable $e) {
print "CAUGHT [" . get_class($e) . "]: " . $e->getMessage() . "\n";
}
// prints: CAUGHT [ParseError]: syntax error, unexpected token "}", expecting ";" or "{"
// execution continues
Unfortunately, the same is not possible in case the error is a non-abstract method without body:
try {
eval("class C { function foo(); }");
}
catch (\Throwable $e) {
print "CAUGHT [" . get_class($e) . "]: " . $e->getMessage() . "\n";
}
// prints: Fatal error: Non-abstract method C::foo() must contain body in /in/YY0Wu(13) : eval()'d code on line 1
// execution terminates
Would it be possible to implement a throw also for this case?
Beitragsleitfaden
Erste Schritte
- Lies das ganze Issue und danach den Beitragsleitfaden des Projekts.
- Schreib ins Issue, dass du es übernimmst — das erspart doppelte Arbeit.
- Forke das Repository und arbeite in einem Branch.
- Öffne einen Pull Request, der die Issue-Nummer nennt.
Rechercherichtung
Beginne beim eval()-Pfad im PHP-Interpreter und vergleiche, wie der gezeigte Syntaxfehler zu einem ParseError wird, damit, wie die Prüfung auf eine nicht abstrakte Methode die Ausführung derzeit beendet. Verwende die beiden eval()-Beispiele des Issues als Reproducer; fertig ist es, wenn der zweite Fall als ParseError abgefangen werden kann und die Ausführung fortgesetzt wird.
Vom Indexierungsmodell aus dem Issue-Text verfasst.
Bewertung
- Tech-Stack
- php
- Bereich
- compilers
- Issue-Typ
- Feature
- Schwierigkeit
- 4/5
- Geschätzter Aufwand
- 3-5 Tage
- Aktivitätsstatus
- Veraltet
- Klarheit
- Größtenteils klar
- Anfängerfreundlichkeit
- 35/100