> On Jan 17, 2022, at 1:54 PM, Robert Haas <robertmh...@gmail.com> wrote: > > + * dotcnt: how many separators were parsed from the pattern, by reference. > + * Can be NULL. > > But then: > > + Assert(dotcnt != NULL);
Removed the "Can be NULL" part, as that use case doesn't make sense. The caller should always care whether the number of dots was greater than they are prepared to handle. > On a related note, it's unclear why you've added three new arguments > to processSQLNamePattern() but only one of them gets a mention in the > function header comment. Updated the header comments to include all parameters. > It's also pretty clear that the behavior of patternToSQLRegex() is > changing, but the function header comments are not. Updated the header comments for this, too. Also, rebased as necessary:
v5-0001-Reject-patterns-with-too-many-parts-or-wrong-db.patch
Description: Binary data
— Mark Dilger EnterpriseDB: http://www.enterprisedb.com The Enterprise PostgreSQL Company
