On Thu, Jul 23, 2009 at 07:59:29PM +0800, Marek Lindner wrote:
On Saturday 18 July 2009 22:32:50 Andrew Lunn wrote:
Make aggregation.[ch] clean with respect to the 2.6.29 checkpatch script.
Signed-off-by: Andrew Lunn andrew.lunn@ascom.ch
I applied your patch. I noticed you created a function for the calculcation of the hna length. It does not really do much. Would it make sense to transform it into a macro ?
The advantage of a function is you get full type checking. In terms of code size, it makes no difference between a macro and a function. Since it is defined before it is used gcc can and probably will inline it.
While reviewing the code I saw multiple incarnations of the variable own_packet. Sometimes it is char, uint8_t or int. Not sure what effect it might have but I think it would be better to unify that ?!
Sure. At the moment i'm just getting the code look O.K. Then i can start to see if type need sorting out, etc.
Andrew