Perl Critic: запятая используется для разделения операторов - PullRequest
7 голосов
/ 02 февраля 2012

Следующий код не принят Критиком, серьезность 4:

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

Я получаю эту ошибку:

# 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)

Но этот код принимается без каких-либо замечаний:

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

return $result; 

Действительно ли лучше написать мое возвращение, используя временную переменную, или критик ошибается в обнаружении Comma used to separate statements, пока я просто создаю и возвращаю хеш-код?

1 Ответ

7 голосов
/ 02 февраля 2012

Я нашел эту ошибку в версии 1.105, она исчезла в версии 1.116.Это было исправлено где-то посередине.

Исправление не упоминается в журналах изменений, но упоминаются изменения PPI.Возможно, произошла ошибка PPI.

...