Re: [PATCH v4 04/11] xfsprogs: Add ifdef dirent checks where it was miss

To:	Christoph Hellwig <hch@xxxxxxxxxxxxx>
Subject:	Re: [PATCH v4 04/11] xfsprogs: Add ifdef dirent checks where it was missing
From:	Jan Tulak <jtulak@xxxxxxxxxx>
Date:	Tue, 4 Aug 2015 10:19:40 -0400 (EDT)
Cc:	xfs@xxxxxxxxxxx
Delivered-to:	xfs@xxxxxxxxxxx
In-reply-to:	<20150804111812.GD18906@xxxxxxxxxxxxx>
References:	<1438612789-17486-5-git-send-email-jtulak@xxxxxxxxxx> <1438680314-23950-1-git-send-email-jtulak@xxxxxxxxxx> <20150804111812.GD18906@xxxxxxxxxxxxx>
Thread-index:	/v3s+UanmCD+CLaT2Kj94hyqO5KtcA==
Thread-topic:	xfsprogs: Add ifdef dirent checks where it was missing

----- Original Message -----
> From: "Christoph Hellwig" <hch@xxxxxxxxxxxxx>
> 
> Needs to be fixed to avoid > 80 char lines.
> 

I don't see any >80 char line in this patch. O_o
The longest line is 73 in the code and 62 in commit 
message. Though... maybe it can be a tab width issue? 
I use four spaces for a tab...

Cheers,
Jan


-- 
Jan Tulak
jtulak@xxxxxxxxxx

<Prev in Thread]	Current Thread	[Next in Thread>
[PATCH v3 00/11] xfsprogs: OS X partial support, Jan Tulak [PATCH v2 02/11] xfsprogs: Use glibtoolize on osx, Jan Tulak [PATCH v3 03/11] xfsprogs: undefined variable fix, Jan Tulak Re: [PATCH v3 03/11] xfsprogs: undefined variable fix, Christoph Hellwig [PATCH v2 01/11] xfsprogs: Don't Make .po files with gettext disabled, Jan Tulak [PATCH v3 04/11] xfsprogs: Add ifdef dirent checks where it was missing, Jan Tulak Re: [PATCH v3 04/11] xfsprogs: Add ifdef dirent checks where it was missing, Christoph Hellwig Re: [PATCH v3 04/11] xfsprogs: Add ifdef dirent checks where it was missing, Jan Tulak [PATCH v4 04/11] xfsprogs: Add ifdef dirent checks where it was missing, Jan Tulak Re: [PATCH v4 04/11] xfsprogs: Add ifdef dirent checks where it was missing, Christoph Hellwig Re: [PATCH v4 04/11] xfsprogs: Add ifdef dirent checks where it was missing, Jan Tulak <= Re: [PATCH v4 04/11] xfsprogs: Add ifdef dirent checks where it was missing, Dave Chinner Re: [PATCH v4 04/11] xfsprogs: Add ifdef dirent checks where it was missing, Jan Tulak [PATCH v5 04/11] xfsprogs: Add ifdef dirent checks where it was missing, Jan Tulak [PATCH v3 05/11] xfsprogs: Change OS X-specific CFLAGS/LDFLAGS, Jan Tulak Re: [PATCH v3 05/11] xfsprogs: Change OS X-specific CFLAGS/LDFLAGS, Christoph Hellwig [PATCH v3 06/11] xfsprogs: Add includes required for OS X builds, Jan Tulak Re: [PATCH v3 06/11] xfsprogs: Add includes required for OS X builds, Christoph Hellwig [PATCH 06/11] xfsprogs: Add includes required for OS X builds, Jan Tulak [PATCH v3 07/11] xfsprogs: missing and dummy calls for OS X support, Jan Tulak [PATCH v3 08/11] xfsprogs: Add fls check into autoconf, Jan Tulak

Previous by Date:	Re: XFS File system in trouble, Brian Foster
Next by Date:	Re: Bug#793495: xfsprogs fails to build on debian ppc64el, Fernando Seiti Furusato
Previous by Thread:	Re: [PATCH v4 04/11] xfsprogs: Add ifdef dirent checks where it was missing, Christoph Hellwig
Next by Thread:	Re: [PATCH v4 04/11] xfsprogs: Add ifdef dirent checks where it was missing, Dave Chinner
Indexes:	[Date] [Thread] [Top] [All Lists]