Robert Haas <[email protected]> writes:
> + if (!strcmp(oprName, "=>"))
BTW, project standard is to spell that like
> + if (strcmp(oprName, "=>") == 0)
The other way looks confusingly like a "not equal" test.
> + (errmsg("The use of => as an operator name is
> deprecated and may be disallowed in a future release.")));
Also, this doesn't follow message style guidelines. Possibly better:
errmsg("=> is deprecated as an operator name"),
errdetail("This name may be disallowed altogether in future versions of
PostgreSQL.")
regards, tom lane
--
Sent via pgsql-hackers mailing list ([email protected])
To make changes to your subscription:
http://www.postgresql.org/mailpref/pgsql-hackers
