Re: [PATCH] UDF: check for allocated memory for inode data

[Pekka Enberg - Sat, May 12, 2007 at 01:15:14PM +0300]
| Hi Cyrill
| 
| On 5/12/07, Cyrill Gorcunov <[email protected]> wrote:
| >you know I've got a question (may be it's stupid) - what a
| >sense to discard UDF_I_* macroses? I mean as I see they
| >don't slow down execution of the code...
| 
| They make the code harder to read and maintain.
| 

Hi Pekka,

thanks for eplanation. Btw, maybe I also should
convert the UDF coding style to Linux kernel coding
style?

		Cyrill

P.S.
It's strange that I haven't got any comments from
Ben Fennema :(

-
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] UDF: check for allocated memory for inode data
  - From: Cyrill Gorcunov <[email protected]>
- Re: [PATCH] UDF: check for allocated memory for inode data
  - From: Andrew Morton <[email protected]>
- Re: [PATCH] UDF: check for allocated memory for inode data
  - From: Christoph Hellwig <[email protected]>
- Re: [PATCH] UDF: check for allocated memory for inode data
  - From: Cyrill Gorcunov <[email protected]>
- Re: [PATCH] UDF: check for allocated memory for inode data
  - From: Christoph Hellwig <[email protected]>
- Re: [PATCH] UDF: check for allocated memory for inode data
  - From: Cyrill Gorcunov <[email protected]>
- Re: [PATCH] UDF: check for allocated memory for inode data
  - From: "Pekka Enberg" <[email protected]>

Prev by Date: Re: [patch 3/3] clockevents: Fix resume logic - updated version
Next by Date: Re: [PATCH] Add LZO1X compression support to the kernel
Previous by thread: Re: [PATCH] UDF: check for allocated memory for inode data
Next by thread: Kernel BUG: NULL pointer dereference , reference to sys_recvmsg
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]