1
0
mirror of https://github.com/imapsync/imapsync.git synced 2024-11-17 08:12:48 +01:00
imapsync/W/perlcritic_3.out

50 lines
4.8 KiB
Plaintext
Raw Normal View History

2014-11-19 23:08:41 +01:00
Main code has high complexity score (358) at line 1, column 1. Consider refactoring. (Severity: 3)
Expression form of "map" at line 1142, column 51. See page 169 of PBP. (Severity: 4)
Expression form of "map" at line 1152, column 51. See page 169 of PBP. (Severity: 4)
Code structure is deeply nested at line 1586, column 41. Consider refactoring. (Severity: 3)
Too many arguments at line 1770, column 1. See page 182 of PBP. (Severity: 3)
Too many arguments at line 1788, column 1. See page 182 of PBP. (Severity: 3)
Too many arguments at line 1798, column 1. See page 182 of PBP. (Severity: 3)
Too many arguments at line 2032, column 1. See page 182 of PBP. (Severity: 3)
Too many arguments at line 2093, column 1. See page 182 of PBP. (Severity: 3)
Too many arguments at line 2153, column 1. See page 182 of PBP. (Severity: 3)
Too many arguments at line 2256, column 1. See page 182 of PBP. (Severity: 3)
Subroutine "is_valid_directory" does not end with "return" at line 2392, column 1. See page 197 of PBP. (Severity: 4)
"die" used instead of "croak" at line 2433, column 2. See page 283 of PBP. (Severity: 3)
Reused variable name in lexical scope: $imap2 at line 2548, column 9. Invent unique variable names. (Severity: 3)
Regular expression without "/x" flag at line 2567, column 25. See page 236 of PBP. (Severity: 3)
Regular expression without "/x" flag at line 2570, column 20. See page 236 of PBP. (Severity: 3)
Mixed high and low-precedence booleans at line 2571, column 13. See page 70 of PBP. (Severity: 4)
Expression form of "eval" at line 2996, column 13. See page 161 of PBP. (Severity: 5)
Expression form of "eval" at line 3217, column 13. See page 161 of PBP. (Severity: 5)
Subroutine "select_msgs" does not end with "return" at line 3358, column 1. See page 197 of PBP. (Severity: 4)
Subroutine "tests_msgs_from_maxmin" does not end with "return" at line 3497, column 1. See page 197 of PBP. (Severity: 4)
Subroutine "copy_message" with high complexity score (30) at line 3574, column 1. Consider refactoring. (Severity: 3)
Too many arguments at line 3574, column 1. See page 182 of PBP. (Severity: 3)
Too many arguments at line 3665, column 1. See page 182 of PBP. (Severity: 3)
Subroutine "tests_subject" does not end with "return" at line 3759, column 1. See page 197 of PBP. (Severity: 4)
Too many arguments at line 3822, column 1. See page 182 of PBP. (Severity: 3)
Subroutine "sleep_if_needed" does not end with "return" at line 3878, column 1. See page 197 of PBP. (Severity: 4)
Reused variable name in lexical scope: $total_bytes_transferred at line 3879, column 2. Invent unique variable names. (Severity: 3)
Reused variable name in lexical scope: $nb_msg_transferred at line 3879, column 2. Invent unique variable names. (Severity: 3)
Reused variable name in lexical scope: $nb_msg_transferred at line 3891, column 9. Invent unique variable names. (Severity: 3)
Reused variable name in lexical scope: $maxmessagespersecond at line 3891, column 9. Invent unique variable names. (Severity: 3)
Reused variable name in lexical scope: $total_bytes_transferred at line 3912, column 9. Invent unique variable names. (Severity: 3)
Reused variable name in lexical scope: $maxbytespersecond at line 3912, column 9. Invent unique variable names. (Severity: 3)
Mismatched operator at line 4380, column 75. Numeric/string operators and operands should match. (Severity: 3)
Hard tabs used at line 4464, column 10. See page 20 of PBP. (Severity: 3)
Expression form of "eval" at line 4736, column 13. See page 161 of PBP. (Severity: 5)
Too many arguments at line 4892, column 1. See page 182 of PBP. (Severity: 3)
Backtick operator used at line 5205, column 12. Use IPC::Open3 instead. (Severity: 3)
Backtick operator used at line 5225, column 11. Use IPC::Open3 instead. (Severity: 3)
Expression form of "eval" at line 5526, column 43. See page 161 of PBP. (Severity: 5)
Expression form of "eval" at line 5530, column 45. See page 161 of PBP. (Severity: 5)
"$i" is declared but not used at line 5794, column 9. Unused variables clutter code and make it harder to read. (Severity: 3)
Reused variable name in lexical scope: $logfile at line 5846, column 2. Invent unique variable names. (Severity: 3)
Subroutine "teelaunch" does not end with "return" at line 5853, column 1. See page 197 of PBP. (Severity: 4)
Reused variable name in lexical scope: $logfile at line 5854, column 2. Invent unique variable names. (Severity: 3)
Close filehandles as soon as possible after opening them at line 5857, column 2. See page 209 of PBP. (Severity: 4)
"die" used instead of "croak" at line 5858, column 7. See page 283 of PBP. (Severity: 3)
Magic variable "*STDERR" should be assigned as "local" at line 5860, column 10. See pages 81,82 of PBP. (Severity: 4)
One-argument "select" used at line 5861, column 2. See page 224 of PBP. (Severity: 4)