From mboxrd@z Thu Jan 1 00:00:00 1970 From: SF Markus Elfring Date: Tue, 17 Oct 2017 09:11:20 +0000 Subject: Re: [PATCH 4/4] char/snsc: Delete unnecessary braces in five functions Message-Id: List-Id: References: <20171017084919.oa462je5y45otrw7@mwanda> In-Reply-To: <20171017084919.oa462je5y45otrw7@mwanda> MIME-Version: 1.0 Content-Type: text/plain; charset="utf-8" Content-Transfer-Encoding: 8bit To: Dan Carpenter , linux-doc@vger.kernel.org Cc: kernel-janitors@vger.kernel.org, Arnd Bergmann , Greg Kroah-Hartman , LKML >> /* ...or if we want to block, then do so here */ >> - if (down_interruptible(&sd->sd_wbs)) { >> + if (down_interruptible(&sd->sd_wbs)) >> /* something went wrong with wait */ >> return -ERESTARTSYS; >> - } >> } > > Multi-line indents get parenthesis. How do you think about to add such an expectation to the section “3) Placing Braces and Spaces” in the document “coding-style.rst”? Regards, Markus