From mboxrd@z Thu Jan 1 00:00:00 1970 From: Phillip Lougher Subject: Re: Subject: [PATCH 01/16] Squashfs: inode operations Date: Tue, 21 Oct 2008 01:32:43 +0100 Message-ID: <48FD232B.10508@lougher.demon.co.uk> References: <20081017165300.GA8076@logfs.org> Mime-Version: 1.0 Content-Transfer-Encoding: QUOTED-PRINTABLE Return-path: In-Reply-To: <20081017165300.GA8076@logfs.org> Sender: linux-embedded-owner@vger.kernel.org List-ID: Content-Type: text/plain; charset="iso-8859-1"; format="flowed" To: =?UTF-8?B?SsO2cm4gRW5nZWw=?= Cc: akpm@linux-foundation.org, linux-embedded@vger.kernel.org, linux-fsdevel@vger.kernel.org, linux-kernel@vger.kernel.org, tim.bird@am.sony.com J=C3=B6rn Engel wrote: > None of the comments below are a reason against mainline inclusion, i= mo. > They should get handled, but whether that happens before or after a > merge doesn't really matter. Yeah you're right regarding your comments. That's where code-review=20 comes in handy, to spot things you don't see because you're too used to= =20 the code. I'm working on a re-spin incorporating your comments, and it should be=20 ready tomorrow. Thanks for the code-review. Phillip