summaryrefslogtreecommitdiffstats
path: root/perl_checker.src/parser_helper.ml
Commit message (Collapse)AuthorAgeFilesLines
* - ugly hack to workaround uniop calls priority pb, but at least have a ↵Pascal Rigaux2004-12-131-4/+8
| | | | | | | P_uniop instead of using P_mul - add delete as a uniop
* create to_Deref_with_arrow()Pascal Rigaux2004-12-051-3/+6
|
* remove unused functionPascal Rigaux2004-11-261-9/+0
|
* check sub { { ... }; }Pascal Rigaux2004-11-261-8/+12
|
* don't "double quotes are unneeded" for "@m3[1..$#m3]"Pascal Rigaux2004-11-261-1/+2
|
* recognize a form of "find { ... } ..."Pascal Rigaux2004-11-171-0/+7
|
* create string_of_fromparser(), needs tuningPascal Rigaux2004-11-171-28/+78
|
* - new function lines_to_Block() which takes care of calling check_block_sub()Pascal Rigaux2004-11-171-3/+8
| | | | | - anonymous_sub now uses lines_to_Block()
* allow disabling warnings on command-line (and have various warnings level)Pascal Rigaux2004-11-101-114/+113
|
* suggest mkdir() instead of system("mkdir ...")Pascal Rigaux2004-10-141-6/+27
|
* - add char_quote (to help emacs caml mode)Pascal Rigaux2004-10-141-2/+2
| | | | | - change prototype of str_begins_with
* check if_() with only one parameterPascal Rigaux2004-10-141-0/+5
|
* replace "my $foo = ... if <cond>" with "my $foo = <cond> && ..."Pascal Rigaux2004-10-131-0/+16
| | | | | replace "<cond> or my $foo = ..." with "my $foo = !<cond> && ..."
* check 'xxx' if $xxx (=> value is dropped)Pascal Rigaux2004-10-131-26/+29
|
* fix indentationPascal Rigaux2004-10-131-19/+19
|
* $#x == -1 is better written @x == 0Pascal Rigaux2004-10-131-2/+5
|
* check $#l < 0Pascal Rigaux2004-10-131-0/+5
|
* allow join(N("xxx"), ...)Pascal Rigaux2004-10-131-1/+2
|
* you can replace "any { $_ eq ... } @l" with "member(..., @l)"Pascal Rigaux2004-10-131-1/+9
|
* - check push argumentsPascal Rigaux2004-10-131-0/+7
| | | | | - add a test for pop arguments
* disallow the use of the return value of push and unshiftPascal Rigaux2004-10-131-1/+1
|
* you can replace "grep { !member($_, ...) } @l" with "difference2([ @l ], [ ↵Pascal Rigaux2004-08-121-1/+8
| | | | ... ])"
* rewrite to allow a special rule for "grep"Pascal Rigaux2004-08-121-10/+17
|
* check lvalue validityPascal Rigaux2004-08-121-0/+20
|
* "$foo or $foo = ..." can be written "$foo ||= ..."Pascal Rigaux2004-08-111-0/+3
|
* cleanupPascal Rigaux2004-08-111-44/+41
|
* cleanupPascal Rigaux2004-08-111-26/+23
|
* "$a[@a] = ..." is better written "push @a, ..."Pascal Rigaux2004-08-111-0/+4
|
* suggest better for !($foo == $bar) (same for eq, != and ne)Pascal Rigaux2004-08-111-0/+10
|
* many perl_checker enhancements and cleanupPascal Rigaux2004-06-281-2/+2
|
* for flexible detecting of functional patternsPascal Rigaux2004-05-091-9/+13
|
* finishing context checks cleanupPascal Rigaux2004-05-091-50/+46
|
* better contextsPascal Rigaux2004-05-081-91/+86
|
* detect "$xxx == undef"Pascal Rigaux2004-04-301-3/+6
|
* suggest replacing system(qq(foo "$xxx")) with system('foo', $xxx)Pascal Rigaux2004-04-281-0/+8
|
* fix typoPascal Rigaux2004-04-281-0/+1
|
* separate rewriting rules from warningsPascal Rigaux2004-04-281-55/+51
|
* various detection of join() misusePascal Rigaux2004-04-281-0/+9
|
* get_pos_from_expr() (previously named get_pos_from_tree()) is useful, even ↵Pascal Rigaux2004-04-281-0/+34
| | | | in parser_helper
* fix typoPascal Rigaux2004-04-281-1/+1
|
* - deprecate "open F, ..." prefering "open(my $F, ...)"Pascal Rigaux2004-04-281-1/+14
| | | | | - fix an internal_error
* - do check_simple_pattern() directly in from_PATTERN()Pascal Rigaux2004-04-281-2/+4
| | | | | - don't die, log instead for some regexp expressions
* fix warning (would also need fixing detecting of boolean context vs scalarPascal Rigaux2004-03-311-1/+1
| | | | | context)
* in "$a ? $a : xxx", "xxx" can need short circuitPascal Rigaux2004-03-311-1/+1
| | | | | (even if i don't think any value which is_a_scalar can need_short_circuit)
* perl_checker: entries in generated pot file are sorted by filesPascal Rigaux2004-01-091-12/+14
|
* fix checking dropped value in some casesPascal Rigaux2004-01-051-1/+2
|
* use hash_ref() to check priority of the argumentsPascal Rigaux2004-01-051-1/+3
|
* disallowPascal Rigaux2003-12-161-12/+32
| | | | | | | - $a = (1, 2) - my $a = (1, 2) - my ($a, $b) = (1, 2, 3)
* check the number of parameters given to N()Pascal Rigaux2003-12-161-10/+19
|
* - drop internally operator "qw" by rewriting qw(...) into a list of stringsPascal Rigaux2003-12-161-2/+2
| | | | | | - as a side effect "foo" . qw(bar boo) is now detected - allow pop(@l) (with explicit parentheses)