From: Frank Seidel <frank@f-seidel.de>
To: Rabin Vincent <rabin@rab.in>
Subject: Re: [PATCH] Remove unneeded code in sys_getpriority
Date: Sun, 3 Feb 2008 19:10:28 +0100
User-Agent: KMail/1.9.5
Cc: Andrew Morton <akpm@linux-foundation.org>,
       Linux Kernel <linux-kernel@vger.kernel.org>
References: <20080203030429.GB7459@ubuntu> <200802031054.45604.frank@f-seidel.de> <20080203175854.GB3124@ubuntu> (sfid-20080203_190515_304540_24F4CBC029C088AF)
In-Reply-To: <20080203175854.GB3124@ubuntu>
MIME-Version: 1.0
Content-Type: text/plain;
  charset="iso-8859-1"
Content-Transfer-Encoding: 7bit
Content-Disposition: inline
Message-Id: <200802031910.28621.frank@f-seidel.de>
Sender: linux-kernel-owner@vger.kernel.org
Content-Length: 744
Lines: 17

On Sunday 03 February 2008 18:58, Rabin Vincent wrote:
> Do you mean the PRIO_* cases in the switch?  They're still independent
> of position after the patch because they don't fall through.

Yes, sure, this is fully correct now. Just if somehting whatsoever
is put ahead touching retval one need to take care of this here.
But as i don't have enough experience to know if this likely at all i'm
fully fine with it. Just wanted to note one should think about if
its worth the change.

Thanks,
Frank
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/