2012-02-02 4 views
7

フォローコードは評論家によって受け入れられないを分離するために使用される、重大度4:Perlの評論家:コンマは文

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; 

は、一時的な変数を使用して、私のリターンを書くことは本当に優れているのか、検出に評論家が間違っていますa Comma used to separate statements私はハッシュリファレンスを構築して返していますが、

+0

返信 '({....});'? – Ingo

+0

この現象は再現できません。 'Perl :: Critic'のどのバージョンを使っていますか? – Borodin

+0

私は問題を再現できません: 'perlcritic --brutal -s ValuesAndExpressions :: ProhibitCommaSeparatedStatementsso9110962.pl␤so9110962.plsource OK'おそらくPPIとPerl :: Criticをアップグレードしますか? – daxim

答えて

7

バージョン1.105でこのバグが見つかりました。バージョン1.116ではなくなりました。そこのどこかに固定されています。

修正プログラムは変更ログには記載されていませんが、PPIの変更が記載されています。 PPIエラーが発生している可能性があります。

+0

ありがとうございました。最後のバージョンにアップグレードすることは本当に助けになりました(これまでに渡されたコードについても全く新しい批判のリストが作成されました。 – Konerak

関連する問題