Messages in this thread | | | Date | Mon, 26 Jun 2006 11:10:22 -0700 | From | Andrew Morton <> | Subject | Re: [PATCH] Clean up the bootmem allocator |
| |
On Mon, 26 Jun 2006 10:58:11 -0700 Dave Hansen <haveblue@us.ibm.com> wrote:
> > It also removes __init tags in the header file and hopefully make it > > easier to read. > > I think I kinda like when these are present in headers. I usually > stumble across the header declarations before I do the ones in the .c > files, and it is always nice to see the header visually _matching_ > the .c file, and how the variable is intended to be used
__init in headers is pretty useless because the compiler doesn't check it, and they get out of sync relatively frequently. So it you see an __init in a header file, it's quite unreliable and you need to check the definition anyway. - To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/
| |