]> granicus.if.org Git - postgresql/commit
Minor code review for parse_phrase_operator().
authorTom Lane <tgl@sss.pgh.pa.us>
Mon, 26 Jun 2017 14:31:10 +0000 (10:31 -0400)
committerTom Lane <tgl@sss.pgh.pa.us>
Mon, 26 Jun 2017 14:31:10 +0000 (10:31 -0400)
commit00c5e511b94059396150c406f5d71598034a2061
tree88acefe1bcff0b123d45e3b1dee34c0c531d428d
parent59cd3987afd61191483a4cfe8f6a0abfb8c878d6
Minor code review for parse_phrase_operator().

Fix its header comment, which described the old behavior of the <N>
phrase distance operator; we missed updating that in commit 028350f61.
Also, reset errno before strtol() call, to defend against the possibility
that it was already ERANGE at entry.  (The lack of complaints says that
it generally isn't, but this is at least a latent bug.)  Very minor
stylistic improvements as well.

Victor Drobny noted the obsolete comment, I noted the errno issue.
Back-patch to 9.6 where this code was added, just in case the errno
issue is a live bug in some cases.

Discussion: https://postgr.es/m/2b5382fdff9b1f79d5eb2c99c4d2cbe2@postgrespro.ru
src/backend/utils/adt/tsquery.c