summaryrefslogtreecommitdiffstats
path: root/perl_checker.src
Commit message (Collapse)AuthorAgeFilesLines
...
* check for "0 or ..."Pascal Rigaux2003-05-261-0/+5
|
* explicitly disallow <<=, >>= and **= (instead of having a syntax error)Pascal Rigaux2003-05-261-0/+5
|
* check prototype coherence: disallow ($a, @b, $c) or ($a, $o_b, $c)Pascal Rigaux2003-05-261-2/+9
|
* warn spurious space in ( 1, 2) which should be (1, 2)Pascal Rigaux2003-05-231-1/+1
|
* warn $o->method() which should be $o->methodPascal Rigaux2003-05-231-1/+1
|
* suggest using the functional map instead of the imperative foreach whenPascal Rigaux2003-05-231-0/+12
| | | | | possible
* fix detection of grep in scalar contextPascal Rigaux2003-05-231-3/+4
|
* add warning: you can replace "map { if_(..., $_) }" with "grep { ... }"Pascal Rigaux2003-05-231-0/+4
|
* - suggest any instead of grep in scalar contextPascal Rigaux2003-05-233-4/+19
| | | | | - suggest foreach instead of map in empty context
* fix "/^\d+\.\*$/" giving warning "you can remove \".*$\" at the end of your ↵Pascal Rigaux2003-05-231-1/+1
| | | | regexp"
* correctly warn about the missing space in "my ($a,$b, $c);"Pascal Rigaux2003-05-163-9/+12
|
* fix pot generation (have \" instead of \\\")Pascal Rigaux2003-05-161-1/+2
|
* use print_endline_flush instead of prerr_endline to comply with ↵Pascal Rigaux2003-04-301-1/+1
| | | | --restrict-to-files
* map and grep do not return a scalarPascal Rigaux2003-04-301-0/+2
|
* ->can is allowed on any objectPascal Rigaux2003-04-301-1/+1
|
* allow "*foo = sub {}" with no prototypePascal Rigaux2003-04-306-19/+22
|
* \ *is* necessary in "$foo\{"Pascal Rigaux2003-04-301-1/+1
|
* ensure return values are usedPascal Rigaux2003-04-295-61/+138
|
* allow "sub pkg::foo() { ...}"Pascal Rigaux2003-04-282-5/+20
|
* replace warning "unused variable @_" with "if the function doesn't take anyPascal Rigaux2003-04-251-2/+8
| | | | | parameters, please use the empty prototype.\nexample \"sub foo() { ... }\""
* - handle empty prototypesPascal Rigaux2003-04-249-58/+101
| | | | | | | - check number of parameters in method calls (ignoring empty overloaded methods) - allow non-empty prototypes for methods not using @_
* check number of args in method callsPascal Rigaux2003-04-221-12/+35
|
* fix argument checking for Class->method(...) callsPascal Rigaux2003-04-221-1/+1
|
* basic "number of arguments" checkingPascal Rigaux2003-04-177-72/+128
|
* - recognize function parameters and disallow using @_ in the rest of the ↵Pascal Rigaux2003-04-161-1/+20
| | | | | | | function body - suggest to use &f instead of f(@_)
* forbid "unless (...) { ... } else { ... }" and "unless (...) { ... } elsif ↵Pascal Rigaux2003-04-153-1/+7
| | | | (...) { ... }
* add basic "type" checking (using a very liberal lattice)Pascal Rigaux2003-04-155-343/+529
|
* use new types to have stricter type checking (prior to adding maybe_context)Pascal Rigaux2003-04-135-612/+662
|
* allow 333 * `xxx` with no warningPascal Rigaux2003-04-111-1/+1
|
* warn non-useful or non-readable escaped sequences in strings and regexpsPascal Rigaux2003-04-111-15/+60
| | | | | (eg: /^\// should be m|^/|, /xxx\=xxx/ should be /xxx=xxx/ ...)
* warn things like: ($foo) ||= ...Pascal Rigaux2003-04-111-0/+4
|
* enhance non_scalar case for some operators using is_not_a_scalarPascal Rigaux2003-04-101-10/+20
|
* handle "keys %pkg::" (twas broken because keys() is now a ONE_SCALAR_PARA)Pascal Rigaux2003-04-101-0/+1
|
* keys() is a ONE_SCALAR_PARAPascal Rigaux2003-04-101-0/+1
|
* correctly (in Perl way) handle priority for some special unary functions ↵Pascal Rigaux2003-04-103-17/+15
| | | | (length, exists, ref)
* warn xxx == "ia64", xxx eq 2Pascal Rigaux2003-04-101-0/+18
|
* 0.2 is a NUM, not a REVISION (otherwise it gets into a Raw_string)Pascal Rigaux2003-04-101-1/+1
|
* better error message ("please remove the space before the function call"Pascal Rigaux2003-04-101-1/+3
| | | | | instead of "can't handle this nicely")
* warn when using a regexp terminated with .* or .*$ (which is useless)Pascal Rigaux2003-04-101-1/+9
|
* allow to selectively import from @EXPORT instead of only accepting @EXPORT_OKPascal Rigaux2003-04-101-1/+1
|
* handle "local $_ = ..." at toplevelPascal Rigaux2003-04-021-0/+1
|
* simplify handling of variables declared in "if (...) ..." to meet perl's way.Pascal Rigaux2003-04-021-4/+2
| | | | | | | | | | | | | | eg: if (my $i = foo()) { } else { $i; # accessing $i here is allowed } if (my $i = foo()) { } elsif ($i = bar()) { # don't use "my" for $i since it's already in scope }
* enhance warning "... =~ /^foo$/" is better written "... eq 'foo'"Pascal Rigaux2003-04-021-1/+1
|
* fix error message for grep not used with a blockPascal Rigaux2003-04-011-1/+1
|
* - $foo =~ /^dhcp$/ is better written $foo eq 'dhcp'Pascal Rigaux2003-04-013-4/+14
| | | | | - warn "grep(&f, ...)", use "grep { f } ..." instead
* - warn !($var =~ /.../)Pascal Rigaux2003-04-014-5/+19
| | | | | | - warn $a & 0xff == $b better written $a & (0xff == $b) - "not" should not be used, use "!" instead
* warn "map(&f, ...)", use "map { f } ..." insteadPascal Rigaux2003-04-011-0/+6
|
* - warn <always true> || ... (eg: 1 || foo())Pascal Rigaux2003-04-013-7/+40
| | | | | | | | | - warn <always false> || ... - warn <always true> && ... - warn <always false> && ... - suggest @$foo instead of @{$foo} - suggest $foo->[0] instead of ${$foo}[0]
* when a #-PO comment on a same translated string is present twice, it should ↵Pascal Rigaux2003-04-012-6/+4
| | | | | | | put the -PO comment in any case (more precisely, it concatenates all the -PO comments)
* - make "please don't use prototype pre-declaration" a warningPascal Rigaux2003-04-011-2/+11
| | | | | - make a warning for unary plus instead of a "syntax error"