NotificationsYou must be signed in to change notification settings
Fork6
Star31

Commitf818551

committed

Handle alphanumeric characters in matching GUC names

The check for whether all GUCs are present in the sample configfile used the POSIX character class :alpha: which corresponds toalphabet and not alphanumeric. Since GUC names can contain digitsas well we need to use the :alnum: character class instead.Author: Daniel Gustafsson <daniel@yesql.se>Discussion:https://postgr.es/m/2CB04559-B1D8-4558-B6F0-8F09093D629F@yesql.se

1 parente546989 commitf818551Copy full SHA for f818551

File tree

1 file changed

-1

lines changed

src/test/modules/test_misc/t
- 003_check_guc.pl

1 file changed

-1

lines changed

`‎src/test/modules/test_misc/t/003_check_guc.pl‎`

Lines changed: 1 addition & 1 deletion

Original file line number	Diff line number	Diff line change
`@@ -56,7 +56,7 @@`
`56`	`56`	`# file.`
`57`	`57`	`# - Valid configuration options are followed immediately by " = ",`
`58`	`58`	`# with one space before and after the equal sign.`
`59`		`-if ($line =~m/^#?([_[:alpha:]]+) = .*/)`
	`59`	`+if ($line =~m/^#?([_[:alnum:]]+) = .*/)`
`60`	`60`	`{`
`61`	`61`	`# Lower-case conversion matters for some of the GUCs.`
`62`	`62`	`my$param_name =lc($1);`

0 commit comments

Comments

(0)

Movatterモバイル変換

Navigation Menu

Search code, repositories, users, issues, pull requests...

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

Uh oh!

Commitf818551

File tree

1 file changed

1 file changed

`‎src/test/modules/test_misc/t/003_check_guc.pl‎`

0 commit comments