php / php/php-src

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

Đang mở
#8,269 3 bình luận 2 reaction 0 người được giao Xem trên GitHub

Chưa có ai nhận issue này.

Bug Category: Engine Status: Verified
Ngôn ngữ chính
C
Star
40.4k
Fork
8.2k
Merge trung bình
2 ngày 13 giờ
Pull request đã merge (30 ngày)
96

Mô tả

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

Hướng dẫn đóng góp

Mở hướng dẫn đóng góp

Bắt đầu từ đâu

  1. Đọc hết issue, rồi đọc hướng dẫn đóng góp của dự án.
  2. Bình luận trên issue rằng bạn sẽ nhận — tránh hai người làm cùng một việc.
  3. Fork repository và làm thay đổi trên một nhánh.
  4. Mở pull request có tham chiếu số hiệu của issue.

Hướng nghiên cứu

Tái hiện ví dụ generator được cung cấp trên PHP master và so sánh luồng lỗi quan sát được với hành vi catch dự kiến. Sau đó kiểm tra việc xử lý yield trong generator, hành vi return-by-reference và các trường hợp destructor đã nêu. Được xem là hoàn tất khi việc xử lý ngoại lệ dự kiến đã được xác định và được bao phủ cho các kịch bản được báo cáo, mà không vô tình thay đổi hành vi đã được thiết lập.

Do mô hình lập chỉ mục viết ra từ nội dung của issue.

Đánh giá

Công nghệ
php
Lĩnh vực
compilers
Loại issue
Lỗi
Độ khó
4/5
Thời gian dự kiến
3-5 ngày
Mức độ hoạt động
Đình trệ
Độ rõ ràng
Khá rõ ràng
Mức phù hợp với người mới
35/100

Nhận issue mới trong hộp thư của bạn

Bản tóm tắt ngắn những issue GitHub phù hợp với người mới.