aboutsummaryrefslogtreecommitdiff
path: root/src/backend/parser/parse_clause.c
diff options
context:
space:
mode:
authorTom Lane <tgl@sss.pgh.pa.us>2016-03-28 01:00:30 -0400
committerTom Lane <tgl@sss.pgh.pa.us>2016-03-28 01:00:30 -0400
commitd12e5bb79bb535c2df13b76cd7d01f0bb8dc8e4d (patch)
treefffb6e2f681baf125f307fc8519175bc529ecb34 /src/backend/parser/parse_clause.c
parent499a50571c72f41bb1365970d55dae5c8afcb6ba (diff)
downloadpostgresql-d12e5bb79bb535c2df13b76cd7d01f0bb8dc8e4d.tar.gz
postgresql-d12e5bb79bb535c2df13b76cd7d01f0bb8dc8e4d.zip
Code and docs review for commit 3187d6de0e5a9e805b27c48437897e8c39071d45.
Fix up check for high-bit-set characters, which provoked "comparison is always true due to limited range of data type" warnings on some compilers, and was unlike the way we do it elsewhere anyway. Fix omission of "$" from the set of valid identifier continuation characters. Get rid of sanitize_text(), which was utterly inconsistent with any other error report anywhere in the system, and wasn't even well designed on its own terms (double-quoting the result string without escaping contained double quotes doesn't seem very well thought out). Fix up error messages, which didn't follow the message style guidelines very well, and were overly specific in situations where the actual mistake might not be what they said. Improve documentation. (I started out just intending to fix the compiler warning, but the more I looked at the patch the less I liked it.)
Diffstat (limited to 'src/backend/parser/parse_clause.c')
0 files changed, 0 insertions, 0 deletions