diff options
| author | Daniel Gustafsson <dgustafsson@postgresql.org> | 2021-08-31 11:07:04 +0200 |
|---|---|---|
| committer | Daniel Gustafsson <dgustafsson@postgresql.org> | 2021-08-31 11:07:04 +0200 |
| commit | bb466c6b0992a1a21c03239a7b0a87ebadd3bee1 (patch) | |
| tree | ca337e27f75307637ab18311e0c733e03f517587 /src/backend/utils/adt/ruleutils.c | |
| parent | f2bbadce6b5052337a11a33ea6bd8d8aebe2610a (diff) | |
| download | postgresql-bb466c6b0992a1a21c03239a7b0a87ebadd3bee1.tar.gz postgresql-bb466c6b0992a1a21c03239a7b0a87ebadd3bee1.zip | |
Prohibit map and grep in void context
map and grep are not intended to be used as mutators, iterating
with side-effects should be done with for or foreach loops. This
fixes the one occurrence of the pattern, and bumps the perlcritic
policy to severity 5 for the map and grep policies.
Author: Dagfinn Ilmari Mannsåker <ilmari@ilmari.org>
Reviewed-by: Michael Paquier <michael@paquier.xyz>
Reviewed-by: Andrew Dunstan <andrew@dunslane.net>
Reviewed-by: Julien Rouhaud <rjuju123@gmail.com>
Discussion: https://postgr.es/m/87fsvzhhc4.fsf@wibble.ilmari.org
Diffstat (limited to 'src/backend/utils/adt/ruleutils.c')
0 files changed, 0 insertions, 0 deletions