From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1754657AbZKHQ4d (ORCPT ); Sun, 8 Nov 2009 11:56:33 -0500 Received: (majordomo@vger.kernel.org) by vger.kernel.org id S1754534AbZKHQ4c (ORCPT ); Sun, 8 Nov 2009 11:56:32 -0500 Received: from borg.medozas.de ([188.40.89.202]:43780 "EHLO borg.medozas.de" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1754507AbZKHQ4c (ORCPT ); Sun, 8 Nov 2009 11:56:32 -0500 Date: Sun, 8 Nov 2009 17:56:37 +0100 (CET) From: Jan Engelhardt To: =?UTF-8?Q?Andr=C3=A9_Goddard_Rosa?= cc: linux-kernel@vger.kernel.org, James.Bottomley@hansenpartnership.com Subject: Re: [PATCH v4 10/12] string: factorize skip_spaces and export it to be generally available In-Reply-To: Message-ID: References: User-Agent: Alpine 2.00 (LSU 1167 2008-08-23) MIME-Version: 1.0 Content-Type: TEXT/PLAIN; charset=UTF-8 Content-Transfer-Encoding: 8BIT Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org On Saturday 2009-11-07 17:30, André Goddard Rosa wrote: >+char *skip_spaces(const char *str) >@@ -352,10 +366,7 @@ char *strstrip(char *s) > end--; > *(end + 1) = '\0'; > >- while (*s && isspace(*s)) >- s++; >- >- return s; >+ return (char *)skip_spaces(s); > } sparse would probably say "how much more casting to the same type do you want?" It is already char*.