Skip to content

Handle functions with a double dollar sign in the body#81

Open
barkingfoodog wants to merge 1 commit into
eulerto:masterfrom
barkingfoodog:dollar-quoting-string
Open

Handle functions with a double dollar sign in the body#81
barkingfoodog wants to merge 1 commit into
eulerto:masterfrom
barkingfoodog:dollar-quoting-string

Conversation

@barkingfoodog

Copy link
Copy Markdown
Contributor

Check a function or procedure's body for the string $$ to see if it is safe for dollar-quoting the body. If found, expand the dollar quote until it is not found in the body.

@eulerto

eulerto commented Dec 2, 2019

Copy link
Copy Markdown
Owner

@barkingfoodog Why don't you use appendStringLiteralDQ instead of duplicating pg_dump code ?

@barkingfoodog

Copy link
Copy Markdown
Contributor Author

Heh...because I didn't know about that function which is much nicer than what I wrote. I'll update this PR to use it when I get a chance in the next few days. Thanks for pointing out appendStringLiteralDQ!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants