Seems reasonable.  For the last hunk in your patch, though, I would add
a /* translator: */ comment explaining what each of the values is;
otherwise it's just incomprehensible percent-sign-salad for the poor
translator.  Note that if the line is too long you have to use dirty
tricks to avoid pgindent from messing it up (see 673b52753489 for

I'm not sure if we should keep "blk" in the original or make it a
complete word.

Álvaro Herrera      
PostgreSQL Development, 24x7 Support, Remote DBA, Training & Services

Sent via pgsql-hackers mailing list (
To make changes to your subscription:

Reply via email to