Re: [PATCH V3 mmots] fs/hfsplus/wrapper.c: replace shift loop by ilog2

From: Davidlohr Bueso
Date: Tue May 20 2014 - 12:55:38 EST


On Tue, 2014-05-20 at 18:43 +0200, Fabian Frederick wrote:
> Replace while blocksize;shift by ilog2

Fabian, please start getting into the habit of improving your
changelogs. Unlike a lot of OSS projects, the kernel community (usually)
takes pride in having good ones. You send a number of patches but rarely
explain a key concept of software development in general: *WHY* do we
need this.

Thanks,
Davidlohr

--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/