summaryrefslogtreecommitdiffstats
path: root/perl_checker.src/parser_helper.ml
Commit message (Collapse)AuthorAgeFilesLines
* 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)
* - pop behaves just like shiftPascal Rigaux2003-12-161-2/+8
| | | | | - check that the parameter to pop & shift is an array, and not more parameters are given
* fix checking value is dropped in tuplePascal Rigaux2003-12-161-1/+2
|
* - add the tuple(...) contextPascal Rigaux2003-12-161-10/+32
| | | | | | - check that the value xxx in "... or xxx" is dropped - handle "... or ..., ..."
* check what is on the left side of "xxx =~ s///"Pascal Rigaux2003-12-151-2/+8
|
* - check the context on left side of =~Pascal Rigaux2003-12-151-0/+4
| | | | | - !xx has the new context bool
* - specify the return context for N() and N_()Pascal Rigaux2003-12-151-1/+2
| | | | | - more precise position for warning "value is dropped"
* allow to specify the return context of functions instead of always saying ↵Pascal Rigaux2003-12-151-9/+10
| | | | M_unknown
* update "faking a normal lexbuf" code for new ocaml's parsing.mlPascal Rigaux2003-12-151-3/+3
|
* - correctly handle "shift" with no parameter at toplevelPascal Rigaux2003-12-151-2/+1
| | | | | - remove (hopefully) unused code rewriting shift -> shift(@_)
* in the generated .pot, try to use relative filenames (so that the .pot doesn'tPascal Rigaux2003-11-101-1/+1
| | | | | change at each build)
* new warning: you can replace "member($xxx, keys %yyy)" with "exists $yyy{$xxx}"Pascal Rigaux2003-11-101-0/+7
|
* also disallow map {...} %xxxPascal Rigaux2003-10-011-2/+9
|
* warn: foreach (%xxx) { ... }Pascal Rigaux2003-10-011-0/+2
|
* new warning: N_("xxx") . "yyy" is dumb since the string "xxx" will never get ↵Pascal Rigaux2003-09-301-0/+3
| | | | translated
* - allow $_o_XXX parameter name which is both unused and optional (same for ↵Pascal Rigaux2003-08-111-2/+5
| | | | | | | | | | | | $_b_XXX) - shift is a ONE_SCALAR_PARA so that $box->pack_start(shift @l, 0, 0, 4) is parsed correctly - in arrange_global_vars_declared(), don't keep anything in global_vars_declared, better create shadow packages to contain them - much better merging of multiple files defining functions in the same package. This fixes the bad behaviour when using the cache (esp. do_pkgs, but it was even worse with things in ugtk2.pm)
* - disallow return(...), prefering return ...Pascal Rigaux2003-05-271-13/+25
| | | | | - P_call_no_paren has a lower priority than P_comma
* better warning for: print $a . 'foo'Pascal Rigaux2003-05-261-1/+5
|
* add a special case to handle "arch => 1" without going through word_alone()Pascal Rigaux2003-05-261-4/+4
|
* - warn things like: if ($a = 1) { ... }Pascal Rigaux2003-05-261-0/+6
| | | | | - undef is always false
* check for "0 or ..."Pascal Rigaux2003-05-261-0/+5
|