On Tue, 16 Jan 2007, Randy Dunlap wrote: > On Tue, 16 Jan 2007 03:36:16 -0500 (EST) Robert P. J. Day wrote: > > > On Tue, 16 Jan 2007, Ahmed S. Darwish wrote: > > > > > Use ARRAY_SIZE macro in pvrusb2-hdw.c file > > > > > > Signed-off-by: Ahmed S. Darwish <[EMAIL PROTECTED]> > > > > ... snip ... > > > > i'm not sure it's worth submitting multiple patches to convert > > code expressions to the ARRAY_SIZE() macro since i was going to > > wait for the next kernel release, and do that in one fell swoop > > with a single patch. > > > > but if people higher up the food chain think it's a better idea to > > do it a little at a time, that's fine. > > I'm not strictly on the food chain, but these 4 patches to pvrusb2 > should have been sent as one patch IMO.
it's not like i'm trying to be territorial or anything. if someone wants to handle all these changes, go wild. it's just that this discussion of ARRAY_SIZE started back in december: http://lkml.org/lkml/2006/12/17/85 i have a check for possibilities where this change can be done in my "style" script: $ grep -Er "sizeof ?\(?([^\)]+)\)? ?/ ?sizeof ?\(?.*\1.*" . and i also have a script that crawls the tree, making the more obvious changes. (it will miss spots where the expression is broken over two lines, stuff like that, so there will probably be a few stragglers that will have to be done manually.) i haven't done anything with this lately as i was waiting for the new release, whereupon i'm assuming the floodgates will be opened and all the pending changes will finally start to wander upstream. then all these changes might make sense as a multi-part patch. but, as i said, i'll leave this to people much smarter than me. rday - 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/
