Messages in this thread | | | Date | Wed, 06 Aug 2003 17:25:43 -0400 | From | Jeff Garzik <> | Subject | Re: Add identify decoding 4/4 |
| |
Andries.Brouwer@cwi.nl wrote: > No. <linux/ide-identify.h> contains a lot of 1-line static inline > functions, just readable names for current magic bit checks, > and one big function ide_dump_identify_info() that is included as > > #ifdef IDE_IDENTIFY_DEBUG
Yes, I understand how the C pre-processor works, thanks ;-)
I know full well _why_ the big function is in the header; that doesn't address my point: we don't need to be putting big functions in header files. That's why libraries were invented :)
Jeff
- 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/
| |