diff options
| author | Pascal Rigaux <pixel@mandriva.com> | 2004-04-01 16:07:12 +0000 |
|---|---|---|
| committer | Pascal Rigaux <pixel@mandriva.com> | 2004-04-01 16:07:12 +0000 |
| commit | 932a59520de7f0c4cbe973f6ff9d0cdbe24ba66a (patch) | |
| tree | 77287a7ce1d667719f2cc34ca1ebfe71b37308d1 | |
| parent | 066e4e96b34426d3924789e649648048bf8a3f7e (diff) | |
| download | perl-MDK-Common-932a59520de7f0c4cbe973f6ff9d0cdbe24ba66a.tar perl-MDK-Common-932a59520de7f0c4cbe973f6ff9d0cdbe24ba66a.tar.gz perl-MDK-Common-932a59520de7f0c4cbe973f6ff9d0cdbe24ba66a.tar.bz2 perl-MDK-Common-932a59520de7f0c4cbe973f6ff9d0cdbe24ba66a.tar.xz perl-MDK-Common-932a59520de7f0c4cbe973f6ff9d0cdbe24ba66a.zip | |
*** empty log message ***
| -rw-r--r-- | perl_checker.src/test/suggest_better.t | 3 |
1 files changed, 2 insertions, 1 deletions
diff --git a/perl_checker.src/test/suggest_better.t b/perl_checker.src/test/suggest_better.t index aa413cb..9e55e20 100644 --- a/perl_checker.src/test/suggest_better.t +++ b/perl_checker.src/test/suggest_better.t @@ -72,8 +72,9 @@ foreach (@l) { use "push @l2, map { ... ? ... : () } . push @l2, yyy($_) if zzz($_); or sometimes "@l2 = map { ... ? ... : () } ..." } or sometimes "@l2 = map { if_(..., ...) } ..." -if (grep { xxx() } @l) {} in scalar context, use "any" instead of "grep" +if (grep { xxx() } @l) {} in boolean context, use "any" instead of "grep" $xxx ? $yyy : () you may use if_() here beware that the short-circuit semantic of ?: is not kept if you want to keep the short-circuit behaviour, replace () with @{[]} and there will be no warning anymore + |