NotificationsYou must be signed in to change notification settings
Fork4.9k
Star17.7k

Commit52d0862

committed

Adjust regex for test with opening parenthesis in character classes

As written, the test was throwing an error because of an unbalancedparenthesis. The regex used in the test is adjusted to not fail and totest the case of an opening parenthesis in a character class after somenested square brackets.Oversight ind46911e.Discussion:https://postgr.es/m/16ab039d1af455652bdf4173402ddda145f2c73b.camel@cybertec.at

1 parente9e535d commit52d0862Copy full SHA for 52d0862

File tree

2 files changed

-3

lines changed

src/test/regress
- expected
  - strings.out
- sql
  - strings.sql

2 files changed

-3

lines changed

`‎src/test/regress/expected/strings.out`

Lines changed: 7 additions & 2 deletions

Original file line number	Diff line number	Diff line change
`@@ -631,8 +631,13 @@ EXPLAIN (COSTS OFF) SELECT * FROM TEXT_TBL WHERE f1 SIMILAR TO '$[$[:alnum:]$]$'`
`631`	`631`	`(2 rows)`
`632`	`632`
`633`	`633`	`-- Opening parenthesis "("`
`634`		`-EXPLAIN (COSTS OFF) SELECT * FROM TEXT_TBL WHERE f1 SIMILAR TO '([([:alnum:](](';`
`635`		`-ERROR: invalid regular expression: parentheses () not balanced`
	`634`	`+EXPLAIN (COSTS OFF) SELECT * FROM TEXT_TBL WHERE f1 SIMILAR TO '()[([:alnum:](]()';`
	`635`	`+ QUERY PLAN`
	`636`	`+------------------------------------------------------`
	`637`	`+ Seq Scan on text_tbl`
	`638`	`+ Filter: (f1 ~ '^(?:(?:)[([:alnum:](](?:))$'::text)`
	`639`	`+(2 rows)`
	`640`	`+`
`636`	`641`	`-- Caret "^"`
`637`	`642`	`EXPLAIN (COSTS OFF) SELECT * FROM TEXT_TBL WHERE f1 SIMILAR TO '^[^[:alnum:]^[^^][[^^]][\^][[\^]]\^]^';`
`638`	`643`	`QUERY PLAN`

`‎src/test/regress/sql/strings.sql`

Lines changed: 1 addition & 1 deletion

Original file line number	Diff line number	Diff line change
`@@ -204,7 +204,7 @@ EXPLAIN (COSTS OFF) SELECT * FROM TEXT_TBL WHERE f1 SIMILAR TO '.[.[:alnum:].].'`
`204`	`204`	`-- Dollar "$"`
`205`	`205`	`EXPLAIN (COSTS OFF)SELECT*FROM TEXT_TBLWHERE f1 SIMILAR TO'$[$[:alnum:]$]$';`
`206`	`206`	`-- Opening parenthesis "("`
`207`		`-EXPLAIN (COSTS OFF)SELECT*FROM TEXT_TBLWHERE f1 SIMILAR TO'([([:alnum:](](';`
	`207`	`+EXPLAIN (COSTS OFF)SELECT*FROM TEXT_TBLWHERE f1 SIMILAR TO'()[([:alnum:](]()';`
`208`	`208`	`-- Caret "^"`
`209`	`209`	`EXPLAIN (COSTS OFF)SELECT*FROM TEXT_TBLWHERE f1 SIMILAR TO'^[^[:alnum:]^[^^][[^^]][\^][[\^]]\^]^';`
`210`	`210`	`-- Closing square bracket "]" at the beginning of character class`

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!

Commit52d0862

File tree

2 files changed

2 files changed

`‎src/test/regress/expected/strings.out`

`‎src/test/regress/sql/strings.sql`

0 commit comments