From: Robert Yang Subject: Re: [PATCH] create_inode: fix gcc -Wall complaints Date: Wed, 12 Mar 2014 14:34:48 +0800 Message-ID: <53200008.60107@windriver.com> References: <1394595670-11193-1-git-send-email-tytso@mit.edu> <20140312034539.GC26898@thunk.org> <20140312034841.GC31864@birch.djwong.org> <531FFFBD.1010503@windriver.com> Mime-Version: 1.0 Content-Type: text/plain; charset="ISO-8859-1"; format=flowed Content-Transfer-Encoding: 7bit Cc: Ext4 Developers List To: "Darrick J. Wong" , "Theodore Ts'o" Return-path: Received: from mail.windriver.com ([147.11.1.11]:64902 "EHLO mail.windriver.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1755226AbaCLGey (ORCPT ); Wed, 12 Mar 2014 02:34:54 -0400 In-Reply-To: <531FFFBD.1010503@windriver.com> Sender: linux-ext4-owner@vger.kernel.org List-ID: On 03/12/2014 02:33 PM, Robert Yang wrote: > > > On 03/12/2014 11:48 AM, Darrick J. Wong wrote: >> On Tue, Mar 11, 2014 at 11:45:39PM -0400, Theodore Ts'o wrote: >>> On Tue, Mar 11, 2014 at 11:41:10PM -0400, Theodore Ts'o wrote: >>>> We had several functions that were not returning zero on success. Fix >>>> this. >>>> >>>> Signed-off-by: "Theodore Ts'o" >>> >>> One additional gcc -Wall nit which I have not yet fixed: >>> >>> create_inode.c is using ext2fs_inline_data_init() which is currently a >>> private function defined in ext2fsP.h. >>> >>> We either need to make this a publically exported interface (in which >>> case we have to guarantee that it is stable), or we need to find >>> another way to make the right thing happen here. >>> >>> Darrick, Robert, do you have any thoughts or suggestions? >> >> I fixed that up in the cppcheck cleanups patch by moving those declarations to >> ext2fs.h. Maybe I should have shoved the patch closer to the head. >> > > He Darrick and Ted, > s/He/Hi/ Sorry for the typo. // Robert > > Thank you very much! > > // Robert > >> --D >>> >>> - Ted >>> >>> P.S. My bad for not doing a gcc -Wall run on the patches before >>> accepting them. And in the future, I'd appreciate it if people who >>> are preparing patches do a "make gcc-wall" and make sure you're not >>> making things worse. There are plenty of bugs that can be turned up >>> this way. >>> -- >>> To unsubscribe from this list: send the line "unsubscribe linux-ext4" in >>> the body of a message to majordomo@vger.kernel.org >>> More majordomo info at http://vger.kernel.org/majordomo-info.html >> >> > -- > To unsubscribe from this list: send the line "unsubscribe linux-ext4" in > the body of a message to majordomo@vger.kernel.org > More majordomo info at http://vger.kernel.org/majordomo-info.html > >