2012-02-02 11 views
7

đang Follow không được chấp nhận bởi phê bình, mức độ nghiêm trọng 4:Perl phê bình: Comma sử dụng để báo cáo riêng biệt

return { 
    'debug' => $debug, 
    'identifier' => $identifier 
}; 

tôi nhận được lỗi này:

# Perl::Critic found these violations in "filename.pl": 
# Comma used to separate statements at line 356, column 3. See pages 68,71 of PBP. (Severity: 4) 

Nhưng mã này được chấp nhận mà không bất kỳ nhận xét:

my $result = { 
    'debug' => $debug, 
    'identifier' => $identifier 
}; 

return $result; 

Có thực sự tốt hơn để viết trả lại bằng biến tạm thời hoặc là nhà phê bình sai khi phát hiện một Comma used to separate statements trong khi tôi chỉ đang xây dựng và trả về một hashref?

+0

WOuld nó giúp đỡ nếu bạn viết trở lại '({....});'? – Ingo

+0

Tôi không thể sao chép hành vi này. Bạn đang sử dụng phiên bản «Perl :: Critic' nào? – Borodin

+0

Tôi không thể tái tạo vấn đề: 'perlcritic --brutal -s ValuesAndExpressions :: ProhibitCommaSeparatedBáo cáo so9110962.pl␤so9110962.pl source OK' Có lẽ nâng cấp PPI và Perl :: Critic? – daxim

Trả lời

7

Tôi đã tìm thấy lỗi này trong phiên bản 1.105, phiên bản này đã biến mất trong phiên bản 1.116. Nó đã cố định ở đâu đó ở giữa đó.

Sửa lỗi không được đề cập trong nhật ký thay đổi, nhưng thay đổi PPI được đề cập. Có thể đã là lỗi PPI.

+0

Cảm ơn, nâng cấp lên phiên bản cuối cùng thực sự đã giúp (và tạo ra một danh sách hoàn toàn mới về những lời chỉ trích cho mã mà trước đây cũng đã trôi qua!) – Konerak

Các vấn đề liên quan