On 6/13/07, Alan Stern <[EMAIL PROTECTED]> wrote: > On Wed, 13 Jun 2007, Grant Likely wrote: > > > On 6/12/07, Peter Korsgaard <[EMAIL PROTECTED]> wrote: > > > >>>>> "Grant" == Grant Likely <[EMAIL PROTECTED]> writes: > > > > > > Hi, > > > > > > Grant> Rather than c67x00-hub.c being compiled seperately, the > > > Grant> original code had c67x00-hub.c *included* by c67x00-hcd.c. > > > Grant> This is a very bad idea. > > What's so bad about it? It's an elegant solution to the problem of > breaking a very long driver up into smaller, more digestible pieces > without polluting the kernel's namespace with lots of extra global > symbols.
Primarily because it breaks convention. Convention is that you #include .h files, and you compile and link .c files. Convention is important because it reflects the common patterns we use when reading and writing (but mostly reading) code. Yes there are exceptions, and yes it can be done, but there better be a damn good reason for doing so. In this particular case, I really don't think it is warranted. We're not talking about a great deal of code, and we're *already* polluting the kernel namespace with c67x00_* function names because the driver is already in multiple pieces. This issue has also come up on the LKML also. See this thread: http://thread.gmane.org/gmane.linux.kernel/498633 > What's so ugly about breaking a driver up into pieces? Leaving it in > one giant piece would be much more ugly IMO. Breaking into pieces: Good, and I fully agree. Doing it in non-standard way: Not so good as it trades one kind of ugliness for another. Cheers, g. -- Grant Likely, B.Sc., P.Eng. Secret Lab Technologies Ltd. [EMAIL PROTECTED] (403) 399-0195 ------------------------------------------------------------------------- This SF.net email is sponsored by DB2 Express Download DB2 Express C - the FREE version of DB2 express and take control of your XML. No limits. Just data. Click to get it now. http://sourceforge.net/powerbar/db2/ _______________________________________________ linux-usb-devel@lists.sourceforge.net To unsubscribe, use the last form field at: https://lists.sourceforge.net/lists/listinfo/linux-usb-devel
