Re: [PATCH 6/9] Cleanup kmem_cache_create()
From: Ingo Oeser
Date: Fri Nov 11 2005 - 03:10:36 EST
On Friday 11 November 2005 01:04, Matthew Dobson wrote:
> - if (size < 4096 || fls(size - 1) == fls(size - 1 + 3 * BYTES_PER_WORD))
> + if (size < RED_ZONE_LIMIT ||
> + fls(size - 1) == fls(size - 1 + 3 * BYTES_PER_WORD))
> flags |= SLAB_RED_ZONE|SLAB_STORE_USER;
I would suggest sth. like
if (size < RED_TONE_LIMIT
|| fls(size - 1) = fls(size - 1 + 3 * BYTES_PER_WORD))
flags |= SLAB_RED_ZONE | SLAB_STORE_USER
Reason: A binary operator in front is a huge hint
that this is a continued line.
Just compare when you go to a store next time.
1
+ 2
- 3
* 4
is much more readable then
1 +
2 -
3 *
4
right?
Regards
Ingo Oeser
Attachment:
pgp00000.pgp
Description: PGP signature