From mboxrd@z Thu Jan 1 00:00:00 1970 From: SF Markus Elfring Date: Wed, 03 Dec 2014 13:24:58 +0000 Subject: Re: [patch] CodingStyle: add some more error handling guidelines Message-Id: <547F0F2A.3060708@users.sourceforge.net> List-Id: References: <20141202085950.GA13434@mwanda> <547F0297.6030202@users.sourceforge.net> <20141203124511.GR5048@mwanda> <547F0977.7090908@users.sourceforge.net> <20141203132002.GT5048@mwanda> In-Reply-To: <20141203132002.GT5048@mwanda> MIME-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit To: Dan Carpenter Cc: Julia Lawall , linux-doc@vger.kernel.org, linux-kernel@vger.kernel.org, kernel-janitors@vger.kernel.org, Jonathan Corbet , OGAWA Hirofumi , Coccinelle , backports@vger.kernel.org, Johannes Berg , "Luis R. Rodriguez" > Sorry. I misread your email. If the code looks like this: > > foo = kmalloc(); > if (!foo) > goto kmalloc_failed; > > The "kmalloc_failed" doesn't add any information. I find that this such a name approach would fit to your expectation of a source-oriented labeling of these identifiers. > We can see that kmalloc failed from the context. Which name pattern do you find more appropriate in such an use case? Regards, Markus