Michael Paquier <michael.paquier@gmail.com> writes: > On Sat, Jan 23, 2016 at 1:25 AM, Marko Tiikkaja <marko@joh.to> wrote: > + errmsg("identifier contains disallowed chars"), > + errdetail("string \"%s\" is not valid identifier", > + text_to_cstring(qualname)))); > Perhaps, "identifier contains not allowed character" is better?
"disallowed" reads better to me. I agree with expanding "chars" to "characters" though. Also, the errdetail is conveying no actual extra detail AFAICS. I'd go with something like