#761508 libperl-critic-perl: Useless topic variable usage false positive on @_

#761508#5
Date:
2014-09-14 13:47:11 UTC
From:
To:
Hi!

Here's what seems like a false positive that I found while going over
the dpkg code:

,--- critic-topic.pl ---
#!/usr/bin/perl

use strict;
use warnings;

our $VERSION = '0.0';

sub func {
    return 1 if -l $_[0];
    return 0;
}

func('pathname');

1;
`---

Reproducible with:

   $ perlcritic -s BuiltinFunctions::ProhibitUselessTopic critic-topic.pl

Of course unpacking @_ makes this go away, which I'll be doing on that
codebase, but the warning is still wrong IMO.

Thanks,
Guillem