Re: Add identify decoding 4/4

From: Jeff Garzik (jgarzik@pobox.com)
Date: Wed Aug 06 2003 - 16:25:43 EST


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/



This archive was generated by hypermail 2b29 : Thu Aug 07 2003 - 22:00:35 EST