Re: [PATCH 24/25]: Macro for moving average

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



|  I don't like macros that receive as a parameter something that will be
|  changed, its more readable to have it as a inline function and return
|  the new value, even if the resulting code uses more characters, so I'm
|  applying the attached patch and will take care of the fallout, that is
|  small from what I grepped at the_whole_lot.
You are right (please see other comment), I wasn't overly happy with this one.
Your change makes it much better and avoids future problems due to someone
misinterpreting the macro arguments. Many thanks indeed.
-
To unsubscribe from this list: send the line "unsubscribe dccp" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at  http://vger.kernel.org/majordomo-info.html

[Index of Archives]     [Linux Kernel]     [IETF DCCP]     [Linux Networking]     [Git]     [Security]     [Linux Assembly]     [Bugtraq]     [Yosemite]     [MIPS Linux]     [ARM Linux]     [Linux Security]     [Linux RAID]     [Linux SCSI]

  Powered by Linux