Received: from malur.postgresql.org ([2a02:16a8:dc51::56]) by arkaria.postgresql.org with esmtps (TLS1.2:ECDHE_RSA_AES_256_CBC_SHA384:256) (Exim 4.89) (envelope-from ) id 1fq0EP-000417-Vn for pgsql-hackers@arkaria.postgresql.org; Wed, 15 Aug 2018 18:09:18 +0000 Received: from localhost ([127.0.0.1] helo=malur.postgresql.org) by malur.postgresql.org with esmtp (Exim 4.89) (envelope-from ) id 1fq0EN-0003Ga-VU for pgsql-hackers@arkaria.postgresql.org; Wed, 15 Aug 2018 18:09:15 +0000 Received: from makus.postgresql.org ([2001:4800:1501:1::229]) by malur.postgresql.org with esmtps (TLS1.2:ECDHE_RSA_AES_256_CBC_SHA384:256) (Exim 4.89) (envelope-from ) id 1fq0EN-0003GS-L0 for pgsql-hackers@lists.postgresql.org; Wed, 15 Aug 2018 18:09:15 +0000 Received: from sss.pgh.pa.us ([66.207.139.130]) by makus.postgresql.org with esmtps (TLS1.2:ECDHE_RSA_AES_256_CBC_SHA384:256) (Exim 4.89) (envelope-from ) id 1fq0EK-0007Hs-U9 for pgsql-hackers@lists.postgresql.org; Wed, 15 Aug 2018 18:09:14 +0000 Received: from sss1.sss.pgh.pa.us (localhost [127.0.0.1]) by sss.pgh.pa.us (8.14.4/8.14.4) with ESMTP id w7FI98iD017668; Wed, 15 Aug 2018 14:09:08 -0400 From: Tom Lane To: David Steele cc: Robert Haas , Andres Freund , PostgreSQL Hackers , Stephen Frost Subject: Re: C99 compliance for src/port/snprintf.c In-reply-to: <6ae46827-45de-b3e7-4f19-01f9c48b28d4@pgmasters.net> References: <17245.1534289329@sss.pgh.pa.us> <9456.1534347706@sss.pgh.pa.us> <20180815155237.pz5fgfmdo4cgqjta@alap3.anarazel.de> <11138.1534349838@sss.pgh.pa.us> <6ae46827-45de-b3e7-4f19-01f9c48b28d4@pgmasters.net> Comments: In-reply-to David Steele message dated "Wed, 15 Aug 2018 14:03:16 -0400" MIME-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-ID: <17666.1534356548.1@sss.pgh.pa.us> Date: Wed, 15 Aug 2018 14:09:08 -0400 Message-ID: <17667.1534356548@sss.pgh.pa.us> List-Id: List-Help: List-Subscribe: List-Post: List-Owner: List-Archive: Precedence: bulk David Steele writes: > On 8/15/18 12:17 PM, Tom Lane wrote: >> Robert Haas writes: >>> Personally, I'd prefer to >>> continue avoiding // comments and intermingled declarations of >>> variables and code on grounds of style and readability. >> ... which I agree with. > We already have -Wdeclaration-after-statement to prevent mixed > declarations. Not sure what to do about comments except manual enforcement. pgindent will change // comments to /* style, so at least on the timescale of a release cycle, we have enforcement for that. regards, tom lane