Andres Freund wrote: > EEO_SWITCH(op->opcode) > { > EEO_CASE(EEO_DONE): > goto out;
Oh my. > which is a bit annoying. (the EEO_CASE is either a jump label or a case > statement, depending on computed goto availability). > > It seems we could either: > 1) live with the damage > 2) disable pgindent > 3) move the : inside EEO_CASE's definition, and only use {} blocks. I think 3) is nasty because the result doesn't look like normal C. If EEO_CASE() are potentially jump labels, then indentation becomes correct. Why not accept it? It looks odd, but that gives a better hint to the reader about what's going on. -- Álvaro Herrera https://www.2ndQuadrant.com/ PostgreSQL Development, 24x7 Support, Remote DBA, Training & Services -- Sent via pgsql-hackers mailing list (pgsql-hackers@postgresql.org) To make changes to your subscription: http://www.postgresql.org/mailpref/pgsql-hackers