On Wed, Feb 27, 2013 at 10:42:23PM +0100, walter harms wrote: > but i am missing the point why where are two counters. > i is simple. it is used to check the bitfield hw->channel_info.results.scanchannels > n is increased only the when a bit is set. So it would be more easy to simply > count the bits and run the loop about that number. > But i can also imagine that the bitfield act as "enable" and the author actualy > should read &inf->info.chinforesult.result[i]; > > perhaps i am missing the point, could someone tell me were i am wrong ? > I don't understand how you're confused. You've described the code perfectly. One is incremented for every iteration and one is only incremented if we find what we're looking for. Anyway, the point of the cleanup was just so I could add a range check in patch 2/2 but the original code was too ugly to look at. regards, dan carpenter -- To unsubscribe from this list: send the line "unsubscribe kernel-janitors" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html
