Re: [PATCH 1/4] xfrm_hash: kmalloc + memset conversion to kzalloc

• To: "Robert P. J. Day" <[email protected]>
• Subject: Re: [PATCH 1/4] xfrm_hash: kmalloc + memset conversion to kzalloc
• From: "Joonwoo Park" <[email protected]>
• Date: Mon, 26 Nov 2007 19:28:28 +0900
• Cc: [email protected], [email protected]
• Domainkey-signature: a=rsa-sha1; c=nofws; d=gmail.com; s=gamma; h=received:message-id:date:from:to:subject:cc:in-reply-to:mime-version:content-type:content-transfer-encoding:content-disposition:references; b=ogR+s1a6jRuw5oRr8ljA/ROj0NJHoVEyyOaIX3IYsFAHWBLa1u+lyCGwbVRVIUxpvAtWbM5fnQVDoarXW80W8zG4mO8A62fMwqcJvE6HXBlPEkECf+XEWWipHdxj8x3STi63ALm6pCZf1ftnJhpdCuem6uHGvmh3S0Tj/LyKqxA=
• In-reply-to: <[email protected]>
• References: <007c01c8300b$176975a0$9c94fea9@jason> <[email protected]>
• Sender: [email protected]

>
> i believe the more common standard for the above is:
>
>  else if (hashdist) {
>
> to reduce the level of overall indentation, no?
>

No, it was.
Because there was a memset in that indentation, but I made it by
removing memset.

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

---

• References:
  • [PATCH 1/4] xfrm_hash: kmalloc + memset conversion to kzalloc
    • From: "Joonwoo Park" <[email protected]>
  • Re: [PATCH 1/4] xfrm_hash: kmalloc + memset conversion to kzalloc
    • From: "Robert P. J. Day" <[email protected]>

• Prev by Date: Re: [Patch] mm/sparse.c: Improve the error handling for sparse_add_one_section()
• Next by Date: Re: [PATCH] [2.6.24-rc3-mm1] loop cleanup in fs/namespace.c - repost
• Previous by thread: Re: [PATCH 1/4] xfrm_hash: kmalloc + memset conversion to kzalloc
• Next by thread: Re: [PATCH 1/4] xfrm_hash: kmalloc + memset conversion to kzalloc
• Index(es):
  • Date
  • Thread

[Index of Archives]     [Kernel Newbies]     [Netfilter]     [Bugtraq]     [Photo]     [Stuff]     [Gimp]     [Yosemite News]     [MIPS Linux]     [ARM Linux]     [Linux Security]     [Linux RAID]     [Video 4 Linux]     [Linux for the blind]     [Linux Resources]