From mboxrd@z Thu Jan 1 00:00:00 1970 From: Cong Wang Subject: Re: [Patch 1/3] sysctl: refactor integer handling proc code Date: Wed, 05 May 2010 11:02:24 +0800 Message-ID: <4BE0DFC0.30309@redhat.com> References: <20100430082912.5630.82405.sendpatchset@localhost.localdomain> <20100430082925.5630.58453.sendpatchset@localhost.localdomain> Mime-Version: 1.0 Content-Type: text/plain; charset=UTF-8; format=flowed Content-Transfer-Encoding: 7bit Cc: linux-kernel@vger.kernel.org, Octavian Purdila , Eric Dumazet , penguin-kernel@i-love.sakura.ne.jp, netdev@vger.kernel.org, Neil Horman , ebiederm@xmission.com, David Miller , adobriyan@gmail.com To: Changli Gao Return-path: Received: from mx1.redhat.com ([209.132.183.28]:1025 "EHLO mx1.redhat.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1751908Ab0EEC6r (ORCPT ); Tue, 4 May 2010 22:58:47 -0400 In-Reply-To: Sender: netdev-owner@vger.kernel.org List-ID: Changli Gao wrote: > On Fri, Apr 30, 2010 at 4:25 PM, Amerigo Wang wrote: >> + if (*p == '-' && *size > 1) { >> + *neg = 1; > > As neg is bool*, you should use true and false instead of 1 and 0. > Yeah, I only corrected those lines that I touched, I should correct them all. Will fix. Thanks.