tag 658835 + moreinfo
thanks

Hi,

I have no intention of sponsoring this package, but as I did take a
brief look at it before:

| my $config_file = "/home/jabber/aspsms-t/etc/aspsms-web-notify.xml";
Not really useful for programs in /usr/bin.

| use lib "./";
Not really useful for programs in /usr/bin.

Code like
  eval { unlink($passfile); }; if($@) { handle_error(); }
will not catch any errors, see
  perl -E 'eval { unlink "/tmp/does-not-exist"; }; if ($@) { say "There was an 
error." }'

Code that is not or inconsistently indented is hard to read.

Regards,
Ansgar



-- 
To UNSUBSCRIBE, email to debian-mentors-requ...@lists.debian.org
with a subject of "unsubscribe". Trouble? Contact listmas...@lists.debian.org
Archive: http://lists.debian.org/87mx8uogij....@deep-thought.43-1.org

Reply via email to