From mboxrd@z Thu Jan 1 00:00:00 1970 From: SF Markus Elfring Subject: Re: GPU-DRM-TTM: Fine-tuning for several function implementations Date: Fri, 23 Sep 2016 13:07:30 +0200 Message-ID: References: <566ABCD9.1060404@users.sourceforge.net> <4d34446f-05ad-c3ce-5d33-8fb4f25af25c@users.sourceforge.net> <05418fb1-ad66-aba3-bd8c-f6b684a83279@users.sourceforge.net> Mime-Version: 1.0 Content-Type: text/plain; charset=windows-1252 Content-Transfer-Encoding: 7bit Return-path: In-Reply-To: Sender: linux-kernel-owner@vger.kernel.org To: =?UTF-8?Q?Christian_K=c3=b6nig?= Cc: dri-devel@lists.freedesktop.org, Daniel Vetter , David Airlie , Emil Velikov , Julia Lawall , kernel-janitors@vger.kernel.org, LKML List-Id: dri-devel@lists.freedesktop.org > It's just the names like "out" or "restart" perfectly explain why the labels exists. I have got an other impression. > So they fulfill this requirement from the coding style as far as I can see. Short identifiers might look more convenient in some cases because they are quicker to type. > So why do you want to change them? 1. I suggest to select identifiers also for jump labels which are more meaningful and eventually unique for some function implementations. 2. How do you think about to add a single space character before any label? Regards, Markus